GNU bug report logs - #52137
committer.scm wrong type passed to car

Previous Next

To reply to this bug, email your comments to 52137 AT debbugs.gnu.org.

Toggle the display of automated, internal messages from the tracker.

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

From: Jack Hill <jackhill <at> jackhill.us>
To: bug-guix <at> gnu.org
Cc: Ricardo Wurmus <rekado <at> elephly.net>
Subject: committer.scm wrong type passed to car
Date: Sat, 27 Nov 2021 01:41:20 -0500 (EST)

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

Hi Guix and Ricardo,

Running the etc/commiter.scm script with the diff of a patch that I 
recently submitted [0] results in the backtrace below. This might be 
because I was holding the script wrong, but it would be nice to provide a 
more friendly message for people like me who don't remember how to use it 
properly.

[0] https://issues.guix.gnu.org/52134#1

jackhill <at> leyden ~/repos/guix [env]$ git status
On branch committer-test
Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git restore <file>..." to discard changes in working directory)
modified:   gnu/packages/web-browsers.scm

no changes added to commit (use "git add" and/or "git commit -a")
jackhill <at> leyden ~/repos/guix [env]$ git diff
diff --git a/gnu/packages/web-browsers.scm b/gnu/packages/web-browsers.scm
index 7ea2a9a9ec..50a41309d3 100644
--- a/gnu/packages/web-browsers.scm
+++ b/gnu/packages/web-browsers.scm
@@ -51,6 +51,7 @@ (define-module (gnu packages web-browsers)
   #:use-module (gnu packages)
   #:use-module (gnu packages autotools)
   #:use-module (gnu packages backup)
+  #:use-module (gnu packages base)
   #:use-module (gnu packages bison)
   #:use-module (gnu packages compression)
   #:use-module (gnu packages curl)
@@ -125,10 +126,12 @@ (define-public midori
            (assoc-ref glib-or-gtk:%standard-phases
                       'glib-or-gtk-wrap)))))
     (native-inputs
-     `(("glib:bin" ,glib "bin")
+     `(("desktop-file-utils" ,desktop-file-utils) ;for tests
+       ("glib:bin" ,glib "bin")
        ("gtk+:bin" ,gtk+ "bin")
        ("intltool" ,intltool)
-       ("pkg-config" ,pkg-config)))
+       ("pkg-config" ,pkg-config)
+       ("which" ,which))) ;for tests
     (inputs
      `(("adwaita-icon-theme" ,adwaita-icon-theme)
        ("gcr" ,gcr)
@@ -139,10 +142,9 @@ (define-public midori
        ("json-glib" ,json-glib)
        ("libarchive" ,libarchive)
        ("libpeas" ,libpeas)
-       ("libsoup" ,libsoup)
        ("sqlite" ,sqlite)
        ("vala" ,vala)
-       ("webkitgtk" ,webkitgtk)))
+       ("webkitgtk" ,webkitgtk-with-libsoup2)))
     (synopsis "Lightweight graphical web browser")
     (description "@code{Midori} is a lightweight, Webkit-based web browser.
 It features integration with GTK+3, configurable web search engine, bookmark
jackhill <at> leyden ~/repos/guix [env]$ ./etc/committer.scm
Backtrace:
           5 (primitive-load "/home/jackhill/repos/guix/./etc/commit…")
In srfi/srfi-1.scm:
    634:9  4 (for-each #<procedure 7fefd554e4c0 at ice-9/eval.scm:3…> …)
In ice-9/eval.scm:
    619:8  3 (_ #(#(#(#(#(#(#(#(#(#(#(…) …) …) …) …) …) …) …) …) …) …))
    619:8  2 (_ #(#(#(#<directory (guile-user) 7fefd7cbbc80>) # …) …))
   626:19  1 (_ #(#(#(#<directory (guile-user) 7fefd7cbbc80>) # …) …))
In unknown file:
           0 (car ())

ERROR: In procedure car:
In procedure car: Wrong type (expecting pair): ()
Aborting commit due to empty commit message.

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

From: Ricardo Wurmus <rekado <at> elephly.net>
To: Jack Hill <jackhill <at> jackhill.us>
Cc: bug-guix <at> gnu.org
Subject: Re: committer.scm wrong type passed to car
Date: Sat, 27 Nov 2021 09:48:42 +0100

Hi Jack,

> Running the etc/commiter.scm script with the diff of a patch 
> that I
> recently submitted [0] results in the backtrace below. This 
> might be
> because I was holding the script wrong, but it would be nice to
> provide a more friendly message for people like me who don't 
> remember
> how to use it properly.

Thanks for the report!

It’s probably the change in the module header.  When I first wrote 
committer.scm I intended to use it to automate simple package 
updates (for mass updates of CRAN packages).  That never required 
changes to the module header.  I’m sure this can be fixed by 
merely ignoring changes to that form.

-- 
Ricardo

Previous Next

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