GNU bug report logs - #33614
reposurgeon build is broken

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 33614 in the body.
You can then email your comments to 33614 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: Danny Milosavljevic <dannym <at> scratchpost.org>
To: <bug-guix <at> gnu.org>
Subject: reposurgeon build is broken
Date: Tue, 4 Dec 2018 17:47:37 +0100

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

  class OrderedSet(collections.MutableSet):
=== Testing mailbox-out/mailbox-in roundtripping:
  authormatch.fi
/tmp/guix-build-reposurgeon-3.43.drv-0/reposurgeon-3.43/reposurgeon:13680: DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated, and in 3.8 it will stop working
  class OrderedSet(collections.MutableSet):
Traceback (most recent call last):
  File "/tmp/guix-build-reposurgeon-3.43.drv-0/reposurgeon-3.43/reposurgeon", line 1585, in f
    if len(cache) <= pos: cache.append(next(it))
StopIteration

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/tmp/guix-build-reposurgeon-3.43.drv-0/reposurgeon-3.43/reposurgeon", line 13800, in <module>
    main()
  File "/tmp/guix-build-reposurgeon-3.43.drv-0/reposurgeon-3.43/reposurgeon", line 13790, in main
    cmd.Cmd.onecmd(interpreter, interpreter.precmd(arg))
  File "/gnu/store/b7fqhszxl02g6pfm3vw6b3cjz472qrly-python-3.7.0/lib/python3.7/cmd.py", line 217, in onecmd
    return func(arg)
  File "/tmp/guix-build-reposurgeon-3.43.drv-0/reposurgeon-3.43/reposurgeon", line 10761, in do_edit
    self.edit(self.selection, line)
  File "/tmp/guix-build-reposurgeon-3.43.drv-0/reposurgeon-3.43/reposurgeon", line 9339, in edit
    self.do_mailbox_in("<" + tfname)
  File "/tmp/guix-build-reposurgeon-3.43.drv-0/reposurgeon-3.43/reposurgeon", line 10580, in do_mailbox_in
    for commit in self.chosen().commits():
RuntimeError: generator raised StopIteration
make[2]: *** [Makefile:90: mailboxing] Error 1
make[1]: *** [Makefile:24: default] Error 2
make: *** [Makefile:120: check] Error 2

Test suite failed, dumping logs.
Backtrace:
           4 (primitive-load "/gnu/store/k4qpksv3j8yljcb89lc228h78l6â¦")
In ice-9/eval.scm:
   191:35  3 (_ _)
In srfi/srfi-1.scm:
   863:16  2 (every1 #<procedure 80f2c0 at /gnu/store/gfprsx2m62cvqâ¦> â¦)
In /gnu/store/gfprsx2m62cvqbh7ysc9ay9slhijvmal-module-import/guix/build/gnu-build-system.scm:
   799:28  1 (_ _)
    369:6  0 (check #:target _ #:make-flags _ #:tests? _ # _ # _ # _)

/gnu/store/gfprsx2m62cvqbh7ysc9ay9slhijvmal-module-import/guix/build/gnu-build-system.scm:369:6: In procedure check:
Throw to key `srfi-34' with args `(#<condition &invoke-error [program: "make" arguments: ("check" "-j" "4" "ECHO=echo" "target=/gnu/store/0n2bv0yajwi7a8p3dg390i974ja2ldhl-reposurgeon-3.43") exit-status: 2 term-signal: #f stop-signal: #f] 96ca80>)'.

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

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

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: Danny Milosavljevic <dannym <at> scratchpost.org>
Cc: 33614 <at> debbugs.gnu.org
Subject: Re: bug#33614: reposurgeon build is broken
Date: Wed, 13 Jul 2022 09:22:52 -0400

Hi,

I tried updating reposurgeon to see if I could get that resolved, but
reposurgeon is now a Go module, so we'd need to change the build system
and perhaps add a couple supporting Go modules.

Would you like to take a look?

Thanks,

Maxim

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

From: zimoun <zimon.toutoune <at> gmail.com>
To: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
Cc: 33614 <at> debbugs.gnu.org, Danny Milosavljevic <dannym <at> scratchpost.org>
Subject: Re: bug#33614: reposurgeon build is broken
Date: Mon, 17 Oct 2022 22:09:55 +0200

Hi,

On Wed, 13 Jul 2022 at 09:22, Maxim Cournoyer <maxim.cournoyer <at> gmail.com> wrote:

> I tried updating reposurgeon to see if I could get that resolved, but
> reposurgeon is now a Go module, so we'd need to change the build system
> and perhaps add a couple supporting Go modules.
>
> Would you like to take a look?

reposurgeon is broken since a while; [1] is reported on Dec. 2018.

Since no one care enough this package, I am propose to close this issue
by simply removing the package.  WDYT?


1: <https://issues.guix.gnu.org/33614>


Cheers,
simon

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

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: zimoun <zimon.toutoune <at> gmail.com>
Cc: 33614 <at> debbugs.gnu.org, Danny Milosavljevic <dannym <at> scratchpost.org>
Subject: Re: bug#33614: reposurgeon build is broken
Date: Mon, 17 Oct 2022 17:16:48 -0400

Hi,

zimoun <zimon.toutoune <at> gmail.com> writes:

> Hi,
>
> On Wed, 13 Jul 2022 at 09:22, Maxim Cournoyer <maxim.cournoyer <at> gmail.com> wrote:
>
>> I tried updating reposurgeon to see if I could get that resolved, but
>> reposurgeon is now a Go module, so we'd need to change the build system
>> and perhaps add a couple supporting Go modules.
>>
>> Would you like to take a look?
>
> reposurgeon is broken since a while; [1] is reported on Dec. 2018.
>
> Since no one care enough this package, I am propose to close this issue
> by simply removing the package.  WDYT?

I'm fine with it.  I tried updating it, but it is now written in Go and
would require packaging a few new Go packages; the list is here [0].

[0]  https://gitlab.com/esr/reposurgeon/-/blob/master/go.mod

-- 
Thanks,
Maxim

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

From: zimoun <zimon.toutoune <at> gmail.com>
To: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
Cc: 33614 <at> debbugs.gnu.org, Danny Milosavljevic <dannym <at> scratchpost.org>
Subject: Re: bug#33614: reposurgeon build is broken
Date: Sat, 21 Jan 2023 17:01:40 +0100

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

Hi Maxim,

On Mon, 17 Oct 2022 at 17:16, Maxim Cournoyer <maxim.cournoyer <at> gmail.com> wrote:

>> Since no one care enough this package, I am propose to close this issue
>> by simply removing the package.  WDYT?
>
> I'm fine with it.  I tried updating it, but it is now written in Go and
> would require packaging a few new Go packages; the list is here [0].


Please find attach the removal.  Once applied, feel free to close this
report.

Cheers,
simon

[0001-gnu-Remove-reposurgeon.patch (text/x-diff, inline)]

From d10833d01e9d19595ad186ff434f8dd4889d018d Mon Sep 17 00:00:00 2001
From: Simon Tournier <zimon.toutoune <at> gmail.com>
Date: Sat, 21 Jan 2023 16:56:46 +0100
Subject: [PATCH] gnu: Remove reposurgeon.

This package is broken since 2018 and an update requires Go modules.

* gnu/packages/version-control.scm (reposurgeon): Delete variable.
---
 gnu/packages/version-control.scm | 76 --------------------------------
 1 file changed, 76 deletions(-)

diff --git a/gnu/packages/version-control.scm b/gnu/packages/version-control.scm
index ff2fccef0c..78a89cfd5b 100644
--- a/gnu/packages/version-control.scm
+++ b/gnu/packages/version-control.scm
@@ -2399,82 +2399,6 @@ (define-public aegis
 any project with more than one developer, is one of Aegis's major functions.")
     (license license:gpl3+)))
 
-(define-public reposurgeon
-  (package
-    (name "reposurgeon")
-    (version "3.43")
-    (source (origin
-              (method url-fetch)
-              (uri (string-append "http://www.catb.org/~esr/" name "/"
-                                  name "-" version ".tar.xz"))
-              (sha256
-               (base32
-                "1af0z14wcm4bk5a9ysinbwq2fp3lf5f7i8mvwh7286hr3fnagcaz"))
-              (patches (search-patches
-                        "reposurgeon-add-missing-docbook-files.patch"))))
-    (build-system gnu-build-system)
-    (arguments
-     `(#:make-flags
-       (list "ECHO=echo"
-             (string-append "target=" (assoc-ref %outputs "out")))
-       #:phases
-       (modify-phases %standard-phases
-         (add-after 'unpack 'patch-inputs
-           (lambda* (#:key inputs #:allow-other-keys)
-             (let ((tzdata (assoc-ref inputs "tzdata")))
-               (substitute* "reposurgeon"
-                 (("/usr/share/zoneinfo")
-                  (string-append tzdata "/share/zoneinfo")))
-               (substitute* "test/svn-to-svn"
-                 (("/bin/echo") "echo"))
-               #t)))
-         (delete 'configure)            ; no configure script
-         (add-before 'build 'fix-docbook
-           (lambda* (#:key inputs #:allow-other-keys)
-             (substitute* (find-files "." "\\.xml$")
-               (("docbook/docbookx.dtd")
-                (string-append (assoc-ref inputs "docbook-xml")
-                               "/xml/dtd/docbook/docbookx.dtd")))
-             #t))
-         (add-before 'check 'set-up-test-environment
-           (lambda* (#:key inputs #:allow-other-keys)
-             (let ((tzdata (assoc-ref inputs "tzdata")))
-               (setenv "TZDIR" (string-append tzdata "/share/zoneinfo"))
-               #t)))
-         (add-after 'install 'install-emacs-data
-           (lambda* (#:key outputs #:allow-other-keys)
-             (install-file "reposurgeon-mode.el"
-                           (string-append (assoc-ref outputs "out")
-                                          "/share/emacs/site-lisp"))
-             #t)))))
-    (inputs
-     `(("python" ,python-wrapper)
-       ("tzdata" ,tzdata)))
-    (native-inputs
-     (list ;; For building documentation.
-           asciidoc
-           docbook-xml
-           docbook-xsl
-           libxml2
-           xmlto
-           ;; For tests.
-           cvs
-           git
-           mercurial
-           subversion))
-    (home-page "http://www.catb.org/~esr/reposurgeon/")
-    (synopsis "Edit version-control repository history")
-    (description "Reposurgeon enables risky operations that version-control
-systems don't want to let you do, such as editing past comments and metadata
-and removing commits.  It works with any version control system that can
-export and import Git fast-import streams, including Git, Mercurial, Fossil,
-Bazaar, CVS, RCS, and Src.  It can also read Subversion dump files directly
-and can thus be used to script production of very high-quality conversions
-from Subversion to any supported Distributed Version Control System (DVCS).")
-    ;; Most files are distributed under bsd-2, except 'repocutter' which is
-    ;; under bsd-3.
-    (license (list license:bsd-2 license:bsd-3))))
-
 (define-public tig
   (package
     (name "tig")
-- 
2.38.1

Message #24 received at 33614-done <at> debbugs.gnu.org (full text, mbox):

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: zimoun <zimon.toutoune <at> gmail.com>
Cc: 33614-done <at> debbugs.gnu.org, Danny Milosavljevic <dannym <at> scratchpost.org>
Subject: Re: bug#33614: reposurgeon build is broken
Date: Sat, 21 Jan 2023 15:08:25 -0500

Hello,

zimoun <zimon.toutoune <at> gmail.com> writes:

> Hi Maxim,
>
> On Mon, 17 Oct 2022 at 17:16, Maxim Cournoyer <maxim.cournoyer <at> gmail.com> wrote:
>
>>> Since no one care enough this package, I am propose to close this issue
>>> by simply removing the package.  WDYT?
>>
>> I'm fine with it.  I tried updating it, but it is now written in Go and
>> would require packaging a few new Go packages; the list is here [0].
>
>
> Please find attach the removal.  Once applied, feel free to close this
> report.

Applied!

-- 
Thanks,
Maxim

Previous Next

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