From: ian martins <firstname.lastname@example.org> To: Jarmo Hurri <email@example.com> Cc: firstname.lastname@example.org Subject: Re: Babel: parse error when output contains opening bracket Date: Sun, 28 Jun 2020 08:49:14 -0400 [thread overview] Message-ID: <CAC=rjb4T62T_wE8Xyxfn7QmP40Ps2GJM7GB=NxMPOkVYDc6Nbg@mail.gmail.com> (raw) In-Reply-To: <email@example.com> [-- Attachment #1: Type: text/plain, Size: 2287 bytes --] A little while back I submitted a patch to add org-babel integration for haxe and mentioned that the same could be done for java to expand the features of the java integration. There was no response which I took to mean no interest in haxe or java, so I didn't think submitting a patch for java would be fruitful. Then I thought I could just add them to the contrib directory but was mistaken since that goes through the same ML patch workflow. Since you recommend it, I will try submitting a patch for java. I still want to share the haxe integration. What is the best way to do that? -Ian On Sun, Jun 28, 2020 at 1:55 AM Jarmo Hurri <firstname.lastname@example.org> wrote: > > >> ian martins <email@example.com> writes: > >> > >> Would it be possible for us to fix the current version without > >> introducing a new one? Can you identify the parts of your code that fix > >> the issue? > >> > > The existing code creates the java program and runs it correctly, but > > it uses `org-babel-import-elisp-from-file' to interpret the results, > > and that sees the bracket and tries to make the response into a list, > > and errors when it can't. I don't see a quick fix for it. If you allow > > unbalanced brackets but that would be a change in ob-core and would > > probably cause unwanted results in other places. If you don't try to > > convert the output into a list, you can't present java results as > > lists or tables. Really the problem is that ob-java doesn't support > > functional mode, so it tries to guess if scripting mode output should > > be a table or list. The version I wrote supports functional and > > scripting modes and doesn't use `org-babel-import-elisp-from-file'. > > Ok. > > >> I am already a contributor, so if you can post your solution here I > >> can create a patch and give you the credit. > >> > > I would really appreciate that if you are willing, but it's a > > significant change (code is 400 lines, 600 lines of tests and test > > data) and there might be iterations so you might be signing up for > > more than you realize. > > Fair enough. > > 1. Have you considered writing a patch yourself? > > 2. If not, I think you lose nothing by posting your code here and > patiently waiting if I can create something out of it. > > All the best, > > Jarmo > > > [-- Attachment #2: Type: text/html, Size: 2989 bytes --]
next prev parent reply other threads:[~2020-06-28 12:50 UTC|newest] Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-06-25 11:28 Jarmo Hurri 2020-06-27 0:31 ` ian martins 2020-06-27 5:25 ` Jarmo Hurri 2020-06-27 10:29 ` ian martins 2020-06-28 5:55 ` Jarmo Hurri 2020-06-28 12:49 ` ian martins [this message] 2020-06-30 4:13 ` Jarmo Hurri 2020-06-30 10:27 ` ian martins 2020-09-06 9:56 ` Bastien 2020-09-08 21:53 ` ian martins 2020-09-09 8:02 ` Bastien 2020-09-06 9:51 ` Bastien
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style List information: https://www.orgmode.org/ * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to='CAC=rjb4T62T_wE8Xyxfn7QmP40Ps2GJM7GB=NxMPOkVYDc6Nbg@mail.gmail.com' \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --subject='Re: Babel: parse error when output contains opening bracket' \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
Code repositories for project(s) associated with this inbox: https://git.savannah.gnu.org/cgit/emacs/org-mode.git This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).