Page MenuHomeFreeBSD

Permit multiple packing list files to be listed in PLIST.
ClosedPublic

Authored by jhb on Nov 19 2019, 11:27 PM.
Tags
None
Referenced Files
Unknown Object (File)
Dec 4 2024, 7:53 AM
Unknown Object (File)
Nov 23 2024, 7:26 PM
Unknown Object (File)
Nov 21 2024, 7:43 AM
Unknown Object (File)
Nov 20 2024, 1:47 AM
Unknown Object (File)
Nov 15 2024, 2:20 PM
Unknown Object (File)
Nov 13 2024, 3:00 PM
Unknown Object (File)
Nov 6 2024, 6:05 PM
Unknown Object (File)
Oct 17 2024, 11:51 AM
Subscribers

Details

Summary

This permits a port to use separate packing lists for optional components
instead of depending on PLIST_SUB.

Test Plan
  • used to test a split-out pkg-plist file for devel/freebsd-gcc6

Other possible candidates include devel/binutils, base/binutils, and
base/gcc.

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

There may be better ways of doing this, this was just a first cut. You can see an example of this being used in the child revision in the stack (D22451)

I was thinking about adding a @include in plist to allow this, but this is simpler I do like it.

This revision is now accepted and ready to land.Nov 20 2019, 10:22 AM

I think this probably warrants an exp-run before committing?

In D22450#491430, @jhb wrote:

I think this probably warrants an exp-run before committing?

There are only two ports with spaces in ${PLIST}, and both of those are whitespace errors.

[adamw@apnoea /usr/ports] ag '^PLIST=.* ' 
devel/libpeas/Makefile
35:PLIST=          ${.CURDIR}/pkg-plist

devel/luafilesystem-51/Makefile
11:PLIST=          ${MASTERDIR}/pkg-plist

Unless I'm missing something, if ports in your own sample build fine, I don't see where any others would fail.

Ok, I did do some test builds of a "normal" port when testing this as well (just using default PLIST), and I do think it's true that if this broke it would break everything.