Michael Albinus <michael.albi...@gmx.de> wrote: > Bastien <b...@altern.org> writes: > > > Hi Michael, > > Hi everybody, > > > From the org-mode repo, simply run "make test": it will compile the > > files and run the test suite. > > I could reproduce the problem. > > Finally, it is an error in `process-file', which wasn't detected until > now. Thanks to triggering this! > > I've fixed it already in Emacs' trunk. For org-mode, I have used the > following workaround: > > ~/src/org-mode> git diff > diff --git a/lisp/ob-eval.el b/lisp/ob-eval.el > index 23e7143..df95d0a 100644 > --- a/lisp/ob-eval.el > +++ b/lisp/ob-eval.el > @@ -143,6 +143,11 @@ specifies the value of ERROR-BUFFER." > shell-file-name > "/bin/sh")) > exit-status) > + ;; There is an error in `process-file', when `error-file' exists. > + ;; Fixed already in Emacs trunk; for the time being we apply a > + ;; workaround. > + (unless (file-remote-p default-directory) > + (delete-file error-file)) > (if (or replace > (and output-buffer > (not (or (bufferp output-buffer) (stringp output-buffer))))) >
I can confirm that this fixes the local dir case in the simple test I posted previously (and the remote dir case was fixed by the process-file fix previously). > With that workaround, "make test" results in > > Ran 320 tests, 319 results as expected, 1 unexpected (2012-12-21 > 09:13:44+0100) > 7 expected failures > > 1 unexpected results: > FAILED ob-exp/exports-inline > I don't get that failure with the current code, so maybe it has been fixed already (?): ,---- | Ran 334 tests, 334 results as expected (2012-12-21 12:27:17-0500) | 5 expected failures `---- Not sure why I have a different number of tests/expected failures either. But in any case, the main problem is resolved. Thanks for the fixes! Nick
