[PATCH] org-preview-latex-fragment not working with #+latex_header:\input{}

[PATCH] org-preview-latex-fragment not working with #+latex_header:\input{}

[Benjamin Motz]

[-- Attachment #1: Type: text/plain, Size: 749 bytes --]

Bug:

#+latex_header:\input{commands.tex} is not considered in
org-preview-latex-fragment. Reason: the fragment is generated in /tmp
from where the file commands.tex can not be found. In my case,
commands.tex contains essential latex-code for displaying the
latex-fragment, so I consider this to be a bug.

I have set the environment variable TEXINPUTS in
(org-create-formula-image) to work around this bug (see patch). 

However, this is the first time that I have written something like a
patch and furthermore I'm not at all familiar with lisp-programming. So
could you please have a look at my solution and tell me if it's complete
crap and how it could be done better?!

If it's no crap, I presume it should be submitted.

Thank you, Benjamin


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: texinputs.patch --]
[-- Type: text/x-diff, Size: 329 bytes --]

diff --git a/org.el b/org.el.orig
index 4f5bef0..b1c15fb 100644
--- a/org.el
+++ b/org.el.orig
@@ -17040,7 +17040,6 @@ inspection."
       (condition-case nil
 	  (progn
 	    (cd tmpdir)
-	    (setenv "TEXINPUTS" concat(default-directory ":.:"))
 	    (call-process "latex" nil nil nil texfile))
 	(error nil))
       (cd dir))

Re: [PATCH] org-preview-latex-fragment not working with #+latex_header:\input{}

[Bastien]

Hi Benjamin,

Benjamin Motz <b.motz@uni-muenster.de> writes:

> #+latex_header:\input{commands.tex} is not considered in
> org-preview-latex-fragment. Reason: the fragment is generated in /tmp
> from where the file commands.tex can not be found. In my case,
> commands.tex contains essential latex-code for displaying the
> latex-fragment, so I consider this to be a bug.

Did you try \input{/home/absolute/path/commands.tex} ?

> I have set the environment variable TEXINPUTS in
> (org-create-formula-image) to work around this bug (see patch). 
>
> However, this is the first time that I have written something like a
> patch and furthermore I'm not at all familiar with lisp-programming. So
> could you please have a look at my solution and tell me if it's complete
> crap and how it could be done better?!

It complete crap and it could be done better :)

> If it's no crap, I presume it should be submitted.

Setting TEXINPUTS while creating formulas makes sense to me, so I'll be
glad to have a patch for this.

Some comments:

> Thank you, Benjamin
>
> diff --git a/org.el b/org.el.orig

The diff says you are comparing org.el and org.el.orig (the original
file).  With this method, it's easy to have the comparison done the
wrong way. 

The *really really* simple method is to use git and Emacs >=23.4.

~$ git clone git://orgmode.org/org-mode.git
 
Then open the .el file you want to edit.  When you're done editing,
simply `C-x v ='.  It will open a new buffer with the patch.

HTH,

-- 
 Bastien

Re: [PATCH] org-preview-latex-fragment not working with #+latex_header:\input{}

[Vladimir Lomov]

Hello,
** Benjamin Motz [2012-04-20 19:28:43 +0200]:

> Bug:

> #+latex_header:\input{commands.tex} is not considered in
> org-preview-latex-fragment. Reason: the fragment is generated in /tmp
> from where the file commands.tex can not be found. In my case,
> commands.tex contains essential latex-code for displaying the
> latex-fragment, so I consider this to be a bug.

IMHO, you doing it in wrong way, I presume you are using some Linux:
create ~/texmf/tex/latex/ben
and put there your `commands.tex', then `\input{commands.tex}' will pick
up that file.

Of course it is true for LaTeX, if you use plain TeX use
`~/texmf/tex/plain/ben' directorythen.

> I have set the environment variable TEXINPUTS in
> (org-create-formula-image) to work around this bug (see patch). 

Don't do it. If it is Ok to use in isolated environment and only once
the TEXINPUTS variable it is bad practice.

> However, this is the first time that I have written something like a
> patch and furthermore I'm not at all familiar with lisp-programming. So
> could you please have a look at my solution and tell me if it's complete
> crap and how it could be done better?!

> If it's no crap, I presume it should be submitted.

> Thank you, Benjamin


> diff --git a/org.el b/org.el.orig
> index 4f5bef0..b1c15fb 100644
> --- a/org.el
> +++ b/org.el.orig
> @@ -17040,7 +17040,6 @@ inspection."
>        (condition-case nil
>  	  (progn
>  	    (cd tmpdir)
> -	    (setenv "TEXINPUTS" concat(default-directory ":.:"))
>  	    (call-process "latex" nil nil nil texfile))
>  	(error nil))
>        (cd dir))


P.S. I'm using org with latex math code and all works fine (I'm using very
customized setup but it works very well).

---
WBR, Vladimir Lomov

-- 
I knew her before she was a virgin.
		-- Oscar Levant, on Doris Day

Re: [PATCH] org-preview-latex-fragment not working with #+latex_header:\input{}

[Achim Gratz]

Bastien writes:
> Setting TEXINPUTS while creating formulas makes sense to me, so I'll be
> glad to have a patch for this.

Actually, setting TEXINPUTS is fraught with peril and just setting it
without checking if it already exists is risking to break some setups.
If anything there should be a defcustom or better hook function to allow
alteration of the environment (ostensibly not just for TEXINPUTS) before
org calls external processes.


Regards,
Achim.
-- 
+<[Q+ Matrix-12 WAVE#46+305 Neuron microQkb Andromeda XTk Blofeld]>+

Wavetables for the Waldorf Blofeld:
http://Synth.Stromeko.net/Downloads.html#BlofeldUserWavetables