Re: [PATCH] Adding .xsession to guile-wm

2016-09-14 Thread Ludovic Courtès
Alex ter Weele skribis: > Ludovic Courtès writes: > >> I agree with David that it would be good to talk to the upstream >> developer so that the .desktop file is distributed by upstream >> eventually. Could you try to reach them? > > There are

Re: [PATCH] Adding .xsession to guile-wm

2016-09-13 Thread Alex ter Weele
Ludovic Courtès writes: > I agree with David that it would be good to talk to the upstream > developer so that the .desktop file is distributed by upstream > eventually. Could you try to reach them? There are improvements I want to make to guile-wm itself, so I plan on talking to

Re: [PATCH] Adding .xsession to guile-wm

2016-09-13 Thread Ludovic Courtès
Alex ter Weele <alex.ter.we...@gmail.com> skribis: > From 91f74b1b762596527dfa9412c33e39981932fa70 Mon Sep 17 00:00:00 2001 > From: Alex ter Weele <alex.ter.we...@gmail.com> > Date: Sun, 11 Sep 2016 22:18:53 -0400 > Subject: [PATCH] Adding .xsession to guile-wm >

Re: [PATCH] Adding .xsession to guile-wm

2016-09-13 Thread Ludovic Courtès
John Darrington skribis: > On Mon, Sep 12, 2016 at 09:45:22PM -0400, Thompson, David wrote: > On Mon, Sep 12, 2016 at 9:23 PM, Alex ter Weele > wrote: > > I don't believe there is a standard, but different packages do it >

Re: [PATCH] Adding .xsession to guile-wm

2016-09-13 Thread Ludovic Courtès
John Darrington skribis: > Most of the .desktop files in my store do not have the /gnu/store/... on their > Exec= line. Perhaps they should? What is the policy here? I think it’s best to use absolute file names. Ludo’.

Re: [PATCH] Adding .xsession to guile-wm

2016-09-12 Thread John Darrington
On Mon, Sep 12, 2016 at 09:45:22PM -0400, Thompson, David wrote: On Mon, Sep 12, 2016 at 9:23 PM, Alex ter Weele wrote: > I don't believe there is a standard, but different packages do it > different ways. Under my patch, guile-wm will regenerate its

Re: [PATCH] Adding .xsession to guile-wm

2016-09-12 Thread Thompson, David
On Mon, Sep 12, 2016 at 9:23 PM, Alex ter Weele wrote: > I don't believe there is a standard, but different packages do it > different ways. Under my patch, guile-wm will regenerate its .desktop as > part of the build process, so its Exec= should be kept up-to-date. We

Re: [PATCH] Adding .xsession to guile-wm

2016-09-12 Thread Alex ter Weele
I don't believe there is a standard, but different packages do it different ways. Under my patch, guile-wm will regenerate its .desktop as part of the build process, so its Exec= should be kept up-to-date. -Alex

Re: [PATCH] Adding .xsession to guile-wm

2016-09-12 Thread John Darrington
On Mon, Sep 12, 2016 at 09:07:10PM +0300, Efraim Flashner wrote: On Mon, Sep 12, 2016 at 06:05:27PM +0200, John Darrington wrote: > Most of the .desktop files in my store do not have the /gnu/store/... on their > Exec= line. Perhaps they should? What is the policy here? >

Re: [PATCH] Adding .xsession to guile-wm

2016-09-12 Thread Efraim Flashner
On Mon, Sep 12, 2016 at 06:05:27PM +0200, John Darrington wrote: > Most of the .desktop files in my store do not have the /gnu/store/... on their > Exec= line. Perhaps they should? What is the policy here? > > J' > I can comment that enlightenment does, and slim finds it:

Re: [PATCH] Adding .xsession to guile-wm

2016-09-12 Thread John Darrington
it as an available wm. From 91f74b1b762596527dfa9412c33e39981932fa70 Mon Sep 17 00:00:00 2001 From: Alex ter Weele <alex.ter.we...@gmail.com> Date: Sun, 11 Sep 2016 22:18:53 -0400 Subject: [PATCH] Adding .xsession to guile-wm --- gnu/packages/guile-wm.sc

Re: [PATCH] Adding .xsession to guile-wm

2016-09-12 Thread Thompson, David
On Mon, Sep 12, 2016 at 11:56 AM, Alex ter Weele wrote: > Manually making a .desktop is what xmonad and fluxbox do: > http://git.savannah.gnu.org/cgit/guix.git/tree/gnu/packages/wm.scm#n244 > http://git.savannah.gnu.org/cgit/guix.git/tree/gnu/packages/wm.scm#n359 > Hence

Re: [PATCH] Adding .xsession to guile-wm

2016-09-12 Thread Alex ter Weele
Manually making a .desktop is what xmonad and fluxbox do: http://git.savannah.gnu.org/cgit/guix.git/tree/gnu/packages/wm.scm#n244 http://git.savannah.gnu.org/cgit/guix.git/tree/gnu/packages/wm.scm#n359 Hence this approach. I could also work on moving this to a patch. Which is preferred? -Alex

Re: [PATCH] Adding .xsession to guile-wm

2016-09-12 Thread Thompson, David
On Sun, Sep 11, 2016 at 11:01 PM, Alex ter Weele wrote: > Adding an xsession to guile-wm so that display managers will identify it > as an available wm. I think it would be best if this were submitted as a patch to guile-wm upstream instead, and we can include the patch

[PATCH] Adding .xsession to guile-wm

2016-09-12 Thread Alex ter Weele
Adding an xsession to guile-wm so that display managers will identify it as an available wm. From 91f74b1b762596527dfa9412c33e39981932fa70 Mon Sep 17 00:00:00 2001 From: Alex ter Weele <alex.ter.we...@gmail.com> Date: Sun, 11 Sep 2016 22:18:53 -0400 Subject: [PATCH] Adding .xsession to gu