Re: [PATCH] maint: make some shell and perl scripts executable in 'tests/'

[Pádraig Brady]

On 07/22/2013 02:29 PM, Eric Blake wrote:
> On 07/21/2013 05:14 PM, Bernhard Voelker wrote:
>> On 07/22/2013 12:22 AM, Pádraig Brady wrote:
>>> How about something like this instead?
>>>
>>> find tests/ \( -name '*.sh' -o -name '*.pl' \) \! -perm /111 -print |
>>> grep . && { echo ...; exit 1; }
>>
>> Good idea!  I knew it was ugly.
>>
>> Thanks for the review!
>> 2 nits:
>>
>> a) find ! -perm /111 will find only files with *any* of ugo=x set;
>> we need ! -perm -111, don't we?
> 
> -perm /111 is a GNU extension, -perm -111 is required by POSIX.  Are we
> trying to make this check runnable by people that are not using GNU
> findutils? 

For devs, GNU extensions are probably OK ...

 Then again, in this particular case, I think we want -111
> (exact match) rather than /111 (inclusive match), which delays the worry
> about whether we need GNU extensions in the first place.

... but best avoided if we can.

BTW, I notice that GNU -perm +111 is no longer supported.
and the equivalent -perm /111 is now all that's supported.
I would have had some time period where a deprecation warning
was issued TBH.  I've fixed up my scripts/rpm packages in any case.

thanks,
Pádraig.