Message ID | 20190820215043.32077-32-guix@lprndn.info |
---|---|
State | Accepted |
Headers | show |
Series | [bug#37121,01/40] gnu: libxfce4util: Update to 4.14.0. | expand |
Hello, L p R n d n <guix@lprndn.info> skribis: > * gnu/packages/gnome.scm (libxklavier): Update to 5.4. > [source] Switch to git source. > [arguments] Add no-configure phase. Disable xmodmap in #:configure-flags. > [inputs] Add which, intltool, gtk-doc, which, automake, autoconf and libtool. [...] > - (version "5.3") > + (version "5.4") > (source (origin > - (method url-fetch) > - (uri (string-append "mirror://gnome/sources/" name "/" > - version "/" name "-" version ".tar.xz")) > + (method git-fetch) > + (uri > + (git-reference > + (url (string-append "https://anongit.freedesktop.org/git/" name)) > + (commit "396955bd2ba2db34a42b3807b03155fcc11dfe50"))) > (sha256 > (base32 > - "016lpdv35z0qsw1cprdc2k5qzkdi5waj6qmr0a2q6ljn9g2kpv7b")))) > + "1w1x5mrgly2ldiw3q2r6y620zgd89gk7n90ja46775lhaswxzv7a")) > + (file-name (git-file-name name version)))) How does this commit relate to version 5.4? <https://www.freedesktop.org/wiki/Software/LibXklavier/> says that release tarballs are available on the GNOME web site, and there’s no 5.4 there. Was this change necessary for the Xfce upgrade? Thanks, Ludo’.
Hello, Ludovic Courtès <ludo@gnu.org> writes: > Hello, > > L p R n d n <guix@lprndn.info> skribis: > >> * gnu/packages/gnome.scm (libxklavier): Update to 5.4. >> [source] Switch to git source. >> [arguments] Add no-configure phase. Disable xmodmap in #:configure-flags. >> [inputs] Add which, intltool, gtk-doc, which, automake, autoconf and libtool. > > [...] > >> - (version "5.3") >> + (version "5.4") >> (source (origin >> - (method url-fetch) >> - (uri (string-append "mirror://gnome/sources/" name "/" >> - version "/" name "-" version ".tar.xz")) >> + (method git-fetch) >> + (uri >> + (git-reference >> + (url (string-append "https://anongit.freedesktop.org/git/" name)) >> + (commit "396955bd2ba2db34a42b3807b03155fcc11dfe50"))) >> (sha256 >> (base32 >> - "016lpdv35z0qsw1cprdc2k5qzkdi5waj6qmr0a2q6ljn9g2kpv7b")))) >> + "1w1x5mrgly2ldiw3q2r6y620zgd89gk7n90ja46775lhaswxzv7a")) >> + (file-name (git-file-name name version)))) > > How does this commit relate to version 5.4? > <https://www.freedesktop.org/wiki/Software/LibXklavier/> says that > release tarballs are available on the GNOME web site, and there’s no 5.4 > there. That's true, it's only available through git. The commit seems to be 6 years old and after some invastigation, 5.4 appears to be used by other distributions (at least Nixos, Arch and Debian) for several years. I don't know what are upstream's plans nor why it's not available on GNOME's website though. > > Was this change necessary for the Xfce upgrade? Yeah, I'm not really confortable with this either but xfce4-xkb-plugin explicitely ask for at least libxklavier 5.4. On the other hand xfce4-xbd-plugin is not part of xfce's core so that update is arguable. > Thanks, > Ludo’. Have a nice day, L p R n d n
L p R n d n <guix@lprndn.info> skribis: > Ludovic Courtès <ludo@gnu.org> writes: > >> Hello, >> >> L p R n d n <guix@lprndn.info> skribis: >> >>> * gnu/packages/gnome.scm (libxklavier): Update to 5.4. >>> [source] Switch to git source. >>> [arguments] Add no-configure phase. Disable xmodmap in #:configure-flags. >>> [inputs] Add which, intltool, gtk-doc, which, automake, autoconf and libtool. >> >> [...] >> >>> - (version "5.3") >>> + (version "5.4") >>> (source (origin >>> - (method url-fetch) >>> - (uri (string-append "mirror://gnome/sources/" name "/" >>> - version "/" name "-" version ".tar.xz")) >>> + (method git-fetch) >>> + (uri >>> + (git-reference >>> + (url (string-append "https://anongit.freedesktop.org/git/" name)) >>> + (commit "396955bd2ba2db34a42b3807b03155fcc11dfe50"))) >>> (sha256 >>> (base32 >>> - "016lpdv35z0qsw1cprdc2k5qzkdi5waj6qmr0a2q6ljn9g2kpv7b")))) >>> + "1w1x5mrgly2ldiw3q2r6y620zgd89gk7n90ja46775lhaswxzv7a")) >>> + (file-name (git-file-name name version)))) >> >> How does this commit relate to version 5.4? >> <https://www.freedesktop.org/wiki/Software/LibXklavier/> says that >> release tarballs are available on the GNOME web site, and there’s no 5.4 >> there. > > That's true, it's only available through git. The commit seems to be > 6 years old and after some invastigation, 5.4 appears to be used > by other distributions (at least Nixos, Arch and Debian) for several > years. I don't know what are upstream's plans nor why it's not available > on GNOME's website though. OK. I see there’s a ‘libxklavier-5.4’ tag in the Git repo above. Could you explicitly use this tag in the source rather than a commit ID, for clarity? Like so: (git-reference (url …) (commit (string-append "libxklavier-" version))) >> Was this change necessary for the Xfce upgrade? > > Yeah, I'm not really confortable with this either but xfce4-xkb-plugin > explicitely ask for at least libxklavier 5.4. On the other hand > xfce4-xbd-plugin is not part of xfce's core so that update is arguable. OK. Then we’ll make sure to apply the libxklavier upgrade before the Xfce upgrade. Thanks! Ludo’.
diff --git a/gnu/packages/gnome.scm b/gnu/packages/gnome.scm index d5e46e09c1..8b54abbb91 100644 --- a/gnu/packages/gnome.scm +++ b/gnu/packages/gnome.scm @@ -2537,24 +2537,40 @@ and objects.") (define-public libxklavier (package (name "libxklavier") - (version "5.3") + (version "5.4") (source (origin - (method url-fetch) - (uri (string-append "mirror://gnome/sources/" name "/" - version "/" name "-" version ".tar.xz")) + (method git-fetch) + (uri + (git-reference + (url (string-append "https://anongit.freedesktop.org/git/" name)) + (commit "396955bd2ba2db34a42b3807b03155fcc11dfe50"))) (sha256 (base32 - "016lpdv35z0qsw1cprdc2k5qzkdi5waj6qmr0a2q6ljn9g2kpv7b")))) + "1w1x5mrgly2ldiw3q2r6y620zgd89gk7n90ja46775lhaswxzv7a")) + (file-name (git-file-name name version)))) (build-system gnu-build-system) (arguments - '(#:configure-flags + '(#:phases + (modify-phases %standard-phases + (add-after 'unpack 'no-configure + (lambda* _ + (setenv "NOCONFIGURE" "1") + #t))) + #:configure-flags (list (string-append "--with-xkb-base=" (assoc-ref %build-inputs "xkeyboard-config") - "/share/X11/xkb")))) + "/share/X11/xkb") + "--disable-xmodmap-support"))) (native-inputs `(("glib:bin" ,glib "bin") ; for glib-mkenums, etc. ("gobject-introspection" ,gobject-introspection) - ("pkg-config" ,pkg-config))) + ("pkg-config" ,pkg-config) + ("gtk-doc" ,gtk-doc) + ("intltool" ,intltool) + ("which" ,which) + ("autoconf" ,autoconf) + ("automake" ,automake) + ("libtool" ,libtool))) (propagated-inputs ;; Required by libxklavier.pc. `(("glib" ,glib)