From mboxrd@z Thu Jan 1 00:00:00 1970 From: Leo Famulari Subject: Re: [PATCH 02/04] gnu: xdotools: Update to 3.20150503.1 Date: Sat, 6 Feb 2016 16:18:53 -0500 Message-ID: <20160206211853.GA6912@jasmine> References: <87d1safqir.fsf@dustycloud.org> <20160206083550.GD1432@jasmine> <87r3gptybm.fsf@dustycloud.org> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: 8bit Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:51476) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aSAFv-0008Pu-JV for guix-devel@gnu.org; Sat, 06 Feb 2016 16:19:03 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1aSAFr-0007bN-IW for guix-devel@gnu.org; Sat, 06 Feb 2016 16:18:59 -0500 Received: from out4-smtp.messagingengine.com ([66.111.4.28]:42683) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aSAFq-0007bI-VW for guix-devel@gnu.org; Sat, 06 Feb 2016 16:18:55 -0500 Content-Disposition: inline In-Reply-To: <87r3gptybm.fsf@dustycloud.org> 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: Christopher Allan Webber Cc: guix-devel@gnu.org On Sat, Feb 06, 2016 at 11:48:13AM -0800, Christopher Allan Webber wrote: > Leo Famulari writes: > > On Fri, Feb 05, 2016 at 07:48:12PM -0800, Christopher Allan Webber wrote: > > Is this line too long? I can't say for sure looking at the patch in my > > email client. If so, the linter should complain. > > > > [...] > > You're right. I should be using the linter but I was slacking off I > guess... sorry! The linter is our friend and we should visit him or her often :) I do wonder about the tests... if the required "black magic" is having an X server available, we do have some examples of that in the package tree. Otherwise it might be good to explain a little more so that it's easier for future contributors to understand what the problem is. Otherwise LGTM. > > New version attached. > - Chris > > From 5a61858e92625b905b904f57c1efe7f7e23823e5 Mon Sep 17 00:00:00 2001 > From: Christopher Allan Webber > Date: Fri, 5 Feb 2016 15:24:01 -0800 > Subject: [PATCH 1/3] gnu: xdotools: Update to 3.20150503.1 > > * gnu/packages/xdisorg.scm (xdotools): Upgrade to 3.20150503.1. > * gnu/packages/patches/xdotool-fix-makefile.patch: New file. > * gnu-system.am (dist_patch_DATA): Add it. > --- > gnu-system.am | 1 + > gnu/packages/patches/xdotool-fix-makefile.patch | 16 ++++++++++++++++ > gnu/packages/xdisorg.scm | 14 +++++++++----- > 3 files changed, 26 insertions(+), 5 deletions(-) > create mode 100644 gnu/packages/patches/xdotool-fix-makefile.patch > > diff --git a/gnu-system.am b/gnu-system.am > index 95150b6..4d32e59 100644 > --- a/gnu-system.am > +++ b/gnu-system.am > @@ -754,6 +754,7 @@ dist_patch_DATA = \ > gnu/packages/patches/wpa-supplicant-CVE-2015-5314.patch \ > gnu/packages/patches/wpa-supplicant-CVE-2015-5315.patch \ > gnu/packages/patches/wpa-supplicant-CVE-2015-5316.patch \ > + gnu/packages/patches/xdotool-fix-makefile.patch \ > gnu/packages/patches/xf86-video-ark-remove-mibstore.patch \ > gnu/packages/patches/xf86-video-ast-remove-mibstore.patch \ > gnu/packages/patches/xf86-video-geode-glibc-2.20.patch \ > diff --git a/gnu/packages/patches/xdotool-fix-makefile.patch b/gnu/packages/patches/xdotool-fix-makefile.patch > new file mode 100644 > index 0000000..5fed41f > --- /dev/null > +++ b/gnu/packages/patches/xdotool-fix-makefile.patch > @@ -0,0 +1,16 @@ > +Fix Makefile, which missed including a necessary library for compiling in the > +0.5.0 release. > + > +See https://github.com/jordansissel/xdotool/pull/81 for more information. > + > +--- xdotool-3.20150503.1/Makefile 2014-10-20 13:05:34.000000000 -0700 > ++++ xdotool-3.20150503.1/Makefile 2016-02-05 14:44:18.652734272 -0800 > +@@ -28,7 +28,7 @@ > + CFLAGS+=-g # TODO(sissel): Comment before release > + CFLAGS+=$(CPPFLAGS) > + > +-DEFAULT_LIBS=-L/usr/X11R6/lib -L/usr/local/lib -lX11 -lXtst -lXinerama > ++DEFAULT_LIBS=-L/usr/X11R6/lib -L/usr/local/lib -lX11 -lXtst -lXinerama -lxkbcommon > + DEFAULT_INC=-I/usr/X11R6/include -I/usr/local/include > + > + XDOTOOL_LIBS=$(shell pkg-config --libs x11 2> /dev/null || echo "$(DEFAULT_LIBS)") $(shell sh platform.sh extralibs) > \ No newline at end of file > diff --git a/gnu/packages/xdisorg.scm b/gnu/packages/xdisorg.scm > index dc01637..5ca203e 100644 > --- a/gnu/packages/xdisorg.scm > +++ b/gnu/packages/xdisorg.scm > @@ -9,6 +9,7 @@ > ;;; Copyright © 2015 Andy Wingo > ;;; Copyright © 2015 xd1le > ;;; Copyright © 2015 Florian Paul Schmidt > +;;; Copyright © 2016 Christopher Allan Webber > ;;; > ;;; This file is part of GNU Guix. > ;;; > @@ -48,6 +49,7 @@ > #:use-module (gnu packages guile) > #:use-module (gnu packages xml) > #:use-module (gnu packages gtk) > + #:use-module (gnu packages qt) > #:use-module (gnu packages xorg)) > > ;; packages outside the x.org system proper > @@ -107,16 +109,17 @@ avoiding password prompts when X11 forwarding has already been setup.") > (define-public xdotool > (package > (name "xdotool") > - (version "2.20110530.1") > + (version "3.20150503.1") > (source > (origin > (method url-fetch) > (uri (string-append > - "http://semicomplete.googlecode.com/files/" name "-" > - version ".tar.gz")) > + "https://github.com/jordansissel/xdotool/releases/download/v" > + version "/xdotool-" version ".tar.gz")) > (sha256 > (base32 > - "0rxggg1cy7nnkwidx8x2w3c5f3pk6dh2b6q0q7hp069r3n5jrd77")))) > + "1lcngsw33fy9my21rdiz1gs474bfdqcfxjrnfggbx4aypn1nhcp8")) > + (patches (list (search-patch "xdotool-fix-makefile.patch"))))) > (build-system gnu-build-system) > (arguments > '(#:tests? #f ; Test suite requires a lot of black magic > @@ -134,7 +137,8 @@ avoiding password prompts when X11 forwarding has already been setup.") > ("libxext" ,libxext) > ("libxi" ,libxi) > ("libxinerama" ,libxinerama) > - ("libxtst" ,libxtst))) > + ("libxtst" ,libxtst) > + ("libxkbcommon" ,libxkbcommon))) > (home-page "http://www.semicomplete.com/projects/xdotool") > (synopsis "Fake keyboard/mouse input, window management, and more") > (description "Xdotool lets you simulate keyboard input and mouse activity, > -- > 2.6.3 >