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, Apr 7, 4:32 AM
Unknown Object (File)
Mar 3 2024, 9:28 PM
Unknown Object (File)
Mar 3 2024, 9:28 PM
Unknown Object (File)
Mar 3 2024, 9:28 PM
Unknown Object (File)
Mar 3 2024, 9:28 PM
Unknown Object (File)
Mar 3 2024, 2:09 PM
Unknown Object (File)
Jan 11 2024, 8:40 PM
Unknown Object (File)
Jan 3 2024, 1:21 AM
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 3024
Build 3054: 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.