Description Tobias Kortkamp 2019-11-02 05:55:15 UTC

Created attachment 208787 [details]
ponyc.diff

Small update to ponyc 0.33.0

This also adds a new STATIC default option to link with LLVM
statically.  Upstream now supports it and we can avoid having to
have llvm70 installed at runtime just for ponyc.  Installed size
goes up from 6 MiB to 50 MiB which I think is acceptable.

patch-Makefile-ponyc is to avoid the following problem at runtime
when llvm70 is not installed i.e. with STATIC=on:

Warning: environment variable $CC undefined, using /usr/local/bin/clang70 as the linker
Error:
unable to link: /usr/local/bin/clang70 -o ./helloworld -O3 -march=core2 -mcx16   ./helloworld.o -L"/usr/local/lib/pony/0.33.0/bin/" -Wl,-rpath,"/usr/local/lib/pony/0.33.0/bin/" -L"/usr/local/lib/pony/0.33.0/bin/../lib/core2" -Wl,-rpath,"/usr/local/lib/pony/0.33.0/bin/../lib/core2" -L"/usr/local/lib/pony/0.33.0/bin/../packages" -Wl,-rpath,"/usr/local/lib/pony/0.33.0/bin/../packages" -L"/usr/local/lib" -Wl,-rpath,"/usr/local/lib" -Wl,--start-group -Wl,--end-group  -lpthread  -lponyrt  -lm -lexecinfo

PONY_COMPILER seems to only be used to invoke the linker, so hardcoding
/usr/bin/cc as a workaround is hopefully fine.  This is mostly a problem
because CC is overwritten in the port.

Builds fine in Poudriere on 11.3 amd64, 12.0 amd64.