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

[Jan Rękorajski]

On Mon, 13 Jul 2015, Elan Ruusamäe wrote:

> On 11.07.2015 13:02, baggins wrote:
> > commit 697931cdc57afa9b30392073cb727097dfdc79c8
> > Author: Jan Rękorajski <baggins at pld-linux.org>
> > Date:   Sat Jul 11 12:02:38 2015 +0200
> >
> >      - fix xinitrc script perms
> >      - rel 3
> >
> >   imsettings.spec | 4 ++--
> >   1 file changed, 2 insertions(+), 2 deletions(-)
> > ---
> > diff --git a/imsettings.spec b/imsettings.spec
> > index 556547a..fe6f005 100644
> > --- a/imsettings.spec
> > +++ b/imsettings.spec
> > @@ -8,7 +8,7 @@ Summary:	Delivery framework for general Input Method configuration
> >   Summary(pl.UTF-8):	Szkielet do ogólnej konfiguracji method wprowadzania znaków
> >   Name:		imsettings
> >   Version:	1.6.8
> > -Release:	2
> > +Release:	3
> >   License:	LGPL v2+
> >   Group:		Applications/System
> >   Source0:	https://bitbucket.org/tagoh/imsettings/downloads/%{name}-%{version}.tar.bz2
> > @@ -348,7 +348,7 @@ rm -rf $RPM_BUILD_ROOT
> >   %defattr(644,root,root,755)
> >   %doc AUTHORS ChangeLog NEWS README
> >   
> > -%{_sysconfdir}/X11/xinit/xinitrc.d/50-xinput.sh
> > +%attr(755,root,root) %{_sysconfdir}/X11/xinit/xinitrc.d/50-xinput.sh
> >   %{_sysconfdir}/X11/xinit/xinput.d/*.conf
> >   %{_sysconfdir}/xdg/autostart/imsettings-start.desktop
> >
> 
> 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.

-- 
Jan Rękorajski                    | PLD/Linux
SysAdm | baggins<at>pld-linux.org | http://www.pld-linux.org/