GNU bug report logs - #26808
[PATCH] serf: add missing '#:modules' imports

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 26808 in the body.
You can then email your comments to 26808 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: Sergei Trofimovich <slyfox <at> inbox.ru>
To: guix-patches <at> gnu.org
Cc: Sergei Trofimovich <slyfox <at> inbox.ru>
Subject: [PATCH] serf: add missing '#:modules' imports
Date: Sat,  6 May 2017 23:24:08 +0100

Notced as a 'serf' build failure:

/gnu/store/...-serf-1.3.8-guile-builder:1:2956: source expression failed
to match any pattern in form (%modify-phases phases* (delete (quote configure)))

* gnu/packages/web.scm (serf): add missing '#:modules' imports

Signed-off-by: Sergei Trofimovich <slyfox <at> inbox.ru>
---
 gnu/packages/web.scm | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/web.scm b/gnu/packages/web.scm
index 7215dcbed..24141b5ec 100644
--- a/gnu/packages/web.scm
+++ b/gnu/packages/web.scm
@@ -832,7 +832,10 @@ from streaming URLs.  It is a command-line wrapper for the libquvi library.")
        ;;("gss" ,gss)
        ("zlib" ,zlib)))
     (arguments
-     `(#:phases
+     `(#:modules ((guix build gnu-build-system)
+                  (guix build utils)
+                  (srfi srfi-1))
+       #:phases
        ;; TODO: Add scons-build-system and use it here.
        (modify-phases %standard-phases
          (delete 'configure)
-- 
2.12.2

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

From: Danny Milosavljevic <dannym <at> scratchpost.org>
To: Sergei Trofimovich <slyfox <at> inbox.ru>
Cc: 26808 <at> debbugs.gnu.org
Subject: Re: bug#26808: [PATCH] serf: add missing '#:modules' imports
Date: Sun, 7 May 2017 18:13:31 +0200

Hi,

is this for core-updates ? I've applied it to master - but your earlier similar mails mentioned core-updates.

Same question about re2.

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

From: Sergei Trofimovich <slyfox <at> inbox.ru>
To: Danny Milosavljevic <dannym <at> scratchpost.org>
Cc: 26808 <at> debbugs.gnu.org
Subject: Re: bug#26808: [PATCH] serf: add missing '#:modules' imports
Date: Sun, 7 May 2017 17:50:40 +0100

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

On Sun, 7 May 2017 18:13:31 +0200
Danny Milosavljevic <dannym <at> scratchpost.org> wrote:

> Hi,
> 
> is this for core-updates ? I've applied it to master - but your earlier similar mails mentioned core-updates.
> 
> Same question about re2.

Yes, all of those are for 'code-updates' branch.
My apologies. I keep forgetting mentioning it.

-- 

  Sergei

[Message part 2 (application/pgp-signature, inline)]

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

From: Danny Milosavljevic <dannym <at> scratchpost.org>
To: Sergei Trofimovich <slyfox <at> inbox.ru>
Cc: 26808 <at> debbugs.gnu.org
Subject: Re: bug#26808: [PATCH] serf: add missing '#:modules' imports
Date: Sun, 7 May 2017 20:22:50 +0200

Ok, I've pushed both to core-updates as well now.

Previous Next

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