[Build-common-hackers] Bug#712729: The fix for this breaks other packages
fsateler at debian.org
Tue Sep 9 02:30:48 UTC 2014
On Mon, Sep 8, 2014 at 7:51 PM, Simon Ruderich <simon at ruderich.org> wrote:
> On Wed, Sep 03, 2014 at 06:17:52PM -0400, Felipe Sateler wrote:
>> The problem is that such a blacklist could include other invalid shell
>> characters: + comes to mind, but I don't know the entire set of
>> character which are allowed as shell variables, as opposed to the
>> valid make allowed chars.
> Hello Felipe,
> True, but I think those characters won't occur very often and I
> wanted to provide a general fix which won't brake in the future
> when new options are added.
Yes, but there are letters that are invalid as shell characters:
> Package names must consist only of lower case letters (a-z), digits (0-9), plus (+) and minus (-) signs, and periods (.).
>> Maybe a more appropriate way would be to, instead of filtering out the
>> -, filter by matching FLAGS (after the DEB_ prefix)?
> Not all options contain FLAGS in their name, e.g.
Could you propose a patch filtering out the characters plus (+), minus
(-) signs, and periods (.)?
My make-fu is not very good...
More information about the Build-common-hackers