It appears that Lifepillar added a patch to Brew to work around this and then Craig added the fix to Ledger itself so the patch is no longer needed. Right now I can't rebuild Ledger with Homebrew right now because the patch doesn't apply.
Life, could you please roll back your brew patch? Matthew On Wednesday, October 30, 2013 6:41:47 AM UTC-7, Craig Earls wrote: > > I pushed this change to master. > > > On Sun, Oct 27, 2013 at 9:53 AM, Lifepillar > <[email protected]<javascript:> > > wrote: > >> In article <[email protected]<javascript:> >> >, >> Ronald Ip <[email protected] <javascript:>> wrote: >> >> > On 26 Oct, 2013, at 5:09 am, Ronald Ip >> > <[email protected] <javascript:>> wrote: >> > >> > >>> Just to note: I've been building Ledger with only Clang for over a >> year >> > >>> now, >> > >>> so I suspect that it has more to do with some recent change in >> Clang. >> > >>> >> > >> >> > >> It looks like the recent change from 10.8 -> 10.9 is the switch from >> > >> /usr/lib/libstdc++.6.dylib to /usr/lib/libc++.1.dylib. >> > >> >> > >> I don’t understand C++ enough to know if it is this switch among >> others >> > >> that caused the failed build on 10.9/Clang. >> > > >> > > An FYI update, I switched the stdlib to libstdc++ just to see what >> happens >> > > and it compiles past the error. >> > > I know that libc++ and libstdc++ are not ABI compatible, so I’m not >> > > surprised that it fails to link with boost (built with libc++) in the >> > > linking stage. >> > > >> > > The flags I used: >> > > $ export CXXFLAGS="-stdlib=libstdc++"; export >> LDFLAGS="-stdlib=libstdc++” >> > > $ ./acprep opt update >> > > $ nice make >> > > >> > > So I guess, the resolution would be to 1) get code compatibility with >> > > libc++ or 2) compile everything, including dependencies with >> libstdc++. >> > >> > For what it’s worth, I found a solution. While everything looks like it >> > works, I’ve got no idea if this breaks anything else unintentionally. >> > >> > Referencing this: >> > >> http://stackoverflow.com/questions/13148513/does-stdmap-require-the-comparator >> > s-operator-to-be-const/ >> > >> > I made changes in `src/account.h` and `src/commodity.h` (see attached >> patch). >> >> Good catch! The patch works for me, too! >> >> Life >> >> -- >> >> --- >> You received this message because you are subscribed to the Google Groups >> "Ledger" group. >> To unsubscribe from this group and stop receiving emails from it, send an >> email to [email protected] <javascript:>. >> For more options, visit https://groups.google.com/groups/opt_out. >> > > > > -- > Craig, Corona De Tucson, AZ > enderw88.wordpress.com > -- --- You received this message because you are subscribed to the Google Groups "Ledger" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. For more options, visit https://groups.google.com/groups/opt_out.
