Re: [PATCH] ob-vala.el: Add Vala support to Babel

[Christian Garbs <mitch@cgarbs.de>]

Hi!

On Thu, Aug 03, 2017 at 12:49:44PM +0200, Nicolas Goaziou wrote:
> Christian Garbs <mitch@cgarbs.de> writes:

> > Regarding the lexical-binding:  How can I check if byte-compiling is
> > successful?
> 
> M-x emacs-lisp-byte-compile

Byte-compiling works and gave me 5 warnings about unused variables and
arguments which I could fix in different ways.

Currently I run into a problem with the test case:  I have created an
org file containing some Vala code and an .el file with some tests,
but the dependency check at the top of the test always fails:

| (unless (featurep 'ob-vala)
|   (signal 'missing-test-dependency "Support for Vala code blocks"))

Removing the test results in errors.

When I replace those two lines with

| (require 'ob-vala)

the tests run as expected, eg. they fail when I deliberately provoke
an error and succeed otherwise.

I think this is a dependency or build problem, but I don't know what
exactly goes wrong or how to fix it.  I have searched the whole source
for 'ob-C and 'ob-fortran to see how those tests and modules differ
from 'ob-vala, but this yielded no result.

My guess: It is something totally basic and trivial.
Any ideas?

Regards
Christian
-- 
....Christian.Garbs....................................https://www.cgarbs.de

To iterate is human, to recurse divine.  (L. Peter Deutsch)