Re: [O] A weird warning and a test (eager macro expansion) error

2013-11-17 Thread Achim Gratz
Achim Gratz writes: I'll just keep the patch locally for the moment, at least until I've looked at Ert and/or made a test case that can be reoprted to emacs-bugs. If anybody wants to test on Emacs trunk, it is easy enough to apply the patch, I think. This turned out to be a bug in Org after

Re: [O] A weird warning and a test (eager macro expansion) error

2013-11-13 Thread Bastien
Hi Achim, Achim Gratz strom...@nexgo.de writes: This patch fixes the error, but it looks strange (indentation is still unchanged for clarity). Not sure if this an Ert or Emacs error, though. I suggest you handle this the way you prefer: either by committing this in Org before asking

Re: [O] A weird warning and a test (eager macro expansion) error

2013-11-13 Thread Achim Gratz
Bastien writes: I suggest you handle this the way you prefer: either by committing this in Org before asking emacs-devel@, or by asking first then see of this needs to be fixed upstream? I won't have time to digg this issue further, sorry. I'll just keep the patch locally for the moment, at

Re: [O] A weird warning and a test (eager macro expansion) error

2013-11-12 Thread Achim Gratz
[resent] Bastien writes: Thanks -- I remember we had this issue before. I'm just surprise I'm the first one to report it, I assume many people use a recent Emacs with macro expansion done this way. This patch fixes the error, but it looks strange (indentation is still unchanged for clarity).

Re: [O] A weird warning and a test (eager macro expansion) error

2013-11-05 Thread Achim Gratz
Bastien writes: | Loading /home/guerry/install/git/org-mode/testing/lisp/test-ob.el (source)... | Eager macro-expansion failure: (void-variable test-line) | Symbol's value as variable is void: test-line | mk/targets.mk:99: recipe for target 'test' failed | make: *** [test] Error 255 `

Re: [O] A weird warning and a test (eager macro expansion) error

2013-11-05 Thread Bastien
Hi Achim, Achim Gratz strom...@nexgo.de writes: This is caused by the following constructs in test-ob.el: (ert-deftest test-org-babel/inline-src_blk-default-results-replace-line-1 () (let ((test-line src_sh{echo 1})) ;; src_ at bol line 1... (org-test-with-temp-text

[O] A weird warning and a test (eager macro expansion) error

2013-11-04 Thread Bastien
Hi all, I've got this warning when compiling: , | Compiling /home/guerry/install/git/org-mode/lisp/ob-R.el... | | In end of data: | ob-R.el:401:1:Warning: the function `org-every' is not known to be defined. ` (There is no org-every in ob-R.el directly.) And ~$ make test fails like