Opened 2 years ago

Closed 2 years ago

Last modified 2 years ago

#472 closed defect (fixed)

CL:DIRECTORY with wildcard in name

Reported by: Mark Evenson Owned by:
Priority: major Milestone: 1.8.0
Component: other Version: 1.7.2-dev
Keywords: pathname ansi Cc:
Parent Tickets:

Description

(directory "/tmp/**")
The value NIL is not of type FIXNUM.
   [Condition of type TYPE-ERROR]

Restarts:
 0: [RETRY] Retry SLIME REPL evaluation request.
 1: [*ABORT] Return to SLIME's top level.
 2: [ABORT] Abort thread.

Backtrace:
  0: (INVOKE-DEBUGGER #<TYPE-ERROR {7ED158A9}>)
  1: org.armedbear.lisp.Lisp.error(Lisp.java:382)
  2: (POSITION-IF #<LOCAL-FUNCTION IN COMPONENT-MATCH-WILD-P {7BF6BC72}> "**" :START NIL)
  3: (SYSTEM::COMPONENT-MATCH-WILD-P "com.apple.launchd" "**" NIL)
  4: (SYSTEM::COMPONENT-MATCH-P "com.apple.launchd" "**" NIL)
  5: (PATHNAME-MATCH-P "com.apple.launchd.cyc79CMFEO" "**")
  6: (DIRECTORY "/tmp/**")
  7: (SYSTEM::%EVAL (DIRECTORY "/tmp/**"))

Change History (6)

comment:1 Changed 2 years ago by Mark Evenson

Summary: CL:DIRECTORY with wild-inferiors failureCL:DIRECTORY with wildcard in name

Broken at least back to abcl-1.3.2

comment:2 Changed 2 years ago by Mark Evenson

And DIRECTORY on a WILD-INFERIORS is also borked:

(directory "/tmp/**/")

comment:3 Changed 2 years ago by Mark Evenson

Also, we don't return the truename of all entries like we should.

comment:4 in reply to:  3 Changed 2 years ago by Mark Evenson

Replying to Mark Evenson:

Also, we don't return the truename of all entries like we should.

Actually, we decided not to return TRUENAME for DIRECTORY in order to be able to get the contents of a directory with dangling symlinks.

The extension to ANSI via :RESOLVE-SYMLINKS was added as
<https://abcl.org/trac/ticket/340>, in which it was argued that
symlinks should be considered contents of a directory, and that in
any event, performing a DIRECTORY on a dangling symlink should not
signal an error.

See <https://abcl.org/trac/changeset/14624> for additional information on implementation decision.

comment:5 Changed 2 years ago by Mark Evenson

Resolution: fixed
Status: newclosed

Addressed in r15369.

comment:6 Changed 2 years ago by Mark Evenson

Milestone: 1.7.21.8.0

Milestone renamed

Note: See TracTickets for help on using tickets.