mbox series

[bug#69778,gnome-team,v2,0/2] Make the glibmm-2.76 variable public in (gnu packages glib)

Message ID cover.1710439598.git.vivien@planete-kraus.eu
Headers show
Series Make the glibmm-2.76 variable public in (gnu packages glib) | expand

Message

Vivien Kraus March 14, 2024, 6:06 p.m. UTC
Dear Guix,

Telegram-desktop wants to be built with glibmm@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@2.76.

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


base-commit: 2ee54513196bad8e663e78ac695b6ffa0da49051

Comments

Liliana Marie Prikler March 14, 2024, 7:17 p.m. UTC | #1
Am Donnerstag, dem 14.03.2024 um 19:06 +0100 schrieb Vivien Kraus:
> Dear Guix,
> 
> Telegram-desktop wants to be built with glibmm@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.
Christopher Baines March 16, 2024, 11:04 a.m. UTC | #2
Liliana Marie Prikler <liliana.prikler@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@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
Maxim Cournoyer March 17, 2024, 1:31 a.m. UTC | #3
Hi,

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

> Dear Guix,
>
> Telegram-desktop wants to be built with glibmm@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.
Vivien Kraus March 17, 2024, 7:29 a.m. UTC | #4
Hello!

Le samedi 16 mars 2024 à 21:31 -0400, Maxim Cournoyer a écrit :
> > Telegram-desktop wants to be built with glibmm@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
Liliana Marie Prikler March 17, 2024, 8:30 a.m. UTC | #5
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@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?
Vivien Kraus March 21, 2024, 6:25 p.m. UTC | #6
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
Liliana Marie Prikler March 21, 2024, 6:49 p.m. UTC | #7
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.
Vivien Kraus March 21, 2024, 7:51 p.m. UTC | #8
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.