Re: flaws in xinitrc-ng

2012-01-30 Thread Tomasz Pala
On Mon, Jan 30, 2012 at 07:34:48 +0200, Elan Ruusamäe wrote: >>> > for first problem, make all -x scripts executed, not sourced? >> If you want to execute some code, just create #!sh script with proper >> commands and source it. That's the same rule we follow for many other >> scripts using sth.d

Re: flaws in xinitrc-ng

2012-01-29 Thread Elan Ruusamäe
On 01/30/2012 12:17 AM, Tomasz Pala wrote: > for first problem, make all -x scripts executed, not sourced? If you want to execute some code, just create #!sh script with proper commands and source it. That's the same rule we follow for many other scripts using sth.d for config inclusion (e.g. l

Re: flaws in xinitrc-ng

2012-01-29 Thread Tomasz Pala
On Sun, Jan 29, 2012 at 16:58:33 +0200, Elan Ruusamäe wrote: > 78# Run all system xinitrc shell scripts. > 79for i in /etc/X11/xinit/xinitrc.d/* ; do > 80if [ -x "$i" ]; then > 81. "$i" > 82fi > 83done > > this code has several flaws: > > 1

Re: flaws in xinitrc-ng

2012-01-29 Thread Jan Rękorajski
On Sun, 29 Jan 2012, Jacek Konieczny wrote: > On Sun, Jan 29, 2012 at 04:58:33PM +0200, Elan Ruusamäe wrote: > > how to proceed here? > > for first problem, make all -x scripts executed, not sourced? > > Aren't the scripts supposed to set session environment in different > ways? Running them in a

Re: flaws in xinitrc-ng

2012-01-29 Thread Jacek Konieczny
On Sun, Jan 29, 2012 at 04:58:33PM +0200, Elan Ruusamäe wrote: > how to proceed here? > for first problem, make all -x scripts executed, not sourced? Aren't the scripts supposed to set session environment in different ways? Running them in a subprocess won't help in this task… Greets, Jac

Re: flaws in xinitrc-ng

2012-01-29 Thread Jan Rękorajski
On Sun, 29 Jan 2012, Elan Ruusamäe wrote: > hi > > our latest and greatest release > http://svn.pld-linux.org/trac/svn/browser/xinitrc-ng/trunk/xinitdefs?annotate=blame&rev=12296 > >

flaws in xinitrc-ng

2012-01-29 Thread Elan Ruusamäe
hi our latest and greatest release http://svn.pld-linux.org/trac/svn/browser/xinitrc-ng/trunk/xinitdefs?annotate=blame&rev=12296