Boost C++ Libraries: Ticket #6133: MacOSX compilation problem and how to fix it!

attachment set

rtelyuk@… — Thu, 17 Nov 2011 16:17:15 GMT

attachment → FolderComparisonReport_files.zip

diff

version changed

anonymous — Tue, 29 Nov 2011 11:30:35 GMT

version Boost 1.47.0 → Boost 1.48.0

component changed; owner set

viboes — Mon, 12 Dec 2011 22:40:21 GMT

owner set to Joel de Guzman
component None → fusion

Joel de Guzman — Tue, 13 Dec 2011 00:32:35 GMT

The problem is that nil is part of the API and can break existing code that uses it. If I change it to nil_t then there will potentially be breakage. A better patch is to limit the effects to the Mac using #ifdefs. I'll gladly apply such a patch if you can prepare one.

Also, please provide a standard patch file. I can't process your FolderComparisonReport_files.

attachment set

rtelyuk@… — Tue, 13 Dec 2011 11:45:17 GMT

attachment → boost_patchs.zip

new standart path files for fusion, iostreams, phoenix

rtelyuk@… — Tue, 13 Dec 2011 11:46:41 GMT

Hi guys! Sorry for html diff. I created a new patches for fusion, iostreams, phoenix (see attached file: boost_patchs.zip) For create I use these utils: http://gnuwin32.sourceforge.net/packages/diffutils.htm Path's was created based on boost 1.47 sources.

One question. Should I create three tickets (separate for: fusion, iostreams, phoenix) or we can stay in this?

attachment set

rtelyuk@… — Tue, 13 Dec 2011 11:53:02 GMT

attachment → make_diff.bat

bath file for diff (may be useful)

attachment set

rtelyuk@… — Tue, 13 Dec 2011 12:16:27 GMT

attachment → boost_patchs.2.zip

standart diff files with context (-u flag)

Joel de Guzman — Wed, 14 Dec 2011 00:58:39 GMT

One question. Should I create three tickets (separate for: fusion, iostreams, phoenix) or we can stay in this?

Yes, please. I can take care of fusion and phoenix, but not iostreams. Also, take note that there's phoenix2[1] and phoenix3[2] now in Boost. I'll need to have patches for both.

[1] boost/spirit/phoenix [2] boost/phoenix

sasha.kurylenko@… — Tue, 13 Mar 2012 10:31:24 GMT

Please add 'fusion/container/list/cons_fwd.hpp' to your patch.

status changed; resolution set

Kohei Takahashi — Thu, 05 Jul 2018 14:13:19 GMT

status new → closed
resolution → fixed

This is already fixed in 1.55.0(fusion), 1.56.0(spirit).

btw, I couldn't find out nil from iostream and phoenix.