Bastien <b...@gnu.org> writes: > Eric Schulte <eric.schu...@gmx.com> writes: > >> Bastien <b...@gnu.org> writes: >> >>> Eric Schulte <eric.schu...@gmx.com> writes: >>> >>>> Hmm, these tests should only be run if the "gfortran" executable is in >>>> your path. >>> >>> So let's take them out of the test suite when gfortran is not present? >> >> That is what we do currently, or at least that is what we're supposed to >> be doing currently... From the top of test-ob-fortran.el >> >> ;; -*- emacs-lisp -*- >> (org-test-for-executable "gfortran") >> (unless (featurep 'ob-fortran) >> (signal 'missing-test-dependency "Support for Fortran code blocks")) > > Maybe this should be > > (unless (and (org-test-for-executable "gfortran") (featurep 'ob-fortran)) > (signal 'missing-test-dependency "Support for Fortran code blocks")) > > ? > > to also ignore the test when gfortran does not exist? Wild guess.
Nope, `org-test-for-executable' signals it's own error, but if we changed `org-test-for-executable' to return a boolean and then used your example above it would be more readable. Best, -- Eric Schulte http://cs.unm.edu/~eschulte
