[packages/imsettings] - fix xinitrc script perms - rel 3

Elan Ruusamäe glen at pld-linux.org
Tue Jul 14 00:19:54 CEST 2015


On 13.07.2015 21:02, Jan Rękorajski wrote:
>> >this is wrong. these scripts there are sourced, not executed
>> >
>> >if you change is to introduce /bin/sh dependency, then just do chmod in
>> >%install, but leave %files as 0640
>> >
>> >the same applies to systemd,sabayon spec changes of yours
> This is not wrong, they are sourced iff the script is executable.

well. *that* is wrong.

other system locations that source *.sh (checked /etc/shrc.d, 
/etc/profile.d), do not require execute bit

and in fact you should not execute these, as they may require additional 
env (which is present when sourced from main script)


...

and even the extra check for "x" bit  to avoid executing backups, blows 
up, as vim backups (*~) have exactly the same permissions as original 
(if possible) and *.sh glob should exclude backups.

-- 
glen



More information about the pld-devel-en mailing list