math/R: set SONAME for some R libraries
ClosedPublic
Actions

Authored by jrm on Jul 17 2017, 1:17 AM.

Details

Reviewers

dbn
pawel

Commits

rP446199: math/R: set SONAME for libraries under lib/R/lib

Summary

These are some suggested tweaks to the patch for math/R submitted by pawel@
in Bug 219776.

Add links in ${PREFIX}/lib for the libraries that we are using as system libraries.
Set LIBVER to ${PORTVERSION:R}, i.e., R's major_version.minor_version

Repository

rP FreeBSD ports repository

Lint

No Lint Coverage

Unit

No Test Coverage

Build Status

Buildable 10517
Build 10926: arc lint + arc unit

Minor tweak to how the links in ${PREFIX}/lib are created.

math/R/Makefile
149–159	I wonder if this should not be replaced by USE_LDCONFIG= ${PREFIX}/lib/R/lib

math/R/Makefile
149–159	Upstream makes a big deal about the libraries under lib/R/lib/ not being system libraries. If you are interested in the gory details, this thread spells it out: https://stat.ethz.ch/pipermail/r-devel/2016-November/073397.html. I thought creating links under lib/ would make it clear that we are intending to make these system libraries.

math/R/Makefile
149–159	Mmmmm, but does any port actually needs this ?

math/R/Makefile
149–159	I find this additional links messy and fail to see benefits. Interested consumers can get correct linker flags from pkg-config: % pkg-config --libs libR -Wl,--export-dynamic -L/usr/local/lib -Wl,-rpath=/usr/local/lib/gcc5 -L/usr/local/lib/gcc5 -B/usr/local/bin -fstack-protector -L/usr/local/lib/R/lib -lR So my vote here goes for USE_LDCONFIG= ${PREFIX}/lib/R/lib like in my initial patch.