testing/README: need to enable sh and org for the current test

testing/README: need to enable sh and org for the current test

[Yasushi SHOJI]

Hello,

With the current HEAD(release_7.9.3d-894-gfe805ed), it seems we need
to load "sh" and "org" for babel language to run the test sucessfully.

#+BEGIN_SRC elisp
  (org-babel-do-load-languages
   'org-babel-load-languages
   '((emacs-lisp . t)
     (sh . t)
     (org . t)))
  (setq org-confirm-babel-evaluate nil)
#+END_SRC

Am I missing something? Does it work as documented in testing/README
for everyone?

if not, should we patch the README like this? (just an idea, not completed)

diff --git a/testing/README b/testing/README
index 8ce7a19..9601ea7 100644
--- a/testing/README
+++ b/testing/README
@@ -15,12 +15,16 @@ The simplest way to run the Org-mode test suite is from the command
 line with the following invocation.  Note that the paths below are
 relative to the base of the Org-mode directory.
 
+Also note that many of the current tests uses babel evaluation...
+
 #+BEGIN_SRC sh :dir (expand-file-name "..")
   # For Emacs earlier than 24, add -L /path/to/ert
   emacs -Q --batch \
         -L lisp/ -L testing/ -L testing/lisp -l lisp/org.el \
         -l lisp/org-id.el -l testing/org-test.el \
-        --eval "(progn (org-reload) (setq org-confirm-babel-evaluate nil))" \
+        --eval "(progn (org-reload) (setq org-confirm-babel-evaluate nil) \
+        (org-babel-do-load-languages 'org-babel-load-languages \
+        '((emacs-lisp . t) (sh . t) (org . t))))" \
         -f org-test-run-batch-tests
 #+END_SRC
 
@@ -43,7 +47,12 @@ load and run the test suite with the following commands.
      (require 'org-test)
    #+END_SRC
 
-2) Then run the test suite,
+2) Disable babel evaluation confirmation
+   #+BEGIN_SRC emacs-lisp
+     (setq org-confirm-babel-evaluate)
+   #+END_SRC
+
+3) Then run the test suite,
    #+BEGIN_SRC emacs-lisp
      (org-test-run-all-tests)
    #+END_SRC
-- 
           yashi

Re: testing/README: need to enable sh and org for the current test

[Bastien]

Hi Yasushi,

Yasushi SHOJI <yashi@atmark-techno.com> writes:

> With the current HEAD(release_7.9.3d-894-gfe805ed), it seems we need
> to load "sh" and "org" for babel language to run the test sucessfully.

Indeed.

> #+BEGIN_SRC elisp
>   (org-babel-do-load-languages
>    'org-babel-load-languages
>    '((emacs-lisp . t)
>      (sh . t)
>      (org . t)))
>   (setq org-confirm-babel-evaluate nil)
> #+END_SRC
>
> Am I missing something? 

Nope -- I applied your patch in master, thanks!

-- 
 Bastien