Re: Subject: [PATCH] Fix DISPLAY error on exporting org with plantuml to html

[Max Nikulin <manikulin@gmail.com>]

On 07/06/2022 13:34, Dr. Arne Babenhauserheide wrote:
> 
> lin Sun writes:
>> This patch will re-submit the solution based on the last rev.

>> diff --git a/lisp/ob-plantuml.el b/lisp/ob-plantuml.el
>> index 6e1b1b1..37a631b 100644
>> --- a/lisp/ob-plantuml.el
>> +++ b/lisp/ob-plantuml.el
>> @@ -122,6 +122,7 @@ This function is called by `org-babel-execute-src-block'."
>>   				((not (file-exists-p org-plantuml-jar-path))
>>   				 (error "Could not find plantuml.jar at %s" org-plantuml-jar-path))
>>   				(t `(,java
>> +                                     "-Djava.awt.headless=true"
> 
> This looks good and low-risk to me.

Could you, please, clarify why -headless in `org-plantuml-args' is not 
enough and -Djava.awt.headless=true should be added despite it looks 
redundant. Another question is why this option should be applied to jar 
but not to plantuml launcher script. The commit dropping this option has 
a link to discussion whether this argument is necessary and hard coded 
in such way.

> 
>>   				     "-jar"
>>   				     ,(shell-quote-argument (expand-file-name org-plantuml-jar-path))
>>                                        ,@org-plantuml-args))))