From mboxrd@z Thu Jan 1 00:00:00 1970 From: Cyrill Schenkel Subject: Re: [PATCH] gnu: Add xcompmgr Date: Fri, 23 Oct 2015 16:45:16 +0200 Message-ID: <87r3kl63ur.fsf@SIRIUSGUIX00.i-did-not-set--mail-host-address--so-tickle-me> References: <87si517rav.fsf@SIRIUSGUIX00.i-did-not-set--mail-host-address--so-tickle-me> <87si51or31.fsf@denknerd.org> <87oafpoq8a.fsf@denknerd.org> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="=-=-=" Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:60665) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Zpbj3-0001HQ-C1 for guix-devel@gnu.org; Fri, 23 Oct 2015 08:45:42 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Zpbiz-0001TL-PL for guix-devel@gnu.org; Fri, 23 Oct 2015 08:45:41 -0400 Received: from mail-wi0-x235.google.com ([2a00:1450:400c:c05::235]:32895) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Zpbiz-0001T8-Cx for guix-devel@gnu.org; Fri, 23 Oct 2015 08:45:37 -0400 Received: by wijp11 with SMTP id p11so75808952wij.0 for ; Fri, 23 Oct 2015 05:45:36 -0700 (PDT) In-Reply-To: <87oafpoq8a.fsf@denknerd.org> (Paul van der Walt's message of "Fri, 23 Oct 2015 12:04:37 +0200") 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: Paul van der Walt , guix-devel@gnu.org --=-=-= Content-Type: text/plain Hello Paul Paul van der Walt writes: > > From dc8d0477c0644e9d54ebba6af955c3d4e29270fc Mon Sep 17 00:00:00 2001 > > From: Cyrill Schenkel > > Date: Fri, 23 Oct 2015 13:25:11 +0200 > > Subject: [PATCH] gnu: Add xcompmgr > > > > * gnu/packages/xorg.scm (xcompmgr): New variable. > > --- > > gnu/packages/xorg.scm | 41 +++++++++++++++++++++++++++++++++++++++++ > > 1 file changed, 41 insertions(+) > > > > diff --git a/gnu/packages/xorg.scm b/gnu/packages/xorg.scm > > index b39bbd4..d51cd30 100644 > > --- a/gnu/packages/xorg.scm > > +++ b/gnu/packages/xorg.scm > > @@ -4,6 +4,7 @@ > > [...] > > + (arguments > > + `(#:phases (modify-phases %standard-phases > > + (add-before 'configure 'autogen > > + (lambda _ > > + (setenv "NOCONFIGURE" "t") > > + (zero? (system* "./autogen.sh"))))))) > > + (native-inputs > > [...] > > Mark Weaver recently told me that autogen phases should be after 'unpack > instead of before 'configure, since the MIPS platform (and perhaps > others) sometimes need to adjust the output of autogen.sh. Yes, you're right. I changed it accordingly. > ...and another thing while i'm at it, although i'm less sure about this > one... > > On 2015-10-23 at 11:46, quoth Paul van der Walt: >>> + (setenv "NOCONFIGURE" "t") > > Does that do what i think it does? If so, why not just use something > like the following? > > (modify-phases %standard-phases > (delete 'configure) > ...) It does what you think it does. If I did omit it, I'd need to patch the `autogen.sh` file so that it passes the neccessary parameters to the configure script and I'd also need to set the neccessary environment variables. It seems to me that setting the `NOCONFIGURE` environment variable is by far the simpler solution. Regards, Cyrill --=-=-= Content-Type: text/x-patch; charset=utf-8 Content-Disposition: inline; filename=0001-gnu-Add-xcompmgr.patch Content-Transfer-Encoding: quoted-printable >From 55122d4c14231f3646db902bc2b88469fc96666e Mon Sep 17 00:00:00 2001 From: Cyrill Schenkel Date: Fri, 23 Oct 2015 16:38:16 +0200 Subject: [PATCH] gnu: Add xcompmgr * gnu/packages/xorg.scm (xcompmgr): New variable. --- gnu/packages/xorg.scm | 41 +++++++++++++++++++++++++++++++++++++++++ 1 file changed, 41 insertions(+) diff --git a/gnu/packages/xorg.scm b/gnu/packages/xorg.scm index b39bbd4..e237754 100644 --- a/gnu/packages/xorg.scm +++ b/gnu/packages/xorg.scm @@ -4,6 +4,7 @@ ;;; Copyright =C2=A9 2014, 2015 Eric Bavier ;;; Copyright =C2=A9 2015 Ludovic Court=C3=A8s ;;; Copyright =C2=A9 2015 Eric Dvorsak +;;; Copyright =C2=A9 2015 Cyrill Schenkel ;;; ;;; This file is part of GNU Guix. ;;; @@ -5439,3 +5440,43 @@ perl programs to display windows and graphics on X11= servers.") ;; of the extension modules in the directory Protocol/Ext: see those f= iles ;; for details)." (license (package-license perl)))) + +(define-public xcompmgr + (package + (name "xcompmgr") + (version "1.1.7") + (source + (origin + ;; there's no current tarball + (method git-fetch) + (uri (git-reference + (url "http://anongit.freedesktop.org/git/xorg/app/xcompmgr.gi= t") + (commit (string-append name "-" version)))) + (sha256 + (base32 + "04swkrm3gk689wrjc418bd3n25w8r20kg1xfbn5j8d7mx1r5gf16")) + (file-name (string-append name "-" version)))) + (build-system gnu-build-system) + (arguments + `(#:phases (modify-phases %standard-phases + (add-after 'unpack 'autogen + (lambda _ + (setenv "NOCONFIGURE" "t") + (zero? (system* "sh" "autogen.sh"))))))) + (native-inputs + `(("pkg-config" ,pkg-config) + ("autoconf" ,autoconf) + ("automake" ,automake))) + (inputs + `(("libX11" ,libx11) + ("libXext" ,libxext) + ("libXcomposite" ,libxcomposite) + ("libXfixes" ,libxfixes) + ("libXdamage" ,libxdamage) + ("libXrender" ,libxrender))) + (synopsis "X Compositing manager using RENDER") + (description "xcompmgr is a sample compositing manager for X servers +supporting the XFIXES, DAMAGE, RENDER, and COMPOSITE extensions. It enabl= es +basic eye-candy effects.") + (home-page "http://cgit.freedesktop.org/xorg/app/xcompmgr/") + (license (license:x11-style "file://COPYING")))) --=20 2.5.0 --=-=-=--