Opened 11 years ago

Closed 4 years ago

#6133 closed Bugs (fixed)

MacOSX compilation problem and how to fix it!

Reported by: rtelyuk@… Owned by: Joel de Guzman
Milestone: To Be Determined Component: fusion
Version: Boost 1.48.0 Severity: Problem
Keywords: fusion, iostreams, phoenix Cc:

Description

Hi boost society! I'd like to propose my fix that allow useing boost under macosx. Hope it would helpfull.

Fix for: fusion, iostreams, phoenix

p.s. Plz see reports in attachment.

Attachments (4)

FolderComparisonReport_files.zip​ (76.2 KB ) - added by rtelyuk@… 11 years ago.
diff
boost_patchs.zip​ (4.0 KB ) - added by rtelyuk@… 11 years ago.
new standart path files for fusion, iostreams, phoenix
make_diff.bat​ (2.5 KB ) - added by rtelyuk@… 11 years ago.
bath file for diff (may be useful)
boost_patchs.2.zip​ (6.3 KB ) - added by rtelyuk@… 11 years ago.
standart diff files with context (-u flag)

Download all attachments as: .zip

Change History (11)

by rtelyuk@…, 11 years ago

Attachment: FolderComparisonReport_files.zip​ added

diff

comment:1 by anonymous, 11 years ago

Version: Boost 1.47.0 → Boost 1.48.0

comment:2 by viboes, 11 years ago

Component: None → fusion
Owner: set to Joel de Guzman

comment:3 by Joel de Guzman, 11 years ago

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.

by rtelyuk@…, 11 years ago

Attachment: boost_patchs.zip​ added

new standart path files for fusion, iostreams, phoenix

comment:4 by rtelyuk@…, 11 years ago

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?

by rtelyuk@…, 11 years ago

Attachment: make_diff.bat​ added

bath file for diff (may be useful)

by rtelyuk@…, 11 years ago

Attachment: boost_patchs.2.zip​ added

standart diff files with context (-u flag)

comment:5 by Joel de Guzman, 11 years ago

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

comment:6 by sasha.kurylenko@…, 11 years ago

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

comment:7 by Kohei Takahashi, 4 years ago

Resolution: → fixed
Status: new → closed

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

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

Note: See TracTickets for help on using tickets.