Page MenuHomeFreeBSD

Make SET_LATE_CONRFIGURE_ARGS work with custom CONFIGURE_CMD
ClosedPublic

Authored by jbeich on Nov 14 2015, 10:52 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Aug 17, 9:54 PM
Unknown Object (File)
Thu, Aug 14, 9:15 PM
Unknown Object (File)
Thu, Aug 14, 9:15 PM
Unknown Object (File)
Wed, Aug 13, 9:01 PM
Unknown Object (File)
Sun, Aug 3, 8:33 AM
Unknown Object (File)
Sun, Aug 3, 3:32 AM
Unknown Object (File)
Fri, Jul 25, 2:29 AM
Unknown Object (File)
Jun 19 2025, 1:55 PM
Subscribers

Details

Summary

rP383570 likely forgot to run grep(1) over Mk/*. For completeness sake I've converted everything else as well.

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 1138
Build 1142: arc lint + arc unit

Event Timeline

jbeich retitled this revision from to Make SET_LATE_CONRFIGURE_ARGS work with custom CONFIGURE_CMD.
jbeich updated this object.
jbeich edited the test plan for this revision. (Show Details)
jbeich added reviewers: portmgr, Perl.

This look good to me for the non-perl part.

jbeich edited edge metadata.

Rebased.

I don't really get what this does, but it looks harmless.

SET_LATE_CONFIGURE_ARGS is used to pass things like --mandir for GNU_CONFIGURE=yes ports but currently doesn't work with CONFIGURE_CMD (unlike CONFIGURE_SCRIPT). D4157 exposes the bug upon more consumers. Notably, lang/gcc will break without this change:

$ make check-plist
====> Checking for pkg-plist issues (check-plist)
===> Parsing plist
===> Checking for items in STAGEDIR missing from pkg-plist
Error: Orphaned: share/info/cpp.info
Error: Orphaned: share/info/cppinternals.info
Error: Orphaned: share/info/gcc.info
Error: Orphaned: share/info/gccinstall.info
Error: Orphaned: share/info/gccint.info
Error: Orphaned: share/info/gfortran.info
Error: Orphaned: share/info/libgomp.info
Error: Orphaned: share/info/libitm.info
Error: Orphaned: share/info/libquadmath.info
Error: Orphaned: share/man/man1/cpp48.1
Error: Orphaned: share/man/man1/g++48.1
Error: Orphaned: share/man/man1/gcc48.1
Error: Orphaned: share/man/man1/gcov48.1
Error: Orphaned: share/man/man1/gfortran48.1
Error: Orphaned: share/man/man7/fsf-funding.7
Error: Orphaned: share/man/man7/gfdl.7
Error: Orphaned: share/man/man7/gpl.7
===> Checking for items in pkg-plist which are not in STAGEDIR
Error: Missing: man/man1/cpp48.1.gz
Error: Missing: man/man1/g++48.1.gz
Error: Missing: man/man1/gcc48.1.gz
Error: Missing: man/man1/gcov48.1.gz
Error: Missing: man/man1/gfortran48.1.gz
===> Error: Plist issues found.

This look good to me for the non-perl part.

In D4158#122848, @mat wrote:

I don't really get what this does, but it looks harmless.

Are you giving a weak approval to be rubberstamped after exp-run?

This revision was automatically updated to reflect the committed changes.