I'm curious about your thoughts on this patch. It adds an NLS option to p5-Locale-libintl, to disable the gettext dependency.
The problem is that texinfo, used by a huge proportion of the ports tree, has an NLS option to disable gettext, but it's brought in as a runtime dependency anyway because of p5-Locale-libintl.
With this patch, texinfo builds happily without gettext being around. p5-Locale-libintl also passes all the included tests.
I think it's OK but I didn't see the patch. :)
And of course I deleted the patch as soon as I submitted the bug report. I’ll regenerate it once I get home.
Created attachment 204962 [details]
Patches work better when they actually exist
Okay, let’s see if it’s any easier to evaluate a patch when it’s actually attached
It works fine in a clean environment (gettext-runtime is absent). I've added an extra patch for Makefile.PL to ensure it does not use gettext-runtime when it's already installed.
Good call! I hadn't considered building outside poudriere.
A commit references this bug:
Date: Sun Jun 23 19:21:20 UTC 2019
New revision: 504980
Add NLS option
Submitted by: adamw