From mboxrd@z Thu Jan 1 00:00:00 1970 From: Leo Famulari Subject: Re: [PATCH]: xfce4-power-manager: new variable Date: Thu, 3 Dec 2015 14:01:20 -0500 Message-ID: <20151203190120.GA18176@jasmine> References: <56604C07.2040900@gmx.net> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:54017) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1a4Z8G-0007OJ-0N for guix-devel@gnu.org; Thu, 03 Dec 2015 14:01:33 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1a4Z8A-00071G-W4 for guix-devel@gnu.org; Thu, 03 Dec 2015 14:01:31 -0500 Received: from out4-smtp.messagingengine.com ([66.111.4.28]:45932) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1a4Z8A-0006y4-PO for guix-devel@gnu.org; Thu, 03 Dec 2015 14:01:26 -0500 Content-Disposition: inline In-Reply-To: <56604C07.2040900@gmx.net> List-Id: "Development of GNU Guix and the GNU System distribution." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org Sender: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org To: Florian Paul Schmidt Cc: guix-devel@gnu.org On Thu, Dec 03, 2015 at 03:04:55PM +0100, Florian Paul Schmidt wrote: > Hi, > > attached is a patch for a first stab at packaging xfce4-power-manager. Since > I used a different package as a base for this one there might be some > unneeded inputs. I wonder if you guys have a tip for creating a little > scheme snippet that would try all combinations of inputs to determine the > minimal set for which ones the package build still succeeds? ;) [lazy flo] The potential issue with this approach is that a lot of software will build successfully even if some dependencies are missing. And then it will fail at runtime, sometimes only in certain scenarios. Not that I haven't thought about doing the same thing... :) Ideally each program would have a test suite that exercised every code path. Maybe AFL would be useful there? That sounds expensive but for very complex systems, it would be worth it, because in my limited experience packaging software, I have found that many upstream developers are not aware of the dependency graph of their software. It's fun to send them the output of "guix graph". > > From 5ca12ed7fd62792b856aa686e66dfc224e16bf35 Mon Sep 17 00:00:00 2001 > From: Florian Paul Schmidt > Date: Thu, 3 Dec 2015 14:57:10 +0100 > Subject: [PATCH] xfce4-power-manager: new variable > > --- > gnu/packages/xfce.scm | 38 ++++++++++++++++++++++++++++++++++++++ > 1 file changed, 38 insertions(+) > > diff --git a/gnu/packages/xfce.scm b/gnu/packages/xfce.scm > index e213fb6..41e193a 100644 > --- a/gnu/packages/xfce.scm > +++ b/gnu/packages/xfce.scm > @@ -646,6 +646,43 @@ menubar and the window decorations are hidden) that > helps you to save space > on your desktop.") > (license gpl2+))) > > +(define-public xfce4-power-manager > + (package > + (name "xfce4-power-manager") > + (version "1.4.3") > + (source (origin > + (method url-fetch) > + (uri (string-append "http://archive.xfce.org/xfce/4.12" > + "/src/" name "-" version ".tar.bz2")) > + (sha256 > + (base32 > + "04909sfc2nrj2wg9cw6y9y2r9yrp3l3vc201sy1gaiap67fi33h1")))) > + (build-system gnu-build-system) > + (arguments > + '(#:configure-flags '("--enable-gtk3"))) > + (native-inputs > + `(("pkg-config" ,pkg-config) > + ("intltool" ,intltool))) > + (inputs > + `(("exo" ,exo) > + ("garcon", garcon) > + ("lbxrandr" ,libxrandr) > + ("libwnck" ,libwnck-1) > + ("upower" ,upower) > + ("libnotify" ,libnotify) > + ("libxfce4ui" ,libxfce4ui))) > + (native-search-paths > + (list (search-path-specification > + (variable "X_XFCE4_LIB_DIRS") > + (files '("lib/xfce4"))))) > + (home-page "http://www.xfce.org/") > + (synopsis "Xfce Power Manager") > + (description > + "Power manager daemon for Xfce and associated settings page. > +Controls backlight, power saving, etc.") > + ;; Libraries are under LGPLv2.1+, and programs under GPLv2+. > + (license (list gpl2+ lgpl2.1+)))) > + > (define-public xfce > (package > (name "xfce") > @@ -689,6 +726,7 @@ on your desktop.") > ("xfce4-session" ,xfce4-session) > ("xfce4-settings" ,xfce4-settings) > ("xfce4-terminal" ,xfce4-terminal) > + ("xfce4-power-manager" ,xfce4-power-manager) > ("xfconf" ,xfconf) > ("xfdesktop" ,xfdesktop) > ("xfwm4" ,xfwm4) > -- > 2.5.0 > > > Flo > > > > From 5ca12ed7fd62792b856aa686e66dfc224e16bf35 Mon Sep 17 00:00:00 2001 > From: Florian Paul Schmidt > Date: Thu, 3 Dec 2015 14:57:10 +0100 > Subject: [PATCH] xfce4-power-manager: new variable > > --- > gnu/packages/xfce.scm | 38 ++++++++++++++++++++++++++++++++++++++ > 1 file changed, 38 insertions(+) > > diff --git a/gnu/packages/xfce.scm b/gnu/packages/xfce.scm > index e213fb6..41e193a 100644 > --- a/gnu/packages/xfce.scm > +++ b/gnu/packages/xfce.scm > @@ -646,6 +646,43 @@ menubar and the window decorations are hidden) that helps you to save space > on your desktop.") > (license gpl2+))) > > +(define-public xfce4-power-manager > + (package > + (name "xfce4-power-manager") > + (version "1.4.3") > + (source (origin > + (method url-fetch) > + (uri (string-append "http://archive.xfce.org/xfce/4.12" > + "/src/" name "-" version ".tar.bz2")) > + (sha256 > + (base32 > + "04909sfc2nrj2wg9cw6y9y2r9yrp3l3vc201sy1gaiap67fi33h1")))) > + (build-system gnu-build-system) > + (arguments > + '(#:configure-flags '("--enable-gtk3"))) > + (native-inputs > + `(("pkg-config" ,pkg-config) > + ("intltool" ,intltool))) > + (inputs > + `(("exo" ,exo) > + ("garcon", garcon) > + ("lbxrandr" ,libxrandr) > + ("libwnck" ,libwnck-1) > + ("upower" ,upower) > + ("libnotify" ,libnotify) > + ("libxfce4ui" ,libxfce4ui))) > + (native-search-paths > + (list (search-path-specification > + (variable "X_XFCE4_LIB_DIRS") > + (files '("lib/xfce4"))))) > + (home-page "http://www.xfce.org/") > + (synopsis "Xfce Power Manager") > + (description > + "Power manager daemon for Xfce and associated settings page. > +Controls backlight, power saving, etc.") > + ;; Libraries are under LGPLv2.1+, and programs under GPLv2+. > + (license (list gpl2+ lgpl2.1+)))) > + > (define-public xfce > (package > (name "xfce") > @@ -689,6 +726,7 @@ on your desktop.") > ("xfce4-session" ,xfce4-session) > ("xfce4-settings" ,xfce4-settings) > ("xfce4-terminal" ,xfce4-terminal) > + ("xfce4-power-manager" ,xfce4-power-manager) > ("xfconf" ,xfconf) > ("xfdesktop" ,xfdesktop) > ("xfwm4" ,xfwm4) > -- > 2.5.0 >