Page MenuHomeFreeBSD
Differential D665

Add option helper for PLIST_SUB
ClosedPublic
Actions

Authored by antoine on Aug 21 2014, 2:12 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 5, 4:40 PM
Unknown Object (File)
Sat, Nov 2, 9:43 PM
Unknown Object (File)
Oct 18 2024, 2:31 AM
Unknown Object (File)
Sep 24 2024, 3:56 AM
Unknown Object (File)
Sep 23 2024, 6:58 AM
Unknown Object (File)
Sep 16 2024, 9:37 PM
Unknown Object (File)
Sep 7 2024, 7:47 PM
Unknown Object (File)
Sep 5 2024, 2:45 AM
View All Files
Subscribers
koobs

Details

Reviewers
mat
Group Reviewers
portmgr
Commits
rP365573: Add an option helper for PLIST_SUB
Summary

Add option helper for PLIST_SUB
Suggested by koobs

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

antoine updated this revision to Diff 1198.Aug 21 2014, 2:12 PM
antoine retitled this revision from to Add option helper for PLIST_SUB.
antoine updated this object.
antoine edited the test plan for this revision. (Show Details)
antoine added a reviewer: koobs.
Herald added a reviewer: portmgr. · View Herald TranscriptAug 21 2014, 2:12 PM
koobs edited edge metadata.Aug 21 2014, 2:35 PM

This is super awesome, thanks for creating this @antoine!

Will help remove this kind of thing:

.if ${PORT_OPTIONS:MPYTHON}
PLIST_SUB+=     PYTHON_VER="${PYTHON_VER}" VERSION="${PORTVERSION}"
.endif
koobs accepted this revision.Aug 21 2014, 2:35 PM
koobs edited edge metadata.
mat accepted this revision.Aug 21 2014, 4:07 PM
mat added a reviewer: mat.
This revision is now accepted and ready to land.Aug 21 2014, 4:07 PM
mat added a comment.Aug 21 2014, 4:30 PM

Please, also add it to the porter's handbook, in head/en_US.ISO8859-1/books/porters-handbook/makefiles/chapter.xml about line 4188 (and add INFO too, it kinda crept through.)

antoine planned changes to this revision.Aug 21 2014, 4:54 PM

Mk/bsd.apache.mk:${module}_PLIST_SUB= "@comment "
Mk/bsd.apache.mk:${module}_PLIST_SUB= ""
Mk/bsd.apache.mk:PLIST_SUB+= MOD_${module}=${${module}_PLIST_SUB}
x11-themes/slim-themes/Makefile:NEW_PLIST_SUB:= ${PLIST_SUB} ${n}=""
x11-themes/slim-themes/Makefile:PLIST_SUB:= ${NEW_PLIST_SUB}
x11-themes/slim-themes/Makefile:NEW_PLIST_SUB:= ${PLIST_SUB} ${n}="@comment "
x11-themes/slim-themes/Makefile:PLIST_SUB:= ${NEW_PLIST_SUB}

antoine updated this revision to Diff 1202.Aug 21 2014, 5:37 PM
antoine edited edge metadata.

Rename a few variables to avoid confusion

This revision is now accepted and ready to land.Aug 21 2014, 5:37 PM
koobs added a subscriber: koobs.Aug 21 2014, 5:39 PM

Unqualified to review this shell magic. Taking my name off reviewers (previous change accepted).

koobs resigned from this revision.Aug 21 2014, 5:39 PM
koobs removed a reviewer: koobs.
antoine closed this revision.Aug 21 2014, 6:07 PM
antoine updated this revision to Diff 1203.

Closed by commit rP365573 (authored by @antoine).

Revision Contents
Changeset List

PathSize
Mk/
4 lines

Diff 1198

View Options

Mk/bsd.options.mk

Loading...