diff mbox series

[bug#37121,32/40] gnu: libxklavier: Update to 5.4.

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

Commit Message

L p R n d n Aug. 20, 2019, 9:50 p.m. UTC
* 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.
---
 gnu/packages/gnome.scm | 32 ++++++++++++++++++++++++--------
 1 file changed, 24 insertions(+), 8 deletions(-)

Comments

Ludovic Courtès Aug. 28, 2019, 4:25 p.m. UTC | #1
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’.
L p R n d n Aug. 28, 2019, 8:33 p.m. UTC | #2
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
Ludovic Courtès Aug. 29, 2019, 12:24 p.m. UTC | #3
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 mbox series

Patch

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)