Re: [PATCH] IMPORTANT: (possibly) incompatible Change

[Carsten Dominik <carsten.dominik@gmail.com>]

Hi Eric,

I have fixed this issue in the definition of the variable value instead
of in the code.  Bastien did define `org-export-latex-packages'
as a list of lists instead of a list of cons cells.  So we should
stick with this structure - which is also better because it is
more easily extended, for example if we later need another parameter
to modify the sequence of some packages.

Thanks.

- Carsten

On Apr 2, 2010, at 3:29 AM, Eric Schulte wrote:

> After updating to the current git head, I have to make the following
> changes for latex image generation to work.
>
> diff --git a/lisp/org.el b/lisp/org.el
> index dc45871..443f881 100644
> --- a/lisp/org.el
> +++ b/lisp/org.el
> @@ -2936,7 +2936,7 @@ appears on the page."
> ;; when formatting latex fragments.  Originally it was part of the
> ;; LaTeX exporter, which is why the name includes "export".
> (defcustom org-export-latex-default-packages-alist
> -  '(("AUTO" . "inputenc")
> +  '(("" . "inputenc")
>     ("T1"   . "fontenc")
>     (""     . "graphicx")
>     (""     . "longtable")
> @@ -15247,9 +15247,9 @@ Some of the options can be changed using the  
> variable
> 		  (concat "\n"
> 			  (mapconcat (lambda(p)
> 				       (if (equal "" (car p))
> -					   (format "\\usepackage{%s}" (cadr p))
> +					   (format "\\usepackage{%s}" (cdr p))
> 					 (format "\\usepackage[%s]{%s}"
> -						 (car p) (cadr p))))
> +						 (car p) (cdr p))))
> 				     (append
> 				      org-export-latex-default-packages-alist
> 				      org-export-latex-packages-alist)
>
> The "AUTO" change is because the AUTO.def file is not present on my
> fairly complete Ubuntu texlive latex install
>
> : ERROR: LaTeX Error: File `AUTO.def' not found.
>
> The other change is because `org-export-latex-default-packages- 
> alist' is
> now a simple cons cell rather than a list so cadr was throwing errors.
>
> It seems to me these may be general problems, not just specific to my
> setup.
>
> Best -- Eric
>
> Carsten Dominik <carsten.dominik@gmail.com> writes:
>
>> Dear all,
>>
>> I have just checked in an important change - if you use LaTeX
>> export, you need to be aware of it.
>>
>> 1. Org contains now a much better system for handling special  
>> entities
>>   that are written like LaTeX macros, for example \therefore,
>> \emptyset,
>>   etc.  I will write more about this in the release notes for 6.35.
>> But
>>   already now thanks go to Ulf Stegemann without whom this would not
>>   have happened.
>>
>> 2. I could no longer keep the old setup for LaTeX export in
>>   org-export-latex-classes.  The disadvantage was that whenever you
>>   needed to make changes to the header, you would fix the value of
>> this
>>   variable so that any changes I'd make in the future would not be
>> visible
>>   to you.
>>
>>   The way this is solved now is (excerpt from the upcoming release
>> notes)
>>
>> -----------------------------------------------------------------------------
>> * =org-export-latex-classes= no longer should be customized for  
>> packages
>>
>> The HEADER part of this variable should now only contain the
>> documentclass macro, nothing else - at least normally.  All the
>> package calls via usepackage should go into
>> org-export-latex-packages-alist.  I moved all the default packages
>> that into a new variable org-export-latex-default-packages-alist.
>> This will allow me to add more packages (as needed) in the
>> future, withour requiring you to erase and then redo your
>> configuration of org-export-latex-classes.
>>
>> So if you have customized this variable, please remove once more
>> (hopefully for the last time) your customization, so that it can
>> revert to its now much simpler default value.  Put all your
>> package definitions into org-export-latex-packages-alist.
>> I hope this works, and we will not get conflicts because of the
>> sequence in which packages are called.  If there are problems,
>> please let me know so that we can find a solution.
>> -----------------------------------------------------------------------------
>>
>> I have not yet put this onto the master branch, but I will soon.
>> If you want to help testing this new setup, please check out the  
>> branch
>> new-entity-support from the git repo and let me know if you run  
>> into any
>> problems.
>>
>> Thanks!
>>
>> - Carsten (and Ulf)
>>
>>
>>
>> _______________________________________________
>> Emacs-orgmode mailing list
>> Please use `Reply All' to send replies to the list.
>> Emacs-orgmode@gnu.org
>> http://lists.gnu.org/mailman/listinfo/emacs-orgmode

- Carsten