R, because R may be configured without libR.so.
ClosedPublic
Actions

Authored by jrm on Nov 27 2016, 2:30 AM.

Details

Reviewers

swills
AMDmi3
bacon4000_gmail.com

Commits

rP427397: science/afni: Use BUILD_DEPENDS/RUN_DEPENDS instead of LIB_DEPENDS for

Summary

science/afni: Use BUILD_DEPENDS/RUN_DEPENDS instead of LIB_DEPENDS for

math/R, because R may be configured without libR.so

Also, after some planned changes ${PREFIX}/lib/R/lib will not be registered
with ldconfig. While here, fix some stage-qa warnings about indirect
dependencies.

Test Plan

portlint -AC: OK
testport: OK (poudriere: 10.3-RELEASE-p10, i386)
testport: OK (poudriere: 10.3-RELEASE-p10, amd64)
testport: OK (poudriere: 11.0-RELEASE-p1, i386)
testport: OK (poudriere: 11.0-RELEASE-p1, amd64)

Diff Detail

Repository

rP FreeBSD ports repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

jrm updated this revision to Diff 22534.Nov 27 2016, 2:30 AM

jrm retitled this revision from to science/afni: Use BUILD_DEPENDS/RUN_DEPENDS instead of LIB_DEPENDS for math/R, because R may be configured without libR.so..

jrm updated this object.

jrm edited the test plan for this revision. (Show Details)

jrm added reviewers: swills, AMDmi3, bacon4000_gmail.com.

Herald added a subscriber: mat. · View Herald TranscriptNov 27 2016, 2:30 AM

jrm added a child revision: D8650: Fix build issue when the LTO option is enabled [1], do not register ${PREFIX}/lib/R/lib with ldconfig [2], and fix typo..Nov 27 2016, 3:52 AM

Patch tested and approved.

AMDmi3 added inline comments.Nov 28 2016, 2:11 PM