Re: [PATCH] possible bugfix in ob-java

2020-09-23 Thread Bastien
Hi John,

Applied as a409e1033 in master.

Thanks a lot!

-- 
 Bastien



[PATCH] possible bugfix in ob-java

2020-09-13 Thread John Herrlin

Hey,

Trying to do the following:

 #+HEADER: :classname se/my_test_package/Main
 #+HEADER: :dir src
 #+HEADER: :cmdline -classpath src:.
 #+BEGIN_SRC java :results output code
   package se.my_test_package;

   class Main {
   public static void main(String[] args) {
   System.out.println("Hey from Emacs Org mode!");
   }
   }
 #+END_SRC

I expected this to create Main.java and Main.class in directory
src/se/my_test_package/ before running =javac=. Instead I get this:

File is missing: "Opening output file", "No such file or directory"
"/tmp/src/se/my-test-package/Main.java"

In the patch I rearrange so package dirs are created before the
Main.java file is written to file system. It fixes the provided case for
me.

>From 618b736d895b1e762cdcc8026a673aa6207a0a8b Mon Sep 17 00:00:00 2001
From: John Herrlin 
Date: Sun, 13 Sep 2020 18:07:22 +0200
Subject: [PATCH] ob-java: create package dirs before write to file

* lisp/ob-java.el (org-babel-execute:java): create package-name
directories before writing to file.

TINYCHANGE
---
 lisp/ob-java.el | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/lisp/ob-java.el b/lisp/ob-java.el
index f5edc6d53..dc5efe5fb 100644
--- a/lisp/ob-java.el
+++ b/lisp/ob-java.el
@@ -60,12 +60,12 @@ parameters may be used, like javac -verbose"
 	 (cmdline (or (cdr (assq :cmdline params)) ""))
 	 (cmdargs (or (cdr (assq :cmdargs params)) ""))
 	 (full-body (org-babel-expand-body:generic body params)))
-(with-temp-file src-file (insert full-body))
-(org-babel-eval
- (concat org-babel-java-compiler " " cmpflag " " src-file) "")
 ;; created package-name directories if missing
 (unless (or (not packagename) (file-exists-p packagename))
   (make-directory packagename 'parents))
+(with-temp-file src-file (insert full-body))
+(org-babel-eval
+ (concat org-babel-java-compiler " " cmpflag " " src-file) "")
 (let ((results (org-babel-eval (concat org-babel-java-command
" " cmdline " " classname " " cmdargs) "")))
   (org-babel-reassemble-table
-- 
2.28.0