GNU bug report logs - #69778
[PATCH gnome-team] gnu: telegram: Build with glibmm@2.76.

Previous Next

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

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

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

From: Vivien Kraus <vivien <at> planete-kraus.eu>
To: guix-patches <at> gnu.org
Subject: [PATCH gnome-team] gnu: telegram: Build with glibmm <at> 2.76.
Date: Wed, 13 Mar 2024 17:45:07 +0100

* gnu/packages/telegram.scm (glibmm-for-telegram-desktop): New variable.
(telegram-desktop): Use glibmm-for-telegram-desktop instead of glibmm.

Change-Id: I0ec4ebc549dd867db22d68ee6317d1630cf1bfd8
---

Dear Guix,

While we are waiting for the telegram-desktop update in #67493, here is a
minimally intrusive way to keep it working on gnome-team, by using an older
version of glibmm.

Best regards,

Vivien

 gnu/packages/telegram.scm | 19 ++++++++++++++++++-
 1 file changed, 18 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/telegram.scm b/gnu/packages/telegram.scm
index 7d02824efa..16d36b2c7c 100644
--- a/gnu/packages/telegram.scm
+++ b/gnu/packages/telegram.scm
@@ -74,7 +74,9 @@ (define-module (gnu packages telegram)
   #:use-module ((guix licenses) #:prefix license:)
   #:use-module (guix packages)
   #:use-module (guix gexp)
+  #:use-module (guix download)
   #:use-module (guix git-download)
+  #:use-module (guix utils)
   #:use-module (guix build-system cmake)
   #:use-module (guix build-system copy)
   #:use-module (guix build-system glib-or-gtk)
@@ -267,6 +269,21 @@ (define tgcalls-for-telegram-desktop
      (base32
       "193m2gkvipijqbfd6a8mhg9nd63wlnshzgspk3pip57vk21l709z"))))
 
+(define-public glibmm-for-telegram-desktop
+  (hidden-package
+   (package
+     (inherit glibmm)
+     (name "glibmm-for-telegram-desktop")
+     (version "2.76.0")
+     (source (origin
+               (method url-fetch)
+               (uri (string-append "mirror://gnome/sources/glibmm/"
+                                   (version-major+minor version)
+                                   "/glibmm-" version ".tar.xz"))
+               (sha256
+                (base32
+                 "1cia8vrpwzn8zwalws42mga5hi965840m5s8dvfzv55xx86dhdw6")))))))
+
 (define-public webrtc-for-telegram-desktop
   (let ((commit "0532942ac6176a66ef184fb728a4cbb02958fc0b")
         (revision "389"))
@@ -500,7 +517,7 @@ (define-public telegram-desktop
            fcitx5-qt
            ffmpeg
            glib
-           glibmm
+           glibmm-for-telegram-desktop
            gtk+
            hime
            hunspell

base-commit: 2ee54513196bad8e663e78ac695b6ffa0da49051
-- 
2.41.0

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

From: Liliana Marie Prikler <liliana.prikler <at> gmail.com>
To: Vivien Kraus <vivien <at> planete-kraus.eu>, 69778 <at> debbugs.gnu.org
Cc: rg <at> raghavgururajan.name, maxim.cournoyer <at> gmail.com
Subject: Re: [bug#69778] [PATCH gnome-team] gnu: telegram: Build with
 glibmm <at> 2.76.
Date: Wed, 13 Mar 2024 22:59:14 +0100

Am Mittwoch, dem 13.03.2024 um 17:45 +0100 schrieb Vivien Kraus:
> * gnu/packages/telegram.scm (glibmm-for-telegram-desktop): New
> variable.
> (telegram-desktop): Use glibmm-for-telegram-desktop instead of
> glibmm.
> 
> Change-Id: I0ec4ebc549dd867db22d68ee6317d1630cf1bfd8
> ---
> 
> Dear Guix,
> 
> While we are waiting for the telegram-desktop update in #67493, here
> is a minimally intrusive way to keep it working on gnome-team, by
> using an older version of glibmm.
> 
> Best regards,
> 
> Vivien
> 
>  gnu/packages/telegram.scm | 19 ++++++++++++++++++-
>  1 file changed, 18 insertions(+), 1 deletion(-)
> 
> diff --git a/gnu/packages/telegram.scm b/gnu/packages/telegram.scm
> index 7d02824efa..16d36b2c7c 100644
> --- a/gnu/packages/telegram.scm
> +++ b/gnu/packages/telegram.scm
> @@ -74,7 +74,9 @@ (define-module (gnu packages telegram)
>    #:use-module ((guix licenses) #:prefix license:)
>    #:use-module (guix packages)
>    #:use-module (guix gexp)
> +  #:use-module (guix download)
>    #:use-module (guix git-download)
> +  #:use-module (guix utils)
>    #:use-module (guix build-system cmake)
>    #:use-module (guix build-system copy)
>    #:use-module (guix build-system glib-or-gtk)
> @@ -267,6 +269,21 @@ (define tgcalls-for-telegram-desktop
>       (base32
>        "193m2gkvipijqbfd6a8mhg9nd63wlnshzgspk3pip57vk21l709z"))))
>  
> +(define-public glibmm-for-telegram-desktop
> +  (hidden-package
> +   (package
> +     (inherit glibmm)
> +     (name "glibmm-for-telegram-desktop")
> +     (version "2.76.0")
> +     (source (origin
> +               (method url-fetch)
> +               (uri (string-append "mirror://gnome/sources/glibmm/"
> +                                   (version-major+minor version)
> +                                   "/glibmm-" version ".tar.xz"))
> +               (sha256
> +                (base32
> +                
> "1cia8vrpwzn8zwalws42mga5hi965840m5s8dvfzv55xx86dhdw6")))))))
> +
Canonically, inherited packages are put in the same files as the
package they inherit.

Cheers

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

From: Vivien Kraus <vivien <at> planete-kraus.eu>
To: 69778 <at> debbugs.gnu.org
Cc: rg <at> raghavgururajan.name, vivien <at> planete-kraus.eu, liliana.prikler <at> gmail.com,
 maxim.cournoyer <at> gmail.com
Subject: [PATCH gnome-team v2 1/2] gnu: Add glibmm-2.76.
Date: Thu, 14 Mar 2024 18:29:27 +0100

* gnu/packages/glib.scm (glibmm-2.76): New variable.

Change-Id: I6d38e7640f9362ca7b6d26611db914da74ead53d
---
 gnu/packages/glib.scm | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/gnu/packages/glib.scm b/gnu/packages/glib.scm
index aef2b7a941..5848ebc7aa 100644
--- a/gnu/packages/glib.scm
+++ b/gnu/packages/glib.scm
@@ -995,6 +995,20 @@ (define glibmm
 useful for C++.")
     (license license:lgpl2.1+)))
 
+(define-public glibmm-2.76
+  (package
+    (inherit glibmm)
+    (name "glibmm")
+    (version "2.76.0")
+    (source (origin
+              (method url-fetch)
+              (uri (string-append "mirror://gnome/sources/glibmm/"
+                                  (version-major+minor version)
+                                  "/glibmm-" version ".tar.xz"))
+              (sha256
+               (base32
+                "1cia8vrpwzn8zwalws42mga5hi965840m5s8dvfzv55xx86dhdw6"))))))
+
  (define-public glibmm-2.66
    (package
     (inherit glibmm)
-- 
2.41.0

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

From: Vivien Kraus <vivien <at> planete-kraus.eu>
To: 69778 <at> debbugs.gnu.org
Cc: rg <at> raghavgururajan.name, vivien <at> planete-kraus.eu, liliana.prikler <at> gmail.com,
 maxim.cournoyer <at> gmail.com
Subject: [PATCH gnome-team v2 0/2] Make the glibmm-2.76 variable public in
 (gnu packages glib)
Date: Thu, 14 Mar 2024 19:06:38 +0100

Dear Guix,

Telegram-desktop wants to be built with glibmm <at> 2.76. I originally added it to
(gnu packages telegram), but in this revision, I add it to (gnu packages
glib).  What do you think?

Best regards,

Vivien

Vivien Kraus (2):
  gnu: Add glibmm-2.76.
  gnu: telegram: Build with glibmm <at> 2.76.

 gnu/packages/glib.scm     | 14 ++++++++++++++
 gnu/packages/telegram.scm |  2 +-
 2 files changed, 15 insertions(+), 1 deletion(-)


base-commit: 2ee54513196bad8e663e78ac695b6ffa0da49051
-- 
2.41.0

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

From: Vivien Kraus <vivien <at> planete-kraus.eu>
To: 69778 <at> debbugs.gnu.org
Cc: rg <at> raghavgururajan.name, vivien <at> planete-kraus.eu, liliana.prikler <at> gmail.com,
 maxim.cournoyer <at> gmail.com
Subject: [PATCH gnome-team v2 2/2] gnu: telegram: Build with glibmm <at> 2.76.
Date: Wed, 13 Mar 2024 17:45:07 +0100

* gnu/packages/telegram.scm (telegram-desktop): Use glibmm-2.76 instead of glibmm.

Change-Id: I0ec4ebc549dd867db22d68ee6317d1630cf1bfd8
---
 gnu/packages/telegram.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gnu/packages/telegram.scm b/gnu/packages/telegram.scm
index 7d02824efa..358374a5bb 100644
--- a/gnu/packages/telegram.scm
+++ b/gnu/packages/telegram.scm
@@ -500,7 +500,7 @@ (define-public telegram-desktop
            fcitx5-qt
            ffmpeg
            glib
-           glibmm
+           glibmm-2.76
            gtk+
            hime
            hunspell
-- 
2.41.0

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

From: Liliana Marie Prikler <liliana.prikler <at> gmail.com>
To: Vivien Kraus <vivien <at> planete-kraus.eu>, 69778 <at> debbugs.gnu.org
Cc: rg <at> raghavgururajan.name, maxim.cournoyer <at> gmail.com
Subject: Re: [PATCH gnome-team v2 0/2] Make the glibmm-2.76 variable public
 in (gnu packages glib)
Date: Thu, 14 Mar 2024 20:17:35 +0100

Am Donnerstag, dem 14.03.2024 um 19:06 +0100 schrieb Vivien Kraus:
> Dear Guix,
> 
> Telegram-desktop wants to be built with glibmm <at> 2.76. I originally
> added it to (gnu packages telegram), but in this revision, I add it
> to (gnu packages glib).  What do you think?
I won't be able to build it on my machine, but assuming you tested it
LGTM.

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

From: Christopher Baines <mail <at> cbaines.net>
To: Liliana Marie Prikler <liliana.prikler <at> gmail.com>
Cc: 69778-done <at> debbugs.gnu.org, Vivien Kraus <vivien <at> planete-kraus.eu>,
 guix-patches <at> gnu.org
Subject: Re: [bug#69778] [PATCH gnome-team v2 0/2] Make the glibmm-2.76
 variable public in (gnu packages glib)
Date: Sat, 16 Mar 2024 11:04:27 +0000

[Message part 1 (text/plain, inline)]

Liliana Marie Prikler <liliana.prikler <at> gmail.com> writes:

> Am Donnerstag, dem 14.03.2024 um 19:06 +0100 schrieb Vivien Kraus:
>> Dear Guix,
>> 
>> Telegram-desktop wants to be built with glibmm <at> 2.76. I originally
>> added it to (gnu packages telegram), but in this revision, I add it
>> to (gnu packages glib).  What do you think?
>
> I won't be able to build it on my machine, but assuming you tested it
> LGTM.

Thanks both, I've pushed this to gnome-team as
aa9aebce615f014070797e7068baca40b4f25c37.

Chris

[signature.asc (application/pgp-signature, inline)]

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

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: Liliana Marie Prikler <liliana.prikler <at> gmail.com>
Cc: 69778 <at> debbugs.gnu.org, rg <at> raghavgururajan.name,
 Vivien Kraus <vivien <at> planete-kraus.eu>
Subject: Re: [bug#69778] [PATCH gnome-team] gnu: telegram: Build with
 glibmm <at> 2.76.
Date: Sat, 16 Mar 2024 21:30:15 -0400

Hi,

Liliana Marie Prikler <liliana.prikler <at> gmail.com> writes:

> Am Mittwoch, dem 13.03.2024 um 17:45 +0100 schrieb Vivien Kraus:
>> * gnu/packages/telegram.scm (glibmm-for-telegram-desktop): New
>> variable.
>> (telegram-desktop): Use glibmm-for-telegram-desktop instead of
>> glibmm.
>> 
>> Change-Id: I0ec4ebc549dd867db22d68ee6317d1630cf1bfd8
>> ---
>> 
>> Dear Guix,
>> 
>> While we are waiting for the telegram-desktop update in #67493, here
>> is a minimally intrusive way to keep it working on gnome-team, by
>> using an older version of glibmm.
>> 
>> Best regards,
>> 
>> Vivien
>> 
>>  gnu/packages/telegram.scm | 19 ++++++++++++++++++-
>>  1 file changed, 18 insertions(+), 1 deletion(-)
>> 
>> diff --git a/gnu/packages/telegram.scm b/gnu/packages/telegram.scm
>> index 7d02824efa..16d36b2c7c 100644
>> --- a/gnu/packages/telegram.scm
>> +++ b/gnu/packages/telegram.scm
>> @@ -74,7 +74,9 @@ (define-module (gnu packages telegram)
>>    #:use-module ((guix licenses) #:prefix license:)
>>    #:use-module (guix packages)
>>    #:use-module (guix gexp)
>> +  #:use-module (guix download)
>>    #:use-module (guix git-download)
>> +  #:use-module (guix utils)
>>    #:use-module (guix build-system cmake)
>>    #:use-module (guix build-system copy)
>>    #:use-module (guix build-system glib-or-gtk)
>> @@ -267,6 +269,21 @@ (define tgcalls-for-telegram-desktop
>>       (base32
>>        "193m2gkvipijqbfd6a8mhg9nd63wlnshzgspk3pip57vk21l709z"))))
>>  
>> +(define-public glibmm-for-telegram-desktop
>> +  (hidden-package
>> +   (package
>> +     (inherit glibmm)
>> +     (name "glibmm-for-telegram-desktop")
>> +     (version "2.76.0")
>> +     (source (origin
>> +               (method url-fetch)
>> +               (uri (string-append "mirror://gnome/sources/glibmm/"
>> +                                   (version-major+minor version)
>> +                                   "/glibmm-" version ".tar.xz"))
>> +               (sha256
>> +                (base32
>> +                
>> "1cia8vrpwzn8zwalws42mga5hi965840m5s8dvfzv55xx86dhdw6")))))))
>> +
> Canonically, inherited packages are put in the same files as the
> package they inherit.

Yes.  That's important to avoid the dreaded circular module import
problem (info "(guix) Cyclic Module Dependencies").

-- 
Thanks,
Maxim

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

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: Vivien Kraus <vivien <at> planete-kraus.eu>
Cc: 69778 <at> debbugs.gnu.org, rg <at> raghavgururajan.name, liliana.prikler <at> gmail.com
Subject: Re: [PATCH gnome-team v2 0/2] Make the glibmm-2.76 variable public
 in (gnu packages glib)
Date: Sat, 16 Mar 2024 21:31:44 -0400

Hi,

Vivien Kraus <vivien <at> planete-kraus.eu> writes:

> Dear Guix,
>
> Telegram-desktop wants to be built with glibmm <at> 2.76. I originally added it to
> (gnu packages telegram), but in this revision, I add it to (gnu packages
> glib).  What do you think?

Is there an incompatibility between our latest glibmm and this version,
or is it just the build system being overly strict?  If the later, it'd
be better to patch it so that it allows our current version.

-- 
Thanks,
Maxim

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

From: Vivien Kraus <vivien <at> planete-kraus.eu>
To: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
Cc: 69778 <at> debbugs.gnu.org, rg <at> raghavgururajan.name, liliana.prikler <at> gmail.com
Subject: Re: [PATCH gnome-team v2 0/2] Make the glibmm-2.76 variable public
 in (gnu packages glib)
Date: Sun, 17 Mar 2024 08:29:09 +0100

Hello!

Le samedi 16 mars 2024 à 21:31 -0400, Maxim Cournoyer a écrit :
> > Telegram-desktop wants to be built with glibmm <at> 2.76. I originally
> > added it to
> > (gnu packages telegram), but in this revision, I add it to (gnu
> > packages
> > glib).  What do you think?
> 
> Is there an incompatibility between our latest glibmm and this
> version,
> or is it just the build system being overly strict?  If the later,
> it'd
> be better to patch it so that it allows our current version.

I did not choose the correct words, sorry. There is an incompatibility
between the 2.76 API and the 2.78 that we use, and this version of the
telegram package can only build against 2.76.

Best regards,

Vivien

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

From: Liliana Marie Prikler <liliana.prikler <at> gmail.com>
To: Vivien Kraus <vivien <at> planete-kraus.eu>, Maxim Cournoyer
 <maxim.cournoyer <at> gmail.com>
Cc: 69778 <at> debbugs.gnu.org, rg <at> raghavgururajan.name
Subject: Re: [PATCH gnome-team v2 0/2] Make the glibmm-2.76 variable public
 in (gnu packages glib)
Date: Sun, 17 Mar 2024 09:30:14 +0100

Am Sonntag, dem 17.03.2024 um 08:29 +0100 schrieb Vivien Kraus:
> Hello!
> 
> Le samedi 16 mars 2024 à 21:31 -0400, Maxim Cournoyer a écrit :
> > > Telegram-desktop wants to be built with glibmm <at> 2.76. I originally
> > > added it to (gnu packages telegram), but in this revision, I add
> > > it to (gnu packages glib).  What do you think?
> > 
> > Is there an incompatibility between our latest glibmm and this
> > version, or is it just the build system being overly strict?  If
> > the later, it'd be better to patch it so that it allows our current
> > version.
> 
> I did not choose the correct words, sorry. There is an
> incompatibility between the 2.76 API and the 2.78 that we use, and
> this version of the telegram package can only build against 2.76.
Could you give a more detailed explanation of this API incompatibility?
The whole point of major versions would be to avoid this kind of thing,
no?

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

From: Vivien Kraus <vivien <at> planete-kraus.eu>
To: Liliana Marie Prikler <liliana.prikler <at> gmail.com>, Maxim Cournoyer
 <maxim.cournoyer <at> gmail.com>
Cc: 69778 <at> debbugs.gnu.org, rg <at> raghavgururajan.name
Subject: Re: [PATCH gnome-team v2 0/2] Make the glibmm-2.76 variable public
 in (gnu packages glib)
Date: Thu, 21 Mar 2024 19:25:39 +0100

Le dimanche 17 mars 2024 à 09:30 +0100, Liliana Marie Prikler a écrit :
> > I did not choose the correct words, sorry. There is an
> > incompatibility between the 2.76 API and the 2.78 that we use, and
> > this version of the telegram package can only build against 2.76.
> Could you give a more detailed explanation of this API
> incompatibility?
> The whole point of major versions would be to avoid this kind of
> thing,
> no?
I don’t have much information, other than “it does not build with
glibmm 2.78”. The Github merge request for a relevant issue does not
give much information either:

https://github.com/desktop-app/lib_base/pull/186

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

From: Liliana Marie Prikler <liliana.prikler <at> gmail.com>
To: Vivien Kraus <vivien <at> planete-kraus.eu>, Maxim Cournoyer
 <maxim.cournoyer <at> gmail.com>
Cc: 69778 <at> debbugs.gnu.org, rg <at> raghavgururajan.name
Subject: Re: [PATCH gnome-team v2 0/2] Make the glibmm-2.76 variable public
 in (gnu packages glib)
Date: Thu, 21 Mar 2024 19:49:32 +0100

Am Donnerstag, dem 21.03.2024 um 19:25 +0100 schrieb Vivien Kraus:
> Le dimanche 17 mars 2024 à 09:30 +0100, Liliana Marie Prikler a
> écrit :
> > > I did not choose the correct words, sorry. There is an
> > > incompatibility between the 2.76 API and the 2.78 that we use,
> > > and
> > > this version of the telegram package can only build against 2.76.
> > Could you give a more detailed explanation of this API
> > incompatibility?
> > The whole point of major versions would be to avoid this kind of
> > thing, no?
> I don’t have much information, other than “it does not build with
> glibmm 2.78”. The Github merge request for a relevant issue does not
> give much information either:
> 
> https://github.com/desktop-app/lib_base/pull/186
Hmm, could we still use that patch and make it "Fix build with glibmm-
2.78"

From a cursory glance, it looks like some type signatures might have
changed, but that's a guess.

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

From: Vivien Kraus <vivien <at> planete-kraus.eu>
To: Liliana Marie Prikler <liliana.prikler <at> gmail.com>, Maxim Cournoyer
 <maxim.cournoyer <at> gmail.com>
Cc: 69778 <at> debbugs.gnu.org, rg <at> raghavgururajan.name
Subject: Re: [PATCH gnome-team v2 0/2] Make the glibmm-2.76 variable public
 in (gnu packages glib)
Date: Thu, 21 Mar 2024 20:51:15 +0100

Le jeudi 21 mars 2024 à 19:49 +0100, Liliana Marie Prikler a écrit :
> > https://github.com/desktop-app/lib_base/pull/186
> Hmm, could we still use that patch and make it "Fix build with
> glibmm-
> 2.78"
> 
> From a cursory glance, it looks like some type signatures might have
> changed, but that's a guess.
I tried to find a combination of patches to backport, but it does not
work very well (this patch ignores other things that were changed
earlier, but that we would need to backport). It’s also not possible to
point lib_base to this exact commit, because it is too recent for other
parts of telegram.

This is not great but hopefully a full upgrade of telegram will fix
everything.

Previous Next

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