Re: [gentoo-portage-dev] Re: [PATCHv3 1/2] MEDIUM: misc-functions: Be more quiet when removing non existing INSTALL_MASK

[Zac Medico <zmedico@gentoo.org>]

On 04/21/2015 12:08 PM, Michael Orlitzky wrote:
> On 04/21/2015 01:28 PM, Zac Medico wrote:
>>>
>>> The docs for INSTALL_MASK (man 5 make.conf) don't mention that globs
>>> will work. It's expecting a "space delimited list of file names." Does
>>> it really take a space-delimited list of globs instead? If so, how does
>>> that reconcile with the fact that * could match spaces?
>>
>> How does it conflict?
>>
> 
> I guess it's more of a filenames-with-spaces question. Would this work?
> 
>   INSTALL_MASK="Boyd\ -\ Convex Optimization.pdf"

No, it doesn't seem to work given the current code. For example:

$ INSTALL_MASK="Boyd\ -\ Convex Optimization.pdf"
$ for x in $INSTALL_MASK; do echo "$x" ; done
Boyd\
-\
Convex
Optimization.pdf

> If you can escape the spaces, then the space-separated globs aren't
> ambiguous. I was thinking of something like this:
> 
>   $ /bin/ls B*\ Convex*
>   Boyd - Convex Optimization.pdf
> 
> Normally if you stick something like that in a quoted variable, its
> spaces can be unescaped:
> 
>   INSTALL_MASK="B* Convex*"
> 
> But now it's two globs instead of one. The whole thing makes sense if
> you can leave the space escaped though.

I don't think the inventor of INSTALL_MASK thought about these kinds of
cases.
-- 
Thanks,
Zac