Re: [PATCH] gnu: Add Xfce Notification Daemon.

2017-04-26 Thread Petter
On Wed, 26 Apr 2017 16:39:19 +0200 Marius Bakke wrote: > > Takk! Pushed as b63106089a1eb2ac76303886f926f74967879386. > > By the way, patches should now be sent to the new "guix-patc...@gnu.org" > issue tracker. :-) Supert! :) Yes, I realized this a couple of days ago. I'll write an e-mail disc

Re: [PATCH] gnu: Add Xfce Notification Daemon.

2017-04-26 Thread Marius Bakke
Petter writes: > Hi Efraim, > > On Mon, 24 Apr 2017 22:22:43 +0300 > Efraim Flashner wrote: > >> The collection of inputs makes me wonder if it should be glib-or-gtk >> build-system and not "just" gnu-build-system. I don't have a good way to >> test if its necessary, but I'd look around. > > Her

Re: [PATCH] gnu: Add Xfce Notification Daemon.

2017-04-26 Thread Petter
I forgot to wrap "org.freedesktop.Notifications" in the description in @code{..}, as suggested by Marius. Here it is with this as well. From 922cbeb402ece8dbd0d592489069d7ccda64d702 Mon Sep 17 00:00:00 2001 From: Petter Date: Fri, 21 Apr 2017 15:33:22 +0200 Subject: [PATCH] gnu

Re: [PATCH] gnu: Add Xfce Notification Daemon.

2017-04-26 Thread Petter
27021481339ce9928ce5be17db45be Mon Sep 17 00:00:00 2001 From: Petter Date: Fri, 21 Apr 2017 15:33:22 +0200 Subject: [PATCH] gnu: Add Xfce Notification Daemon. * gnu/packages/xfce.scm (xfce4-notifyd): New variable. --- gnu/packages/xfce.scm | 32 1 file changed,

Re: [PATCH] gnu: Add Xfce Notification Daemon.

2017-04-24 Thread Efraim Flashner
On Mon, Apr 24, 2017 at 08:54:41PM +0200, Petter wrote: > Hei Marius, > > > > +(native-inputs > > > + `(("intltool" ,intltool) > > > + ("pkg-config" ,pkg-config))) > > > +(propagated-inputs > > > + `(("glib:bin" ,glib "bin"))) > > > > Do you know why glib:bin needs to be p

Re: [PATCH] gnu: Add Xfce Notification Daemon.

2017-04-24 Thread Petter
Hei Marius, > > +(native-inputs > > + `(("intltool" ,intltool) > > + ("pkg-config" ,pkg-config))) > > +(propagated-inputs > > + `(("glib:bin" ,glib "bin"))) > > Do you know why glib:bin needs to be propagated here? Looking at > gnome.scm, it's usually a "native-input". If

Re: [PATCH] gnu: Add Xfce Notification Daemon.

2017-04-24 Thread Marius Bakke
Hi Petter, thanks a lot for this! The patch looks good, I only have one question (not very familiar with gtk packaging either, so please excuse my ignorance). > * gnu/packages/xfce.scm (xfce4-notifyd): New variable. [...] > +(native-inputs > + `(("intltool" ,intltool) > + ("pkg-c

Re: [PATCH] gnu: Add Xfce Notification Daemon.

2017-04-24 Thread Petter
s service file first takes effect > after installing. I wasn't having any luck testing directly from > store. Thanks again wingo!) > > Best, > Petter From 2a31bcfa1b39c9403fb2dab3ab073def3ab8e140 Mon Sep 17 00:00:00 2001 From: Petter Date: Fri, 21 Apr 2017 15:33:22 +0200

[PATCH] gnu: Add Xfce Notification Daemon.

2017-04-21 Thread Petter
PetterFrom 4cf9acab0817d2898772830f27c1a11c15926b98 Mon Sep 17 00:00:00 2001 From: Petter Date: Fri, 21 Apr 2017 15:33:22 +0200 Subject: [PATCH] gnu: Add Xfce Notification Daemon. * gnu/packages/xfce.scm (xfce4-notifyd): New variable. --- gnu/packages/xfce.scm | 39 +++ 1 file changed, 39 insertion