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.

> 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.
DEB_BUILD_MAINT_OPTIONS.

> You can do so like this:

Thank you.

Regards
Simon
-- 
+ privacy is necessary
+ using gnupg http://gnupg.org
+ public key id: 0x92FEFDB7E44C32F9

Attachment: signature.asc
Description: Digital signature

Reply via email to