Regression with symbol conflict with SBCL/sb-grovel and 2.006

[Zach Beane]

ASDF cannot be upgraded to 2.006 if sb-grovel has been loaded before. Here's how to reproduce:

    $ sbcl --noinform --no-userinit --no-sysinit
    * (require 'sb-grovel)

    ("SB-GROVEL" "ASDF")
    * (load "asdf.lisp")

    Upgrading ASDF package to version 2.006

    debugger invoked on a SB-EXT:NAME-CONFLICT in thread #<THREAD
          "initial thread" RUNNING
          {1002ABB181}>:
      EXPORT OUTPUT-FILE causes name-conflicts in #<PACKAGE "SB-GROVEL"> between
      the following symbols:
 ASDF::OUTPUT-FILE, SB-GROVEL::OUTPUT-FILE
    See also:
      The ANSI Standard, Section 11.1.1.2.5

    Type HELP for debugger help, or (SB-EXT:QUIT) to exit from SBCL.

    restarts (invokable by number or by possibly-abbreviated name):
      0: [RESOLVE-CONFLICT] Resolve conflict.
      1: [ABORT ] Exit debugger, returning to top level.

    (SB-EXT:NAME-CONFLICT #<PACKAGE "SB-GROVEL"> EXPORT OUTPUT-FILE)[:EXTERNAL]
    0]

It's not all that strange to load sb-grovel, because it's loaded as a requirement of sb-posix.

asdf 2.004 did not have this problem.

[Faré (fahree)]

I posted a fix to sb-grovel some time ago. I'll attach a patch to SBCL...

[Faré (fahree)]

Here's a patch to SBCL.

[Faré (fahree)]

Scratch that, I'm preparing a better patch against HEAD that I'll test first...

[Faré (fahree)]

No, I was confusing with another issue someone had with a bad SBCL installation of SB-GROVEL.

Upgrading SBCL to 2.006 would be nice, but ASDF still has to be able to run in such cases and I'll have to patch it. Grrr.

[Faré (fahree)]

To reproduce the bug:
rlwrap sbcl --eval '(require :sb-posix)' --eval '(require :asdf)' --eval '(format t "~A~%" (asdf:asdf-version))' --eval '(asdf:load-system :asdf)'

[Faré (fahree)]

2.124 has a fix.
Is it important to make a 2.007 release now?
Zach, can you use the master branch for testing, rather than the release branch? This would find bugs earlier.

[Zach Beane (xach)]

I will try to use the master branch for testing and provide prompt bug reports for breakage.

[Zach Beane (xach)]

Regarding the release of 2.007, it would be extremely helpful to me if it were made very soon. 2.006 fixes some important bugs for me, but is unusable because of this issue.

[Faré (fahree)]

OK, released 2.007, because SBCL is worth it.