On Sun, Feb 24, 2013 at 7:14 PM, Robert Goldman rpgoldman@sift.info wrote:
TEST ABORTED: Subprocess run with command "/bin/ln -sf /Users/rpg/lisp/asdf/test/test-multiple.asd /Users/rpg/lisp/asdf/build/test-multiple-too.asd 2>&1" exited with error code 1
No backtrace for this.
I believe the problem is that EXT:SYSTEM on ECL runs whatever shell is the value of SHELL, but your test code is all written for bash, without checking for it.
I use the T shell.
Probably a good thing that I am testing with this unusual configuration!
I suppose the test code should force SHELL to be bash. I don't know if there's a portable way to do this. ASDF used to invoke /bin/sh, in RUN-SHELL-COMMAND, IIRC.
It is possible that there is a bug in asdf/driver:run-program, or maybe one in ECL itself. Can you run again while tracing as follows? (trace asdf/driver:run-program ext:run-program ext:shell)
I think the test-multiple might pass if you the /bin/ in /bin/ln.
Is the "T shell" the same thing as tcsh? Let me try it... Indeed, I can reproduce the failure with SHELL=tcsh make t l=ecl t='test-run-program.script even though this works: SHELL=tcsh make t l=ccl t='test-run-program.script That's because somehow ECL, to get the exit value of a process, reverts to a more primitive call to system, and then relies on "exec > tmpfile ; command" for redirection, independently from the content of command. It looks like this is not portable to tcsh; and probably not to Windows, either. I don't know how to solve this problem portably or to automatically detect a workaround.
Maybe the test script runner should just export SHELL=/bin/sh ?
- ABCL 1.1.0.2 failed test-encodings: [...]
Weird. Looks like the UTF-8-encoded characters were read as MacRoman. Within which sub-test does that occur? A bit more context above would help.
This is the one that failed:
(with-encoding-test (explicit-u8 :utf-8) (def-test-system :test-encoding-explicit-u8 :components ((:file "lambda" :encoding :utf-8))))
Note that to make this work, I added a dynamic variable that bound asdf-test::*test-name* and made the assert-compare-helper incorporate that name in its output.
The dynamic variable was a little yucky, but otherwise I would have had to push the name (as an optional) through the arglist of all of the comparison macros....
This might be useful for debugging, but the implementation is super nasty.
I'm not sure what you changed to "make it work". If explicit request for utf-8 encoding doesn't work, something is probably broken in either asdf and/or abcl.
- Allegro Modern 8.2 failed test-utilities with this error:
TEST ABORTED: Package "ASDF/ACTION" not found. [file position = 3334]
The problem is the defparameter for *ok-symbols*, which gives the symbol names in all caps. I am attaching a patch for this.
Oops. You know, you have commit rights, and should readily push quite uncontroversial patches like this.
I haven't been attending as much as I should, so I wasn't sure how this interacted with the versioning (didn't seem worth a patch level bump, but I wasn't certain).
My principle is I only bump the version when there's a change to the code itself (i.e. asdf.lisp in this case). all the rest I consider ancillary.
—♯ƒ • François-René ÐVB Rideau •Reflection&Cybernethics• http://fare.tunes.org "There could hardly be a more unbearable — and more irrational — world than one in which the most eminent specialists in each field were allowed to proceed unchecked with the realization of their ideals" — Friedrich A. Hayek