diff mbox series

[bug#51415] In the Artanis package the propagated input of guile-json is conflicting with guix, and no series of guix upgrades or guix installs seems to fix this. Therefore, I moved guile-json from a propagated input to an input, which solves the issue, b

Message ID 20211026124602.28283-1-blake@nonconstructivism.com
State New
Headers show
Series [bug#51415] In the Artanis package the propagated input of guile-json is conflicting with guix, and no series of guix upgrades or guix installs seems to fix this. Therefore, I moved guile-json from a propagated input to an input, which solves the issue, b | expand

Checks

Context Check Description
cbaines/comparison success View comparision
cbaines/git branch success View Git branch
cbaines/applying patch success View Laminar job
cbaines/issue success View issue

Commit Message

Blake Shaw Oct. 26, 2021, 12:46 p.m. UTC
---
 gnu/packages/guile-xyz.scm | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

Comments

Leo Famulari Oct. 26, 2021, 5:46 p.m. UTC | #1
On Tue, Oct 26, 2021 at 07:46:02PM +0700, Blake Shaw via Guix-patches via wrote:
> In the Artanis package the propagated input of guile-json is
> conflicting with guix, and no series of guix upgrades or guix installs
> seems to fix this. Therefore, I moved guile-json from a propagated
> input to an input, which solves the issue, but may not be ideal for
> people who don't wish to keep multiple copies of guile-json

Thanks for the report... can you share the warnings or error messages
that you are seeing?

It's likely that guile-json is propagated for a good reason, and that
not propagating it would cause problems for artanis users. Maybe there
is another way to solve the problem that you're experiencing.
Ludovic Courtès Nov. 8, 2021, 9:08 p.m. UTC | #2
Hi,

Leo Famulari <leo@famulari.name> skribis:

> On Tue, Oct 26, 2021 at 07:46:02PM +0700, Blake Shaw via Guix-patches via wrote:
>> In the Artanis package the propagated input of guile-json is
>> conflicting with guix, and no series of guix upgrades or guix installs
>> seems to fix this. Therefore, I moved guile-json from a propagated
>> input to an input, which solves the issue, but may not be ideal for
>> people who don't wish to keep multiple copies of guile-json
>
> Thanks for the report... can you share the warnings or error messages
> that you are seeing?

I think the problem Blake describes is this:

--8<---------------cut here---------------start------------->8---
$ guix install artanis guix -p /tmp/test
La jenaj pakoj estos instalataj:
    artanis 0.5
   guix    1.3.0-11.014f1b6

guix install: error: profile contains conflicting entries for guile-json
guix install: error:   first entry: guile-json@4.5.2 /gnu/store/lxbvzmdcv82ik37z4np3c45p11iic7qx-guile-json-4.5.2
guix install: error:    ... propagated from guix@1.3.0-11.014f1b6
guix install: error:   second entry: guile-json@3.5.0 /gnu/store/6s765nvcy77cla92wxzwhakglzlpv362-guile-json-3.5.0
guix install: error:    ... propagated from artanis@0.5
hint: Try upgrading both `guix' and `artanis', or remove one of them from the profile.
--8<---------------cut here---------------end--------------->8---

Removing Guix from that profile solves the problem:

--8<---------------cut here---------------start------------->8---
$ guix install artanis  -p /tmp/test
La jena pako estos instalata:
    artanis 0.5

La jena derivaĵo estos konstruata:
   /gnu/store/jgd870rpvgxjqnlbsnlqya0libg1107v-profile.drv

building profile with 1 package...

[...]
--8<---------------cut here---------------end--------------->8---

> It's likely that guile-json is propagated for a good reason, and that
> not propagating it would cause problems for artanis users. Maybe there
> is another way to solve the problem that you're experiencing.

Yeah, though it seems that the ‘wrap-art’ phase already does the right
thing.

Blake, could you confirm that Artanis is still usable after this change,
even when guile-json is not around in GUILE_LOAD_PATH?

Thanks,
Ludo’.
Maxim Cournoyer Jan. 3, 2023, 10:34 p.m. UTC | #3
Hello Blake,

[...]

> Blake, could you confirm that Artanis is still usable after this change,
> even when guile-json is not around in GUILE_LOAD_PATH?

Friendly ping, regarding the above question from Ludovic.
diff mbox series

Patch

diff --git a/gnu/packages/guile-xyz.scm b/gnu/packages/guile-xyz.scm
index 355b23f57e..2d08739287 100644
--- a/gnu/packages/guile-xyz.scm
+++ b/gnu/packages/guile-xyz.scm
@@ -169,14 +169,14 @@  (define-public artanis
     (inputs
      `(("guile" ,guile-3.0)
        ("nspr" ,nspr)
-       ("nss" ,nss)))
+       ("nss" ,nss)
+       ("guile-json" ,guile-json-3)))
     ;; FIXME the bundled csv contains one more exported procedure
     ;; (sxml->csv-string) than guile-csv. The author is maintainer of both
     ;; projects.
     ;; TODO: Add guile-dbi and guile-dbd optional dependencies.
     (propagated-inputs
-     `(("guile-json" ,guile-json-3)
-       ("guile-readline" ,guile-readline)
+     `(("guile-readline" ,guile-readline)
        ("guile-redis" ,guile-redis)))
     (native-inputs
      `(("bash"       ,bash)         ;for the `source' builtin
@@ -4870,4 +4870,4 @@  (define-public guile-netlink
 It provides a generic library for writing implementations of a netlink
 protocol, a low-level rtnetlink implementation that uses that library and a
 high-level API for network management that uses rtnetlink.")
-    (license license:gpl3+)))
+    (license license:gpl3+)))