"Robert" == Robert Goldman <rpgoldman@sift.net> writes:
Robert> On 12/20/15 Dec 20 -12:14 PM, Raymond Toy wrote: >>>>>>> "Robert" == Robert Goldman >>>>>>> <rpgoldman@sift.net> writes: >> I thought I had sent a response to this, but I guess not. Robert> I think what we want is this, right? Robert> (defun test () (pathname-match-p Robert> (uiop/common-lisp:user-homedir-pathname) (first Robert> (ext:search-list (cl:user-homedir-pathname))))) Robert> that fails on master, and passes with your patch. Yeah, that's the correct version. Robert> It's not particularly clear to anyone who's not familiar Robert> with CMUCL. Is there a way to illustrate the issue Robert> without reference to EXT:SEARCH-LIST? Or can you provide The whole issue is the search-list and how pathname-match-p doesn't handle it. Here's a separate example of the issue. (setf (search-list "tmp:") '("/tmp/")) (pathname-match-p #p"/tmp/foo.lisp" "tmp:foo.lisp") gives an error. But if you had a logical-pathname host "temp", then (pathname-match-p #p"/tmp/foo.lisp" "temp:*.lisp") doesn't signal an error. It actually returns nil, which doesn't seem quite right if there's a translation ("*.lisp" "/tmp/*.lisp"). Ugh. I hate pathname issues. Robert> comment for this? If I just drop this form into the test Robert> suite, I foresee it creating great confusion for some poor Robert> shlub (possibly me) who reads in > 6 months from now.... A comment is that asdf doesn't want to use search-lists for user-homedir-pathname, Convert it to a physical pathaname. Robert> It seems like we're doing *SOMETHING* with Robert> USER-HOMEDIR-PATHNAME followed by merging that causes Robert> PATHNAME-MATCH-P in this function to error out. Robert> But to be honest, I have no idea what's going wrong, or Robert> why it's ASDF's fault. If you merge a pathname with Robert> USER-HOMEDIR-PATHNAME's return value, it seems to me you Robert> ought to be able to call PATHNAME-MATCH-P on the result. Yes, and indeed (merge-pathnames "foo.lisp" (user-homedir-pathname)) returns "home:foo.lisp" as expected. But pathname-match-p doesn't like search-lists. I guess that's cmucl's fault. Robert> It seems like you're saying it's something about the way Robert> that search-lists deal with nonexistent directories. Is Robert> that right? Robert> Or is this because when we merge we aren't appropriately Robert> creating something that's a wildcard pathname? I don't know either. I will look into making pathname-match-p understand search-lists. I'll leave it up to you on whether asdf should use a different user-homedir-pathname. I think it makes some sense for any using cmucl with a pathname-match-p fix with newer versions of asdf. -- Ray