GNU bug report logs - #62820
[core-updates] error building xkeyboard-config

Previous Next

Package: guix;

Reported by: Brian Cully <bjc <at> spork.org>

Date: Thu, 13 Apr 2023 21:28:02 UTC

Severity: normal

Done: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>

Bug is archived. No further changes may be made.

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 62820 in the body.
You can then email your comments to 62820 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to bug-guix <at> gnu.org:
bug#62820; Package guix. (Thu, 13 Apr 2023 21:28:02 GMT) Full text and rfc822 format available.

Acknowledgement sent to Brian Cully <bjc <at> spork.org>:
New bug report received and forwarded. Copy sent to bug-guix <at> gnu.org. (Thu, 13 Apr 2023 21:28:02 GMT) Full text and rfc822 format available.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Brian Cully <bjc <at> spork.org>
To: bug-guix <at> gnu.org
Subject: [core-updates] error building xkeyboard-config
Date: Thu, 13 Apr 2023 17:18:09 -0400
Looks like something changed between 2.34 (on master) and 2.36 (on 
core-updates) which caused ‘share/X11/rules/base’ to generate 
improperly.

On core-updates:
--8<---------------cut here---------------start------------->8---
~/src/guix-core-updates $ ./pre-inst-env guix build 
xkeyboard-config
/gnu/store/0yg8hlbvynr5bq8816srgcyjf50sp54h-xkeyboard-config-2.36
--8<---------------cut here---------------end--------------->8---

Which includes the following broken lines in ‘rules/base’:
--8<---------------cut here---------------start------------->8---
! $inetmediakbds = \
		a4_rfkb23 a4techKB21 a4techKBS8 acer_ferrari4k 
		acer_laptop /
		armada asus_laptop benqx btc5090 btc6301urf 
		btc9019u /
		cherrybluea cherryblueb cherrycyboard chicony042 /
		compalfl90 compaqik13 compaqik18 creativedw7000 /
		cymotionlinux dellm65 dellusbmm dexxa diamond 
		dtk2000 /
		emachines ennyah_dkb1008 fscaa1667g genius 
		geniuscomfy /
		geniuscomfy2 geniuskb19e hp5xx hpdv5 hpi6 hpxe3gc 
		hpxe3gf /
		hpxe4xxx hpxt1000 hpzt11xx inspiron latitude /
		logidinovo logidinovoedge logitech_base 
		logitech_g15 /
		microsoft4000 microsoft7000 microsoftmult 
		microsoftpro /
		microsoftprooem mx1998 mx2500 mx2750 pc105 
		precision_m /
		presario propeller samsung4500 samsung4510 
		scorpius /
		silvercrest sk1300 sk2500 sk7100 sp_inet 
		targa_v811 /
		thinkpad thinkpad60 tm2030USB-102 tm2030USB-106 /
		toshiba_s3000 trust trustda trust_slimline 
		unitekkb1925
--8<---------------cut here---------------end--------------->8---

Note the forward slash as a line continuation character, rather 
than a backslash, which is used everywhere. The same file on 
master:
--8<---------------cut here---------------start------------->8---
~/src/guix $ ./pre-inst-env guix build xkeyboard-config
/gnu/store/8mszv7v6kqdyavpvf8zb7kkagaan5vri-xkeyboard-config-2.34
--8<---------------cut here---------------end--------------->8---

This is the equivalent block:
--8<---------------cut here---------------start------------->8---
! $inetmediakbds = acer_ferrari4k acer_laptop btc5090 btc9019u 
 cherryblueb \
		cherrybluea herrycyboard chicony042 compaqik13 
		compaqik18 \
		armada presario dellm65 inspiron dellusbmm diamond 
		\
		ennyah_dkb1008 genius geniuscomfy2 hpi6 hpxe3gc 
		hpxe3gf \
		hpxt1000 hpzt11xx hpdv5 hpxe4xxx hp5xx thinkpad60 
		ogitech_base \
		logidinovo logidinovoedge logitech_g15 mx1998 
		mx2500 mx2750 \
		microsoft4000 microsoft7000 microsoftprooem 
		microsoftmult \
		propeller samsung4500 samsung4510 sk1300 sk2500 
		sk7100 \
		toshiba_s3000 trust trustda cymotionlinux 
		silvercrest \
		emachines benqx unitekkb1925 creativedw7000 
		compalfl90 \
		pc105 a4techKB21 a4techKBS8 a4_rfkb23 asus_laptop 
		btc6301urf \
		dexxa dtk2000 fscaa1667g geniuskb19e geniuscomfy 
		latitude \
		microsoftpro precision_m scorpius sp_inet 
		targa_v811 thinkpad \
		tm2030USB-102 tm2030USB-106 trust_slimline
--8<---------------cut here---------------end--------------->8---

This is causing issues when building my system config, as 
‘(keyboard-layout "us")’ with core-updates returns a syntax error 
due to the above.

I'm not sure what's going on here, as it doesn't appear as though 
the package has any custom rules. It did change from 
‘gnu-build-system’ to ‘meson-build-system’ with the version bump, 
though, so maybe there's something in there?

-bjc




Information forwarded to bug-guix <at> gnu.org:
bug#62820; Package guix. (Sat, 15 Apr 2023 01:17:02 GMT) Full text and rfc822 format available.

Message #8 received at 62820 <at> debbugs.gnu.org (full text, mbox):

From: Felix Lechner <felix.lechner <at> lease-up.com>
To: 62820 <at> debbugs.gnu.org
Cc: Felix Lechner <felix.lechner <at> lease-up.com>, Brian Cully <bjc <at> spork.org>
Subject: [PATCH] gnu: xkeyboard-config: Update to 2.37. (Closes: #62820,
 #58402)
Date: Fri, 14 Apr 2023 18:15:49 -0700
For legacy reasons, Guix used to offer several version of Meson, but in a
recent core-updates branch all versions prior to 1.1.0 had been retired. [1]
By comparison, the most recent version available in the master branch is
0.63.2, which was released just seven months before meson 1.1.0 was cut. [2]

Meanwhile, some software may not have been ready for that transition. I have
not been able to confirm the suspicion for version 2.36 of xkeyboard-config
via online searches or the respective Git log, but updating xkeyboard-config
to 2.37 solved what has been reported as the last remaining issue in
core-updates [3] for me locally.

Here is the same excerpt quoted in Issue#62820 but without the offending
forward slashes for multi-line continuations. It was produced after this patch
was applied to commit 3bacd3c7 of core-updates:

! $inetmediakbds = \
		a4_rfkb23 a4techKB21 a4techKBS8 acer_ferrari4k acer_laptop \
		armada asus_laptop benqx btc5090 btc6301urf btc9019u \
		cherrybluea cherryblueb cherrycyboard chicony042 \
		compalfl90 compaqik13 compaqik18 creativedw7000 \
		cymotionlinux dellm65 dellusbmm dexxa diamond dtk2000 \
		emachines ennyah_dkb1008 fscaa1667g genius geniuscomfy \
		geniuscomfy2 geniuskb19e hp5xx hpdv5 hpi6 hpxe3gc hpxe3gf \
		hpxe4xxx hpxt1000 hpzt11xx inspiron latitude \
		logidinovo logidinovoedge logitech_base logitech_g15 \
		microsoft4000 microsoft7000 microsoftmult microsoftpro \
		microsoftprooem mx1998 mx2500 mx2750 pc105 precision_m \
		presario propeller samsung4500 samsung4510 scorpius \
		silvercrest sk1300 sk2500 sk7100 sp_inet targa_v811 \
		thinkpad thinkpad60 tm2030USB-102 tm2030USB-106 \
		toshiba_s3000 trust trustda trust_slimline unitekkb1925

The data above was extracted from the file

    /gnu/store/5j9synbgzbdjjf2gkyfy9mj83d213qkh-xkeyboard-config-2.37/share/X11/xkb/rules/base

That path contains an extra component 'xkb' relative to the location
identified in Issue#62820 but I believe the path 'share/X11/rules/base' there
was misquoted accidentally.

For the sake of completeness, I'll also mention that xkeyboard-config was
likewise updated in core-updates. The version originally went from 2.34 to
2.36 [4] before being updated to 2.37 herein.

[1] https://git.savannah.gnu.org/cgit/guix.git/commit/?id=311255adc0d1c3b3aa0f6bed5af58e718f0607b2
[2] https://github.com/mesonbuild/meson/tags
[3] https://lists.gnu.org/archive/html/guix-devel/2023-04/msg00214.html
[4] https://issues.guix.gnu.org/57872

* gnu/packages/xorg.scm (xkeyboard-config): Update to 2.37.
---
 gnu/packages/xorg.scm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/xorg.scm b/gnu/packages/xorg.scm
index c510547559..94a59bd205 100644
--- a/gnu/packages/xorg.scm
+++ b/gnu/packages/xorg.scm
@@ -3709,7 +3709,7 @@ (define-public xkbutils
 (define-public xkeyboard-config
   (package
     (name "xkeyboard-config")
-    (version "2.36")
+    (version "2.37")
     (source (origin
               (method url-fetch)
               (uri (string-append
@@ -3717,7 +3717,7 @@ (define-public xkeyboard-config
                     "xkeyboard-config-" version ".tar.xz"))
               (sha256
                (base32
-                "158m7r6ga7w12ry35q6d0z6hilbpj9h7ilw56h55478n58lv26qz"))))
+                "0v4clrxrjygpy5arnb2x3ksvsah5mdprsarhqzbi0qjbmjjq64zb"))))
     (build-system meson-build-system)
     (inputs (list libx11 xkbcomp-intermediate))
     (native-inputs (list gettext-minimal libxslt perl pkg-config python))

base-commit: 3bacd3c76a58ebe70f98be654f09cbd4166093ab
-- 
2.39.2





Reply sent to Maxim Cournoyer <maxim.cournoyer <at> gmail.com>:
You have taken responsibility. (Sat, 15 Apr 2023 04:19:02 GMT) Full text and rfc822 format available.

Notification sent to Brian Cully <bjc <at> spork.org>:
bug acknowledged by developer. (Sat, 15 Apr 2023 04:19:02 GMT) Full text and rfc822 format available.

Message #13 received at 62820-done <at> debbugs.gnu.org (full text, mbox):

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: Felix Lechner <felix.lechner <at> lease-up.com>
Cc: 62820-done <at> debbugs.gnu.org, Brian Cully <bjc <at> spork.org>
Subject: Re: bug#62820: [core-updates] error building xkeyboard-config
Date: Sat, 15 Apr 2023 00:18:00 -0400
Hi,

Felix Lechner <felix.lechner <at> lease-up.com> writes:

[...]

> diff --git a/gnu/packages/xorg.scm b/gnu/packages/xorg.scm
> index c510547559..94a59bd205 100644
> --- a/gnu/packages/xorg.scm
> +++ b/gnu/packages/xorg.scm
> @@ -3709,7 +3709,7 @@ (define-public xkbutils
>  (define-public xkeyboard-config
>    (package
>      (name "xkeyboard-config")
> -    (version "2.36")
> +    (version "2.37")
>      (source (origin
>                (method url-fetch)
>                (uri (string-append
> @@ -3717,7 +3717,7 @@ (define-public xkeyboard-config
>                      "xkeyboard-config-" version ".tar.xz"))
>                (sha256
>                 (base32
> -                "158m7r6ga7w12ry35q6d0z6hilbpj9h7ilw56h55478n58lv26qz"))))
> +                "0v4clrxrjygpy5arnb2x3ksvsah5mdprsarhqzbi0qjbmjjq64zb"))))
>      (build-system meson-build-system)
>      (inputs (list libx11 xkbcomp-intermediate))
>      (native-inputs (list gettext-minimal libxslt perl pkg-config python))
>
> base-commit: 3bacd3c76a58ebe70f98be654f09cbd4166093ab

I've updated to 2.38 on core-updates now (I'll push soon).  Thanks for
the report and fix suggestion.

-- 
Thanks,
Maxim




bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Sat, 13 May 2023 11:24:05 GMT) Full text and rfc822 format available.

This bug report was last modified 2 years and 34 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.