Page MenuHomeFreeBSD
Differential D14014

Fix PLIST_SUB being too big and PLIST_SUB_SED getting bigger than _POSIX2_LINE_MAX.
ClosedPublic
Actions

Authored by mat on Jan 22 2018, 2:26 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Jul 27, 5:11 PM
Unknown Object (File)
Sun, Jul 26, 7:00 PM
Unknown Object (File)
Sat, Jul 25, 6:09 PM
Unknown Object (File)
Fri, Jul 24, 12:46 PM
Unknown Object (File)
Wed, Jul 22, 9:10 PM
Unknown Object (File)
Tue, Jul 21, 10:35 AM
Unknown Object (File)
Mon, Jul 20, 6:50 AM
Unknown Object (File)
Tue, Jul 14, 6:46 PM
View All Files
Subscribers
asomers

Details

Reviewers
asomers
Group Reviewers
portmgr
Commits
rP475361: Extract PLIST_SUB substitutions into a sed script.
Summary

And it fixes it by putting everything from PLIST_SUB_SED into a file and asking sed to read that file.

Fixes the problem from https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=222355

Diff Detail

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

Event Timeline

mat created this revision.Jan 22 2018, 2:26 PM
Harbormaster completed remote builds in B14523: Diff 38308.Jan 22 2018, 2:26 PM
mat edited the summary of this revision. (Show Details)Jan 22 2018, 2:29 PM
asomers added a subscriber: asomers.Jan 22 2018, 6:17 PM

I can no longer reproduce the original issue. Something in my environment changed and now sed's command line is just under the limit. However, i can confirm that this patch doesn't break anything at least.

mat updated this revision to Diff 45646.Jul 21 2018, 9:22 AM
mat edited the summary of this revision. (Show Details)

rebase

Harbormaster completed remote builds in B18228: Diff 45646.Jul 21 2018, 9:22 AM
asomers accepted this revision.Jul 21 2018, 5:03 PM

With an up-to-date ports tree and an 11.1 jail, www/rubygem-passenger works just fine but textproc/rubygem-gitlab-markup fails as reported. With the patch, both ports work fine.

This revision is now accepted and ready to land.Jul 21 2018, 5:03 PM
Closed by commit rP475361: Extract PLIST_SUB substitutions into a sed script. (authored by mat). · Explain WhyJul 26 2018, 11:10 AM
This revision was automatically updated to reflect the committed changes.
Herald added 1 blocking reviewer(s): portmgr. · View Herald TranscriptJul 26 2018, 11:10 AM

Revision Contents
Changeset List

Diff 45868

View Options

head/Mk/Scripts/check-stagedir.sh

Loading...
View Options

head/Mk/Scripts/check_leftovers.sh

Loading...
View Options

head/Mk/Scripts/plist_sub_sed_sort.sh

Loading...