Re: [PATCH] [xenocara] app/xenodm/config/Xsetup_0 - reduce the number of lines

2020-06-16 Thread Marc Espie
On Tue, Jun 16, 2020 at 08:43:02PM +0100, Raf Czlonka wrote:
> Ping.
> 
> CC'ing espie@ as he committed the initial code.
> 
> Cheers,
> 
> Raf
> 
> On Sun, Jun 07, 2020 at 07:30:39PM BST, Raf Czlonka wrote:
> > Hi all,
> > 
> > I've been running openbsd-backgrounds on all of my desktop machines and
> > thought this can be simplified a bit:
> > 
> > - fewer lines to uncomment
> > - easier to automate, i.e. via one liner, script, config management, etc.
> > - still under 80 columns wide
> > 
> > For your consideration.
> > 
> > Cheers,
> > 
> > Raf
> > 
> > Index: app/xenodm/config/Xsetup_0
> > ===
> > RCS file: /cvs/xenocara/app/xenodm/config/Xsetup_0,v
> > retrieving revision 1.6
> > diff -u -p -r1.6 Xsetup_0
> > --- app/xenodm/config/Xsetup_0  29 Jun 2019 13:33:06 -  1.6
> > +++ app/xenodm/config/Xsetup_0  7 Jun 2020 18:29:16 -
> > @@ -6,9 +6,6 @@ xconsole -geometry 480x130-0-0 -daemon -
> >  #  install package openbsd-backgrounds
> >  #  then uncomment:
> >  #
> > -# if test -x /usr/local/bin/openbsd-wallpaper
> > -# then
> > -#  /usr/local/bin/openbsd-wallpaper
> > -# fi
> > +# test -x /usr/local/bin/openbsd-wallpaper && 
> > /usr/local/bin/openbsd-wallpaper
> >  
> >  # sxpm OpenBSD.xpm &
> 
Even though it's longer, I actually prefer using if  for this kind of thing.



Re: [PATCH] [xenocara] app/xenodm/config/Xsetup_0 - reduce the number of lines

2020-06-16 Thread Raf Czlonka
Ping.

CC'ing espie@ as he committed the initial code.

Cheers,

Raf

On Sun, Jun 07, 2020 at 07:30:39PM BST, Raf Czlonka wrote:
> Hi all,
> 
> I've been running openbsd-backgrounds on all of my desktop machines and
> thought this can be simplified a bit:
> 
> - fewer lines to uncomment
> - easier to automate, i.e. via one liner, script, config management, etc.
> - still under 80 columns wide
> 
> For your consideration.
> 
> Cheers,
> 
> Raf
> 
> Index: app/xenodm/config/Xsetup_0
> ===
> RCS file: /cvs/xenocara/app/xenodm/config/Xsetup_0,v
> retrieving revision 1.6
> diff -u -p -r1.6 Xsetup_0
> --- app/xenodm/config/Xsetup_029 Jun 2019 13:33:06 -  1.6
> +++ app/xenodm/config/Xsetup_07 Jun 2020 18:29:16 -
> @@ -6,9 +6,6 @@ xconsole -geometry 480x130-0-0 -daemon -
>  #  install package openbsd-backgrounds
>  #  then uncomment:
>  #
> -# if test -x /usr/local/bin/openbsd-wallpaper
> -# then
> -#/usr/local/bin/openbsd-wallpaper
> -# fi
> +# test -x /usr/local/bin/openbsd-wallpaper && 
> /usr/local/bin/openbsd-wallpaper
>  
>  # sxpm OpenBSD.xpm &



[PATCH] [xenocara] app/xenodm/config/Xsetup_0 - reduce the number of lines

2020-06-07 Thread Raf Czlonka
Hi all,

I've been running openbsd-backgrounds on all of my desktop machines and
thought this can be simplified a bit:

- fewer lines to uncomment
- easier to automate, i.e. via one liner, script, config management, etc.
- still under 80 columns wide

For your consideration.

Cheers,

Raf

Index: app/xenodm/config/Xsetup_0
===
RCS file: /cvs/xenocara/app/xenodm/config/Xsetup_0,v
retrieving revision 1.6
diff -u -p -r1.6 Xsetup_0
--- app/xenodm/config/Xsetup_0  29 Jun 2019 13:33:06 -  1.6
+++ app/xenodm/config/Xsetup_0  7 Jun 2020 18:29:16 -
@@ -6,9 +6,6 @@ xconsole -geometry 480x130-0-0 -daemon -
 #  install package openbsd-backgrounds
 #  then uncomment:
 #
-# if test -x /usr/local/bin/openbsd-wallpaper
-# then
-#  /usr/local/bin/openbsd-wallpaper
-# fi
+# test -x /usr/local/bin/openbsd-wallpaper && /usr/local/bin/openbsd-wallpaper
 
 # sxpm OpenBSD.xpm &