Karthik Chikmagalur <[email protected]> writes: >> Thanks! >> Applied, onto main, after small changes to the commit message and the >> NEWS entry. >> https://git.savannah.gnu.org/cgit/emacs/org-mode.git/commit/?id=f30f6dc4bc > > I'm working on merging org-latex-preview again, and noticed that there > is a new "xelatex" entry in org-preview-latex-process-alist introduced > in this patch. In the patchset we are providing this is unnecessary > because setting org-latex-compiler to xelatex is all that is required to > generate previews correctly. > > I think this is the correct way to handle it -- notice that the > possible preview methods right now are dvipng, dvisvgm, imagemagick and > xelatex -- one of these is not like the others!
It is conceptually dvisvgm, but svisvgm has latex hard-coded, so it is not different (other than the name). In any case, my thinking is simple: 1. The patch was an improvement 2. new latex preview system is in the works for very long time 3. new latex preview will be breaking anyway 4. So, merge it and change things if/when new latex preview is merged. For you, it should not change much. You already had to change dvisvgm entry in that list in the patch to avoid forcing latex. The only additional change that might be needed is providing an alias that entry. But you don't need to worry about that now. Let's first finish the review that is ongoing. I will do a pass on possible backwards-compatibility improvements later in the review. -- Ihor Radchenko // yantar92, Org mode maintainer, Learn more about Org mode at <https://orgmode.org/>. Support Org development at <https://liberapay.com/org-mode>, or support my work at <https://liberapay.com/yantar92>
