* On 2015-03-31 at 23:00 BST, Austin Wise wrote: > Thanks for the quick response last night! > > Lastly, I tried out clang++ and it tries to link /usr/lib/cxa_finalizer.o. > This file does not exist. It looks like bug 3849 > <https://www.illumos.org/issues/3849> added the relevant symbols and the > PKGSRC LLVM script > <https://github.com/joyent/pkgsrc/blob/trunk/lang/clang/Makefile.common#L83> > tries to get rid of the reference to cxa_finalizer. However it appears > there is an extra dollar sign on line 83 there in the sed script that > prevents the file from being properly edited.
The '$$' is correct, it's make's way of expanding the '$' and passing it to sed. However, I'd rather this was patched properly, I'm not really a fan of the way that has been implemented. > Do you guys take pull requests on that pkgsrc repo in Github? We do! Thanks, -- Jonathan Perkin - Joyent, Inc. - www.joyent.com ------------------------------------------- smartos-discuss Archives: https://www.listbox.com/member/archive/184463/=now RSS Feed: https://www.listbox.com/member/archive/rss/184463/25769125-55cfbc00 Modify Your Subscription: https://www.listbox.com/member/?member_id=25769125&id_secret=25769125-7688e9fb Powered by Listbox: http://www.listbox.com
