Re: Superseded packages

[Leo Famulari]

On Sat, Sep 24, 2016 at 11:14:41AM +0900, Ludovic Courtès wrote:
> Leo Famulari <address@hidden> skribis:
> > +(define-public attic
> > +  (package (inherit borg)
> > +    (name "attic")
> > +    (properties `((superseded . ,borg)))))
> 
> I was thinking that we could keep the ‘attic’ recipe as-is, only with
> this extra ‘properties’ field.
> 
> That way, “guix package -u” and “guix package -i attic” would install
> Borg, but people who insist could still run:
> 
>   guix package -e '(@ (gnu packages backup) attic)'
> 
> Thoughts?

That's a much better idea than what I had proposed.

I found that I had to move the attic package definition below borg's, or
else the borg variable was unbound when building (gnu packages backup):

------
Compiling Scheme modules...
  LOAD     (gnu packages backup)
;;; note: source file ./gnu/packages/backup.scm
;;;       newer than compiled /home/leo/work/guix/gnu/packages/backup.go
Backtrace:
In ice-9/boot-9.scm:
2401: 19 [save-module-excursion #<procedure 135d800 at ice-9/boot-9.scm:4045:3 
()>]
4052: 18 [#<procedure 135d800 at ice-9/boot-9.scm:4045:3 ()>]
1724: 17 [%start-stack load-stack ...]
1729: 16 [#<procedure 13729c0 ()>]
In unknown file:
   ?: 15 [primitive-load "/home/leo/work/guix/./build-aux/compile-all.scm"]
In ice-9/eval.scm:
 432: 14 [eval # #]
In ice-9/boot-9.scm:
 768: 13 [for-each #<procedure load-module-file (a)> 
("gnu/packages/backup.scm")]
2864: 12 [resolve-interface (gnu packages backup) #:select ...]
2789: 11 [#<procedure 12ab540 at ice-9/boot-9.scm:2777:4 (name #:optional 
autoload version #:key ensure)> # ...]
3065: 10 [try-module-autoload (gnu packages backup) #f]
2401: 9 [save-module-excursion #<procedure 171c450 at ice-9/boot-9.scm:3066:17 
()>]
3085: 8 [#<procedure 171c450 at ice-9/boot-9.scm:3066:17 ()>]
In unknown file:
   ?: 7 [primitive-load-path "gnu/packages/backup" ...]
In ice-9/eval.scm:
 432: 6 [eval # ()]
 453: 5 [eval # ()]
 411: 4 [eval # #]
 387: 3 [eval # #]
 387: 2 [eval # #]
 393: 1 [eval # #]
In unknown file:
   ?: 0 [memoize-variable-access! #<memoized borg> #<directory # 1736d80>]

ERROR: In procedure memoize-variable-access!:
ERROR: Unbound variable: borg
------

So the attached patch moves attic after borg and supersedes it. Is it
expected that we have to shuffle the package definition around like
this?

0001-gnu-attic-Superseded-by-borg.patch
Description: Text document