GNU bug report logs

#29106 Build loop after changing pulseaudio and emacs-ido-completing-read+

PackageSource(s)Maintainer(s)
guix PTS Buildd Popcon
Reply or subscribe to this bug. View this bug as an mbox, status mbox, or maintainer mbox

Report forwarded to bug-guix@gnu.org:
bug#29106; Package guix. (Wed, 01 Nov 2017 14:41:03 GMT) (full text, mbox, link).

Acknowledgement sent to Oleg Pykhalov <go.wigust@gmail.com>:
New bug report received and forwarded. Copy sent to bug-guix@gnu.org. (Wed, 01 Nov 2017 14:41:03 GMT) (full text, mbox, link).

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

From: Oleg Pykhalov <go.wigust@gmail.com>
To: bug-guix@gnu.org
Subject: Build loop after changing pulseaudio and emacs-ido-completing-read+
Date: Wed, 01 Nov 2017 17:40:29 +0300
[Message part 1 (text/plain, inline)]
Hello Guix,

I have issue with building loop after changing pulseaudio and
emacs-ido-completing-read+ packages.

                               pulseaudio

diff --git a/gnu/packages/pulseaudio.scm b/gnu/packages/pulseaudio.scm
index 21753cf3f..3c9659b73 100644
--- a/gnu/packages/pulseaudio.scm
+++ b/gnu/packages/pulseaudio.scm
@@ -41,6 +41,7 @@
   #:use-module (gnu packages linux)
   #:use-module (gnu packages m4)
   #:use-module (gnu packages pkg-config)
+  #:use-module (gnu packages qt)
   #:use-module (gnu packages xiph))
 
 (define-public libsndfile
@@ -173,7 +174,8 @@ rates.")
        ("fftwf" ,fftwf)
        ("avahi" ,avahi)
        ("eudev" ,eudev)           ;for the detection of hardware audio devices
-       ("check" ,check)))
+       ("check" ,check)
+       ("python-pyqt" ,python-pyqt)))
     (native-inputs
      `(("pkg-config" ,pkg-config)))
     (propagated-inputs

$ make

$ time ./pre-inst-env guix build --verbosity=5 -n --no-substitutes pulseaudio

$ sort -u ~/screenlog.1

--8<---------------cut here---------------start------------->8---
acquiring global GC lock `/var/guix/gc.lock'
acquiring read lock on `/var/guix/temproots/9147'
acquiring write lock on `/var/guix/temproots/9147'
^C
downgrading to read lock on `/var/guix/temproots/9147'
httmakunitime ./pre-inst-env guix build --verbosity=5 -n --no-substitutes pulseaudio
natsu@magnolia ~/src/guix$ 
real	4m8.823s
sys	0m2.555s
user	4m11.696s
--8<---------------cut here---------------end--------------->8---

                       emacs-ido-completing-read+

diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
index 4c5dd2ba8..7dee7fedf 100644
--- a/gnu/packages/emacs.scm
+++ b/gnu/packages/emacs.scm
@@ -3777,48 +3777,35 @@ restrict the text width to 80 characters.")
 (define-public emacs-ido-completing-read+
   (package
     (name "emacs-ido-completing-read+")
-    (version "3.12")
-    (source (origin
-              (method url-fetch)
-              (uri (string-append "https://raw.githubusercontent.com"
-                                  "/DarwinAwardWinner/ido-ubiquitous/v"
-                                  version "/ido-completing-read+.el"))
-              (file-name (string-append "ido-completing-read+-" version ".el"))
-              (sha256
-               (base32
-                "1cyalb0p7nfsm4n6n9q6rjmvn6adqc0fq8ybnlj3n41n289dkfjf"))))
+    (version "4.5")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (string-append "https://github.com/"
+                           "DarwinAwardWinner/ido-completing-read-plus"
+                           "/archive/" "v" version ".tar.gz"))
+       (file-name (string-append "ido-completing-read+-" version ".el"))
+       (sha256
+        (base32
+         "00p6j3chyir3avdgbkb05hw4rr5xd1vyljfhn7pg67jx04z5a0f3"))))
     (build-system emacs-build-system)
-    (home-page "https://github.com/DarwinAwardWinner/ido-ubiquitous")
-    (synopsis "Replacement for completing-read using ido")
+    (propagated-inputs
+     `(("emacs-ido-completing-read+" ,emacs-ido-completing-read+)
+       ("emacs-s" ,emacs-s)
+       ("emacs-memoize" ,emacs-memoize)))
+    (home-page "https://github.com/DarwinAwardWinner/ido-completing-read-plus")
+    (synopsis "Use ido (nearly) everywhere")
     (description
-     "The ido-completing-read+ function is a wrapper for ido-completing-read.
-Importantly, it detects edge cases that ordinary ido cannot handle and either
-adjusts them so ido can handle them, or else simply falls back to the standard
-Emacs completion function instead.")
+     "@code{emacs-ido-completing-read+} provides ido-style completion for
+almost every function that uses the standard completion function
+completing-read.")
     (license license:gpl3+)))
 
 (define-public emacs-ido-ubiquitous
+  ;; Preserve for people already uses `emacs-ido-ubiquitous'
   (package
-    (name "emacs-ido-ubiquitous")
-    (version "3.12")
-    (source (origin
-              (method url-fetch)
-              (uri (string-append "https://raw.githubusercontent.com"
-                                  "/DarwinAwardWinner/ido-ubiquitous/v"
-                                  version "/ido-ubiquitous.el"))
-              (file-name (string-append "ido-ubiquitous-" version ".el"))
-              (sha256
-               (base32
-                "197ypji0fb6jsdcq40rpnknwlh3imas6s6jbsvkfm0pz9988c3q2"))))
-    (build-system emacs-build-system)
-    (propagated-inputs
-     `(("emacs-ido-completing-read+" ,emacs-ido-completing-read+)))
-    (home-page "https://github.com/DarwinAwardWinner/ido-ubiquitous")
-    (synopsis "Use ido (nearly) everywhere")
-    (description
-     "Ido-ubiquitous enables ido-style completion for almost every function
-that uses the standard completion function completing-read.")
-  (license license:gpl3+)))
+    (inherit emacs-ido-completing-read+)
+    (name "emacs-ido-ubiquitous")))
 
 (define-public emacs-yaml-mode
   (package

$ make

$ time ./pre-inst-env guix build --verbosity=5 -n --no-substitutes emacs-ido-completing-read+

$ sort -u ~/screenlog.1

--8<---------------cut here---------------start------------->8---
acquiring global GC lock `/var/guix/gc.lock'
acquiring read lock on `/var/guix/temproots/11388'
acquiring write lock on `/var/guix/temproots/11388'
^C
downgrading to read lock on `/var/guix/temproots/11388'
natsu@magnolia ~/src/guix$ 
real	4m3.979s
sys	0m1.712s
user	4m6.727s
--8<---------------cut here---------------end--------------->8---

Thanks,
Oleg.

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

Information forwarded to bug-guix@gnu.org:
bug#29106; Package guix. (Wed, 01 Nov 2017 14:48:01 GMT) (full text, mbox, link).

Message #8 received at 29106@debbugs.gnu.org (full text, mbox, reply):

From: Oleg Pykhalov <go.wigust@gmail.com>
To: 29106@debbugs.gnu.org
Subject: Re: bug#29106: Build loop after changing pulseaudio and emacs-ido-completing-read+
Date: Wed, 01 Nov 2017 17:47:29 +0300
[0001-gnu-emacs-ido-completing-read-Update-to-4.5.patch (text/x-patch, inline)]
From 9a012229f41d60610e37639363f3b513a3900465 Mon Sep 17 00:00:00 2001
From: Oleg Pykhalov <go.wigust@gmail.com>
Date: Wed, 1 Nov 2017 15:58:48 +0300
Subject: [PATCH] gnu: emacs-ido-completing-read+: Update to 4.5.

* gnu/packages/emacs.scm (emacs-ido-completing-read+): Update to 4.5.
---
 gnu/packages/emacs.scm | 59 ++++++++++++++++++++------------------------------
 1 file changed, 23 insertions(+), 36 deletions(-)

diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
index ad96093c3..3607e00cb 100644
--- a/gnu/packages/emacs.scm
+++ b/gnu/packages/emacs.scm
@@ -3777,48 +3777,35 @@ restrict the text width to 80 characters.")
 (define-public emacs-ido-completing-read+
   (package
     (name "emacs-ido-completing-read+")
-    (version "3.12")
-    (source (origin
-              (method url-fetch)
-              (uri (string-append "https://raw.githubusercontent.com"
-                                  "/DarwinAwardWinner/ido-ubiquitous/v"
-                                  version "/ido-completing-read+.el"))
-              (file-name (string-append "ido-completing-read+-" version ".el"))
-              (sha256
-               (base32
-                "1cyalb0p7nfsm4n6n9q6rjmvn6adqc0fq8ybnlj3n41n289dkfjf"))))
+    (version "4.5")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (string-append "https://github.com/"
+                           "DarwinAwardWinner/ido-completing-read-plus"
+                           "/archive/" "v" version ".tar.gz"))
+       (file-name (string-append "ido-completing-read+-" version ".el"))
+       (sha256
+        (base32
+         "00p6j3chyir3avdgbkb05hw4rr5xd1vyljfhn7pg67jx04z5a0f3"))))
     (build-system emacs-build-system)
-    (home-page "https://github.com/DarwinAwardWinner/ido-ubiquitous")
-    (synopsis "Replacement for completing-read using ido")
+    (propagated-inputs
+     `(("emacs-ido-completing-read+" ,emacs-ido-completing-read+)
+       ("emacs-s" ,emacs-s)
+       ("emacs-memoize" ,emacs-memoize)))
+    (home-page "https://github.com/DarwinAwardWinner/ido-completing-read-plus")
+    (synopsis "Use ido (nearly) everywhere")
     (description
-     "The ido-completing-read+ function is a wrapper for ido-completing-read.
-Importantly, it detects edge cases that ordinary ido cannot handle and either
-adjusts them so ido can handle them, or else simply falls back to the standard
-Emacs completion function instead.")
+     "@code{emacs-ido-completing-read+} provides ido-style completion for
+almost every function that uses the standard completion function
+completing-read.")
     (license license:gpl3+)))
 
 (define-public emacs-ido-ubiquitous
+  ;; Preserve for people already uses `emacs-ido-ubiquitous'
   (package
-    (name "emacs-ido-ubiquitous")
-    (version "3.12")
-    (source (origin
-              (method url-fetch)
-              (uri (string-append "https://raw.githubusercontent.com"
-                                  "/DarwinAwardWinner/ido-ubiquitous/v"
-                                  version "/ido-ubiquitous.el"))
-              (file-name (string-append "ido-ubiquitous-" version ".el"))
-              (sha256
-               (base32
-                "197ypji0fb6jsdcq40rpnknwlh3imas6s6jbsvkfm0pz9988c3q2"))))
-    (build-system emacs-build-system)
-    (propagated-inputs
-     `(("emacs-ido-completing-read+" ,emacs-ido-completing-read+)))
-    (home-page "https://github.com/DarwinAwardWinner/ido-ubiquitous")
-    (synopsis "Use ido (nearly) everywhere")
-    (description
-     "Ido-ubiquitous enables ido-style completion for almost every function
-that uses the standard completion function completing-read.")
-  (license license:gpl3+)))
+    (inherit emacs-ido-completing-read+)
+    (name "emacs-ido-ubiquitous")))
 
 (define-public emacs-yaml-mode
   (package
-- 
2.14.3


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

Information forwarded to bug-guix@gnu.org:
bug#29106; Package guix. (Sun, 05 Nov 2017 16:08:01 GMT) (full text, mbox, link).

Message #11 received at 29106@debbugs.gnu.org (full text, mbox, reply):

From: ludo@gnu.org (Ludovic Courtès)
To: Oleg Pykhalov <go.wigust@gmail.com>
Cc: 29106@debbugs.gnu.org
Subject: Re: bug#29106: Build loop after changing pulseaudio and emacs-ido-completing-read+
Date: Sun, 05 Nov 2017 17:07:27 +0100
Oleg Pykhalov <go.wigust@gmail.com> skribis:

> From 9a012229f41d60610e37639363f3b513a3900465 Mon Sep 17 00:00:00 2001
> From: Oleg Pykhalov <go.wigust@gmail.com>
> Date: Wed, 1 Nov 2017 15:58:48 +0300
> Subject: [PATCH] gnu: emacs-ido-completing-read+: Update to 4.5.
>
> * gnu/packages/emacs.scm (emacs-ido-completing-read+): Update to 4.5.

Please also mention the ‘emacs-ido-ubiquitous’ changes.


[...]

>  (define-public emacs-ido-ubiquitous
> +  ;; Preserve for people already uses `emacs-ido-ubiquitous'
>    (package
> -    (name "emacs-ido-ubiquitous")
> -    (version "3.12")
> -    (source (origin
> -              (method url-fetch)
> -              (uri (string-append "https://raw.githubusercontent.com"
> -                                  "/DarwinAwardWinner/ido-ubiquitous/v"
> -                                  version "/ido-ubiquitous.el"))
> -              (file-name (string-append "ido-ubiquitous-" version ".el"))
> -              (sha256
> -               (base32
> -                "197ypji0fb6jsdcq40rpnknwlh3imas6s6jbsvkfm0pz9988c3q2"))))
> -    (build-system emacs-build-system)
> -    (propagated-inputs
> -     `(("emacs-ido-completing-read+" ,emacs-ido-completing-read+)))
> -    (home-page "https://github.com/DarwinAwardWinner/ido-ubiquitous")
> -    (synopsis "Use ido (nearly) everywhere")
> -    (description
> -     "Ido-ubiquitous enables ido-style completion for almost every function
> -that uses the standard completion function completing-read.")
> -  (license license:gpl3+)))
> +    (inherit emacs-ido-completing-read+)
> +    (name "emacs-ido-ubiquitous")))

Should it be defined using ‘deprecated-package’ instead?

Also, is the “build loop” you mentioned fixed with this patch?

Thanks,
Ludo’.

Information forwarded to bug-guix@gnu.org:
bug#29106; Package guix. (Fri, 24 Nov 2017 17:27:02 GMT) (full text, mbox, link).

Message #14 received at 29106@debbugs.gnu.org (full text, mbox, reply):

From: ludo@gnu.org (Ludovic Courtès)
To: Oleg Pykhalov <go.wigust@gmail.com>
Cc: 29106@debbugs.gnu.org
Subject: Re: bug#29106: Build loop after changing pulseaudio and emacs-ido-completing-read+
Date: Fri, 24 Nov 2017 18:26:21 +0100
Ping!

ludo@gnu.org (Ludovic Courtès) skribis:

> Oleg Pykhalov <go.wigust@gmail.com> skribis:
>
>> From 9a012229f41d60610e37639363f3b513a3900465 Mon Sep 17 00:00:00 2001
>> From: Oleg Pykhalov <go.wigust@gmail.com>
>> Date: Wed, 1 Nov 2017 15:58:48 +0300
>> Subject: [PATCH] gnu: emacs-ido-completing-read+: Update to 4.5.
>>
>> * gnu/packages/emacs.scm (emacs-ido-completing-read+): Update to 4.5.
>
> Please also mention the ‘emacs-ido-ubiquitous’ changes.
>
>
> [...]
>
>>  (define-public emacs-ido-ubiquitous
>> +  ;; Preserve for people already uses `emacs-ido-ubiquitous'
>>    (package
>> -    (name "emacs-ido-ubiquitous")
>> -    (version "3.12")
>> -    (source (origin
>> -              (method url-fetch)
>> -              (uri (string-append "https://raw.githubusercontent.com"
>> -                                  "/DarwinAwardWinner/ido-ubiquitous/v"
>> -                                  version "/ido-ubiquitous.el"))
>> -              (file-name (string-append "ido-ubiquitous-" version ".el"))
>> -              (sha256
>> -               (base32
>> -                "197ypji0fb6jsdcq40rpnknwlh3imas6s6jbsvkfm0pz9988c3q2"))))
>> -    (build-system emacs-build-system)
>> -    (propagated-inputs
>> -     `(("emacs-ido-completing-read+" ,emacs-ido-completing-read+)))
>> -    (home-page "https://github.com/DarwinAwardWinner/ido-ubiquitous")
>> -    (synopsis "Use ido (nearly) everywhere")
>> -    (description
>> -     "Ido-ubiquitous enables ido-style completion for almost every function
>> -that uses the standard completion function completing-read.")
>> -  (license license:gpl3+)))
>> +    (inherit emacs-ido-completing-read+)
>> +    (name "emacs-ido-ubiquitous")))
>
> Should it be defined using ‘deprecated-package’ instead?
>
> Also, is the “build loop” you mentioned fixed with this patch?
>
> Thanks,
> Ludo’.

Information forwarded to bug-guix@gnu.org:
bug#29106; Package guix. (Wed, 29 Nov 2017 04:47:01 GMT) (full text, mbox, link).

Message #17 received at 29106@debbugs.gnu.org (full text, mbox, reply):

From: Oleg Pykhalov <go.wigust@gmail.com>
To: ludo@gnu.org (Ludovic Courtès)
Cc: 29106@debbugs.gnu.org
Subject: Re: bug#29106: Build loop after changing pulseaudio and emacs-ido-completing-read+
Date: Wed, 29 Nov 2017 07:45:57 +0300
[Message part 1 (text/plain, inline)]
Hello Ludovic,

Apologies for late reply.

ludo@gnu.org (Ludovic Courtès) writes:

> Ping!
>> Also, is the “build loop” you mentioned fixed with this patch?

Sorry, I don't see any patches attached.

I still have never ending pulseaudio derivation build if I add
("python-pyqt" ,python-pyqt) to the inputs field.

--8<---------------cut here---------------start------------->8---
$ ./pre-inst-env guix build --no-substitutes --derivations pulseaudio
--8<---------------cut here---------------end--------------->8---

Thanks,
Oleg.

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

Information forwarded to bug-guix@gnu.org:
bug#29106; Package guix. (Wed, 29 Nov 2017 16:45:01 GMT) (full text, mbox, link).

Message #20 received at 29106@debbugs.gnu.org (full text, mbox, reply):

From: ludo@gnu.org (Ludovic Courtès)
To: Oleg Pykhalov <go.wigust@gmail.com>
Cc: 29106@debbugs.gnu.org
Subject: Re: bug#29106: Build loop after changing pulseaudio and emacs-ido-completing-read+
Date: Wed, 29 Nov 2017 17:43:57 +0100
Hi Oleg,

Oleg Pykhalov <go.wigust@gmail.com> skribis:

> ludo@gnu.org (Ludovic Courtès) writes:
>
>> Ping!
>>> Also, is the “build loop” you mentioned fixed with this patch?
>
> Sorry, I don't see any patches attached.

See <https://bugs.gnu.org/29106> (the issue we’re talking about.)  There
you submitted an emacs-ido-completing-read+ patch¹, and it isn’t clear
to me how it relates to the initial problem you reported.

¹ https://debbugs.gnu.org/cgi/bugreport.cgi?bug=29106#8

> I still have never ending pulseaudio derivation build if I add
> ("python-pyqt" ,python-pyqt) to the inputs field.

OK, I suppose that’s because python-pyqt indirectly depends on
pulseaudio (currently Guix doesn’t try to detect cycles around
derivations, which is why you get this “never-ending” behavior.)

The usual approach to break such cycles is by having a “-minimal”
variant of one of the packages in the loop.

Would it work for you?

Thanks,
Ludo’.

Send a report that this bug log contains spam.

debbugs.gnu.org maintainers <help-debbugs@gnu.org>. Last modified: Wed Apr 16 04:45:52 2025; Machine Name: wallace-server

GNU bug tracking system

Debbugs is free software and licensed under the terms of the GNU Public License version 2. The current version can be obtained from https://bugs.debian.org/debbugs-source/.

Copyright © 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson, 2005-2017 Don Armstrong, and many other contributors.