[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Possible fix for :includes header argument in org-babel C source blo
From: |
Kévin Le Gouguec |
Subject: |
Re: Possible fix for :includes header argument in org-babel C source blocks |
Date: |
Thu, 28 May 2020 12:09:35 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.0.50 (gnu/linux) |
Kévin Le Gouguec <kevin.legouguec@gmail.com> writes:
> That leads me to believe that the coercion was an unintended side-effect
> of (format …).
Never mind, the ORG-NEWS entry for 9.1 shows an example of unquoted
header, so I guess it is intentional.
Here is a patch to fix the regression:
0001-Coerce-symbols-found-in-includes-header-arguments-to.patch
Description: Text Data
And here is a patch to add a test for the unquoted-single-header case,
since otherwise it's hard to tell whether this behaviour is intentional:
0002-Add-test-case-for-symbol-coercion-in-C-Babel-blocks.patch
Description: Text Data
(Is the Org source for the C/C++/D Babel syntax[1] committed somewhere,
BTW? I could not find it in the Org repo.)
I'd like to say that all tests pass now, but I'm getting failures on
master (even without my changes) for two other tests:
> FAILED ob-tangle/jump-to-org
> FAILED test-org-attach/dir
[1] https://orgmode.org/worg/org-contrib/babel/languages/ob-doc-C.html