[PATCH] cairomm@1.14 propagates both libsigc++@2 and libsigc++@3

  • Open
  • quality assurance status badge
Details
One participant
  • Maxime Devos
Owner
unassigned
Submitted by
Maxime Devos
Severity
normal
M
M
Maxime Devos wrote on 16 Mar 2022 11:55
(address . guix-patches@gnu.org)
44851072e063e7887148dfb44d2a7665c7f94607.camel@telenet.be
Hi,

cairomm-1.14 is defined as:

(define-public cairomm-1.14
(package
(inherit cairomm)
[...]
(propagated-inputs
(modify-inputs (package-propagated-inputs cairomm)
(prepend libsigc++-2)))))

Because cairomm already has a libsigc++ (@3), and 'prepend' is used
instead of 'replace', this causes cairomm@1.14 to have both versions
of libsigc++ as inputs. It appears to be introduced with commit
736c5df29ac380413ec45b496c63dcf80672712d.

Unfortunately, according to "guix refresh -l", fixing this will cause
1980 rebuilds (staging branch?).

The attached patch (currently targetting master) fixes it (cairomm
builds).  I'll try to compile an application with the new cairomm
and report if it works.

Greetings,
Maxime.
From 3a6099ab347e76d030a23ab87dd588baaaae4987 Mon Sep 17 00:00:00 2001
From: Maxime Devos <maximedevos@telenet.be>
Date: Wed, 16 Mar 2022 10:48:25 +0000
Subject: [PATCH] gnu: cairomm-1.14: Don't have multiple variants of libsigc++
as input.
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

This solves a propagation issue reported in

* gnu/packages/gtk.scm (cairomm-1.14)[propagated-inputs]: Replace the
libsigc++ input instead of adding an additional libsigc++ input.

Reported-By: Ludovic Courtès <ludo@gnu.org>
---
gnu/packages/gtk.scm | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

Toggle diff (50 lines)
diff --git a/gnu/packages/gtk.scm b/gnu/packages/gtk.scm
index ced9d621aa..2fd6aa8cfc 100644
--- a/gnu/packages/gtk.scm
+++ b/gnu/packages/gtk.scm
@@ -27,7 +27,7 @@
;;; Copyright © 2020, 2021 Maxim Cournoyer <maxim.cournoyer@gmail.com>
;;; Copyright © 2021 Leo Famulari <leo@famulari.name>
;;; Copyright © 2021 Simon Streit <simon@netpanic.org>
-;;; Copyright © 2021 Maxime Devos <maximedevos@telenet.be>
+;;; Copyright © 2021, 2022 Maxime Devos <maximedevos@telenet.be>
;;; Copyright © 2021 Wamm K. D. <jaft.r@outlook.com>
;;;
;;; This file is part of GNU Guix.
@@ -1598,7 +1598,7 @@ library.")
(base32 "1qwdj9xw1w651kqwh82nipbryimm1ir5n3c6q34nphsx576bj9h1"))))
(propagated-inputs
(modify-inputs (package-propagated-inputs cairomm)
- (prepend libsigc++-2)))))
+ (replace "libsigc++" libsigc++-2)))))
(define-public pangomm
(package

base-commit: 29091731a0c6cb649cdfd72297575fe2bb2a9591
prerequisite-patch-id: 35140f4f2873d0b9f4fc8caca6ec2e013ecb830a
prerequisite-patch-id: ed97d14afd166e7b6cac37e3aa87a85246f7e320
prerequisite-patch-id: 7b2139dc21a7c93817ce12320181270476340a09
prerequisite-patch-id: e8f735697c0535afe9335448b16e3e1f308de362
prerequisite-patch-id: eaf1f67c4c07482fb4da81525cbd5dcb1ea2194e
prerequisite-patch-id: 9a15aa08fbbbf110ba76409dcc2a3ab5e0764806
prerequisite-patch-id: 675a3c516f47dfcbaf61d5ad41ca7f3babdd3f20
prerequisite-patch-id: ac188cb61957c9639d0ac125c941950afbdba9c7
prerequisite-patch-id: 557ba1c38241c1d62f464a97fa0575652a13aff9
prerequisite-patch-id: ece168a472eb532b1ebd12a5245a25ec1c6f4918
prerequisite-patch-id: 5eee450b2221d67fbda1e6581d16628394c912a7
prerequisite-patch-id: 623edc835c2c5dfd8c83dcf32e650cfebea42aa0
prerequisite-patch-id: 277ff341648002cb1739c42819684e67409b6645
prerequisite-patch-id: dcac72251d6ee4f34c3ebd64934063e1ee133045
prerequisite-patch-id: c9593e48965f7f033a855b639133147a62ddd0ad
prerequisite-patch-id: 03ae6cfd97b0f89f92cdd244f35b4c4561c31e26
prerequisite-patch-id: ef8cc21f9671087a7bf93224e32852bbb520cd37
prerequisite-patch-id: 54eec153e523b58c3670c48afda9ef50ec44eb8e
prerequisite-patch-id: bc5dfc06e9d67d10a37fbd7ba61939907d93ca7c
prerequisite-patch-id: 0db9692e872bf73242cfec6f8aa390abe14d08f1
prerequisite-patch-id: 36431a656d29e90e8eb218730c64807e2477c9d7
prerequisite-patch-id: 9146aec4a40f7da60a4c64643a9aa0e405567b04
prerequisite-patch-id: ff2daf978d58ec12c25dcbce4e7ee010d337cd54
prerequisite-patch-id: d7dba04b19c96e1b438291c906bc9e6a1ba5477f
--
2.30.2
-----BEGIN PGP SIGNATURE-----

iI0EABYKADUWIQTB8z7iDFKP233XAR9J4+4iGRcl7gUCYjHCHxccbWF4aW1lZGV2
b3NAdGVsZW5ldC5iZQAKCRBJ4+4iGRcl7r3HAQCpJSk4Gvr0liw79OIhnf5S+S6f
t0xvNfqR3tIjLigSkAD7BStA6SrNVenKa5ecUebatoKz09oNA4vsWlCvM82Cnws=
=iMDN
-----END PGP SIGNATURE-----


M
M
Maxime Devos wrote on 16 Mar 2022 12:50
(address . 54418@debbugs.gnu.org)
12d6d05fbf392218667c38dd1601b3fcfb97d1a0.camel@telenet.be
Maxime Devos schreef op wo 16-03-2022 om 11:55 [+0100]:
Toggle quote (4 lines)
> The attached patch (currently targetting master) fixes it (cairomm
> builds).  I'll try to compile an application with the new cairomm
> and report if it works.

The dependent 'gnote' builds and appears to work.

Greetings,
Maxime.
-----BEGIN PGP SIGNATURE-----

iI0EABYKADUWIQTB8z7iDFKP233XAR9J4+4iGRcl7gUCYjHPEBccbWF4aW1lZGV2
b3NAdGVsZW5ldC5iZQAKCRBJ4+4iGRcl7ggyAQC1bUBkvG8188HgJfjYFpDknZyj
BXhAk5FtshAVgkv1hwEAopLQB6f+HWBdFkvBHVkqXticuGsycq1yejlPYhCjuQY=
=17ug
-----END PGP SIGNATURE-----


M
M
Maxime Devos wrote on 16 Mar 2022 13:38
(address . 54418@debbugs.gnu.org)
257a0bfaff5944b06de16e52e7df1a9f5ec666ef.camel@telenet.be
Maxime Devos schreef op wo 16-03-2022 om 11:55 [+0100]:
Toggle quote (3 lines)
> This solves a propagation issue reported in
> <https://issues.guix.gnu.org/54350>.

Looks like this is not sufficient:

$ ./pre-inst-env guix graph --path inkscape libsigc++
inkscape@1.1.1
gtkmm@3.24.5
glibmm@2.70.0
libsigc++@3.0.6
$ ./pre-inst-env guix graph --path inkscape libsigc++@2
inkscape@1.1.1
gtkmm@3.24.5
cairomm@1.14.2
libsigc++@2.9.3

Still seems an improvement to me though.

Greetings,
Maxime.
-----BEGIN PGP SIGNATURE-----

iI0EABYKADUWIQTB8z7iDFKP233XAR9J4+4iGRcl7gUCYjHaWxccbWF4aW1lZGV2
b3NAdGVsZW5ldC5iZQAKCRBJ4+4iGRcl7uQUAQCejqDjndoqbCzx8fqPYMZtzfeC
npRm8VLYFVnqiRUMtgD/XlFRHwrASBnnTDZqSiTh6n6v/T23oLrNZqlNTuau7gY=
=tlDN
-----END PGP SIGNATURE-----


?
Your comment

Commenting via the web interface is currently disabled.

To comment on this conversation send an email to 54418@debbugs.gnu.org

To respond to this issue using the mumi CLI, first switch to it
mumi current 54418
Then, you may apply the latest patchset in this issue (with sign off)
mumi am -- -s
Or, compose a reply to this issue
mumi compose
Or, send patches to this issue
mumi send-email *.patch