Re: Makefile minor issues
Hi all, >>>>> Ikumi Keita writes: > I noticed some minor issues about Makefiles of AUCTeX. > (A) Makefile in latex/ subdirectory > (B) Clean targets in Makefiles > I think that it is straightforward to sort out (B), but (A) is not so > trivial. What do you think about them? I made two patches for (A) and (B). I think it's OK to apply (B), so will install in the git repo. Does anyone have objection to (A), which effectively phases out dvi format and uses pdf instead of it? Regards, Ikumi Keita diff --git a/Makefile.in b/Makefile.in index 666b686f..c0ab5294 100644 --- a/Makefile.in +++ b/Makefile.in @@ -2,7 +2,7 @@ # Maintainer: auctex-devel@gnu.org -# Copyright (C) 2003-2008, 2010, 2013-2015, 2018 Free Software +# Copyright (C) 2003-2008, 2010, 2013-2015, 2018-2019 Free Software # Foundation, Inc. # This file is part of AUCTeX. @@ -360,19 +360,23 @@ uninstall: .el.elc: $(ELCC) -f batch-byte-compile $< -clean: doc/Makefile +clean: doc/Makefile latex/Makefile rm -f $(CLEANFILES) cd doc && $(MAKE) clean + cd latex && $(MAKE) clean -distclean: doc/Makefile +distclean: doc/Makefile latex/Makefile rm -f $(CLEANFILES) $(DISTCLEANFILES) cd doc && $(MAKE) distclean + cd latex && $(MAKE) distclean -maintainer-clean: doc/Makefile +maintainer-clean: doc/Makefile latex/Makefile rm -f $(DISTTEXTS) rm -rf autom4te.cache rm -f $(CLEANFILES) $(DISTCLEANFILES) cd doc && $(MAKE) maintainer-clean + cd latex && $(MAKE) distclean + cd tests && $(MAKE) clean extraclean: maintainer-clean rm -f *~ \#*\# diff --git a/doc/Makefile.in b/doc/Makefile.in index 6f40d825..21cc2e91 100644 --- a/doc/Makefile.in +++ b/doc/Makefile.in @@ -2,7 +2,7 @@ # Maintainer: auctex-devel@gnu.org -# Copyright (C) 2003-2008, 2013-2015, 2018 Free Software Foundation, Inc. +# Copyright (C) 2003-2008, 2013-2015, 2018-2019 Free Software Foundation, Inc. # This file is part of AUCTeX. @@ -208,5 +208,4 @@ distclean: clean rm -f Makefile maintainer-clean: distclean - rm -f INSTALL.windows README CHANGES HISTORY TODO FAQ \ - preview-dtxdoc.texi version.texi $(DISTTEXTS) *.info *.info-* + rm -f preview-dtxdoc.texi version.texi $(DISTTEXTS) *.info *.info-* diff --git a/latex/Makefile.in b/latex/Makefile.in index afb0a62a..f342b6bc 100644 --- a/latex/Makefile.in +++ b/latex/Makefile.in @@ -2,7 +2,7 @@ # Maintainer: auctex-devel@gnu.org -# Copyright (C) 2002-2006, 2008, 2014, 2015 Free Software Foundation, +# Copyright (C) 2002-2006, 2008, 2014, 2015, 2019 Free Software Foundation, # Inc. # This file is part of AUCTeX. @@ -42,7 +42,7 @@ INSTALL = @INSTALL@ INSTALL_DATA = @INSTALL_DATA@ MKINSTALLDIRS = ../mkinstalldirs -.PHONY: all install install-texmf install-texmf-doc uninstall clean +.PHONY: all install install-texmf install-texmf-doc uninstall clean distclean all: $(TEXMFGEN) preview.drv preview-mk.ins preview.dvi preview.pdf @@ -93,5 +93,8 @@ preview.pdf: preview.drv preview.dtx preview.sty $(PDFLATEX) '\nonstopmode \AtBeginDocument{\OnlyDescription}\input{preview.drv}' clean: - rm -f *~ *.aux *.dvi *.drv *.log + rm -f *~ *.aux *.dvi *.drv *.log preview.pdf rm -f $(TEXMFGEN) preview.ins preview-mk.ins + +distclean: clean + rm -f Makefile diff --git a/latex/Makefile.in b/latex/Makefile.in index f342b6bc..ab636475 100644 --- a/latex/Makefile.in +++ b/latex/Makefile.in @@ -44,7 +44,7 @@ MKINSTALLDIRS = ../mkinstalldirs .PHONY: all install install-texmf install-texmf-doc uninstall clean distclean -all: $(TEXMFGEN) preview.drv preview-mk.ins preview.dvi preview.pdf +all: $(TEXMFGEN) preview.drv preview-mk.ins preview.pdf install: install-texmf install-texmf-doc @@ -63,12 +63,12 @@ install-texmf: $(TEXMFGEN) sleep 2; \ fi -install-texmf-doc: preview.dvi +install-texmf-doc: preview.pdf $(MKINSTALLDIRS) $(DESTDIR)$(previewdocdir) - $(INSTALL_DATA) preview.dvi $(DESTDIR)$(previewdocdir) + $(INSTALL_DATA) preview.pdf $(DESTDIR)$(previewdocdir) uninstall: - rm -rf $(DESTDIR)$(previewdocdir)/preview.dvi $(DESTDIR)$(previewtexmfdir) + rm -rf $(DESTDIR)$(previewdocdir)/preview.dvi $(DESTDIR)$(previewdocdir)/preview.pdf $(DESTDIR)$(previewtexmfdir) preview-mk.ins: preview.dtx bootstrap.ins $(TEX) '\nonstopmode \input bootstrap.ins' @@ -88,9 +88,9 @@ preview.dvi: preview.drv preview.dtx preview.sty $(LATEX) '\nonstopmode \input preview.drv' preview.pdf: preview.drv preview.dtx preview.sty - $(PDFLATEX) '\nonstopmode \AtBeginDocument{\OnlyDescription}\input{preview.drv}' - $(PDFLATEX) '\nonstopmode \AtBeginDocument{\OnlyDescription}\input{preview.drv}' - $(PDFLATEX) '\nonstopmode \AtBeginDocument{\OnlyDescription}\input{preview.drv}' + $(PDFLATEX) '\nonstopmode \input{preview.drv}' + $(PDFLATEX) '\nonstopmode \input{preview.drv}' + $(PDFLATEX) '\nonstopmode \input{preview.drv}' clean: rm -f *~ *.aux *.dvi *.drv *.log preview.pdf
[AUCTeX-diffs] GNU AUCTeX branch, master, updated. 22f10dfe29e8310701da583e3847367ea5d7fb8c
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via 22f10dfe29e8310701da583e3847367ea5d7fb8c (commit) from 2b9fc54ec53bc2473b953c7ae728f80e1ede64bc (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit 22f10dfe29e8310701da583e3847367ea5d7fb8c Author: Ikumi Keita Date: Sat Oct 19 22:01:08 2019 +0900 ; * tex-info.el (): Update copyright year. diff --git a/tex-info.el b/tex-info.el index 4f8165d..22e726e 100644 --- a/tex-info.el +++ b/tex-info.el @@ -1,7 +1,7 @@ ;;; tex-info.el --- Support for editing Texinfo source. ;; Copyright (C) 1993, 1994, 1997, 2000, 2001, 2004, 2005, 2006, -;; 2011-2015, 2017, 2018 Free Software Foundation, Inc. +;; 2011-2015, 2017-2019 Free Software Foundation, Inc. ;; Maintainer: auctex-devel@gnu.org ;; Keywords: tex --- Summary of changes: tex-info.el | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) hooks/post-receive -- GNU AUCTeX ___ auctex-diffs mailing list auctex-di...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-diffs
[AUCTeX-commit] GNU AUCTeX branch, master, updated. 22f10dfe29e8310701da583e3847367ea5d7fb8c
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via 22f10dfe29e8310701da583e3847367ea5d7fb8c (commit) from 2b9fc54ec53bc2473b953c7ae728f80e1ede64bc (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit 22f10dfe29e8310701da583e3847367ea5d7fb8c Author: Ikumi Keita Date: Sat Oct 19 22:01:08 2019 +0900 ; * tex-info.el (): Update copyright year. --- Summary of changes: tex-info.el | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) hooks/post-receive -- GNU AUCTeX ___ auctex-commit mailing list auctex-com...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-commit
Re: [AUCTeX-devel] Adjusting filecontents env to new LaTeX
Hi Arash, >>>>> Arash Esbati writes: > newest LaTeX has a change to filecontents environment: > The filecontents environment now supports an optional argument in > which you can specify that it is allowed to overwrite an already > existing file; by default nothing is written if a file with the > given name exists anywhere in the search tree. ... Another change > is that this environment is now allowed anywhere in the document, > which means it provides everything (and more) of what the now > obsolete filecontents package provided. > This means a change to `LaTeX-env-contents which can look like this: This hunk raised error when I actually apply it to patch command, so I applied it manually. The new function works just fine. > One last thing to clarify is how do we like to handle the content in the > environment? I don't like to indent the content, so we could add > `filecontents' and `filecontents*' to `LaTeX-verbatim-environments' > which will also kill filling. Other way is to add them to > `LaTeX-document-regexp' which would allow filling. I think filecontents(*) environments are cousins of verbatim env, so AUCTeX should not indent automatically in them. Regards, Ikumi Keita ___ auctex-devel mailing list auctex-devel@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-devel
[AUCTeX-diffs] GNU AUCTeX branch, master, updated. cc52bce708a77654e1b89048cb8dce484a5e4e88
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via cc52bce708a77654e1b89048cb8dce484a5e4e88 (commit) from 65607e58baeedd85d31c7f1db2e7d1ee25f5031b (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit cc52bce708a77654e1b89048cb8dce484a5e4e88 Author: Ikumi Keita Date: Tue Oct 15 19:41:59 2019 +0900 Fix for revised file access controls of gs (bug#37719) * preview.el.in (preview-prepare-fast-conversion): Add commands for revised file access controls introduced after gs 9.27. (preview-gs-restart): Use "-d" option instead of "-s" because the former does not accept "%d" in OutputFile parameter. Don't wrap the file name into parentheses in line with this change. (preview-gs-flag-error): Adjust the reconstructed gs command line argument so that it matches with the above change. Add comments about limitations of this function. diff --git a/preview.el.in b/preview.el.in index 148f05b..03e1d9b 100644 --- a/preview.el.in +++ b/preview.el.in @@ -727,8 +727,8 @@ Gets the usual PROCESS and STRING parameters, see (setq preview-gs-sequence (list 1))) (setcdr preview-gs-sequence 1) (let* ((process-connection-type nil) - (outfile (format "-dOutputFile=%s" - (preview-ps-quote-filename + (outfile (format "-sOutputFile=%s" + (file-relative-name (format "%s/pr%d-%%d.%s" (car TeX-active-tempdir) (car preview-gs-sequence) @@ -1165,9 +1165,16 @@ NONREL is not NIL." (list file (setq preview-gs-dsc (preview-dsc-parse file)) (setq preview-gs-init-string - (concat (format "{<> setuserparams \ + ;; Add commands for revised file access controls introduced + ;; after gs 9.27 (bug#37719) + (concat (format "systemdict /.addcontrolpath known {%s} if\n" + (mapconcat (lambda (f) + (format "/PermitFileReading %s .addcontrolpath" + (preview-ps-quote-filename f))) +all-files "\n")) + (format "{<> setuserparams \ .locksafe} stopped pop " - (mapconcat 'preview-ps-quote-filename all-files "")) + (mapconcat #'preview-ps-quote-filename all-files "")) preview-gs-init-string (format " %s(r)file /.preview-ST 1 index def %s exec .preview-ST " (preview-ps-quote-filename file) @@ -1257,10 +1264,18 @@ Try \\[ps-run-start] \\[ps-run-buffer] and \ (defun preview-gs-flag-error (ov err) "Make an eps error flag in overlay OV for ERR string." + ;; N.B. Although this code shows command line of gs invocation and + ;; error together via mouse popup menu, they are not necessarilly + ;; associated with each other. There is a case that the command + ;; line is for "[...].prv/tmpXX/pr1-2.png" while the error is + ;; raised for "[...].prv/tmpXX/pr1-1.png". (c.f. bug#37719) (let* ((filenames (overlay-get ov 'filenames)) (file (car (nth 0 filenames))) -(outfile (format "-dOutputFile=%s" - (preview-ps-quote-filename +;; FIXME: This format isn't equal to actual invocation of gs +;; command constructed in `preview-gs-restart', which +;; contains "%d". +(outfile (format "-sOutputFile=%s" + (file-relative-name (car (nth 1 filenames) (ps-open `(lambda() (interactive "@") --- Summary of changes: preview.el.in | 27 +-- 1 file changed, 21 insertions(+), 6 deletions(-) hooks/post-receive -- GNU AUCTeX ___ auctex-diffs mailing list auctex-di...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-diffs
[AUCTeX-devel] New entry in amsmath.el (was Re: Adjusting filecontents env to new LaTeX)
Hi Arash, >>>>> Arash Esbati writes: > Ok, I pushed a change to AUCTeX repo acc. to this. Thanks for your time. I tried the new entry for \overunderset in amsmath.el, but it seems wrong to me with a minimal sample like this. \documentclass{article} \usepackage{amsmath} \begin{document} \(\overunderset{above}{i=below}{\prod}\) \end{document} Isn't the right entry like this? diff --git a/style/amsmath.el b/style/amsmath.el index 65aed4e2..2c06380b 100644 --- a/style/amsmath.el +++ b/style/amsmath.el @@ -65,7 +65,7 @@ '("xrightarrow" ["Below"] "Above") '("overset" "Accent symbol" "Symbol") '("underset" "Accent symbol" "Symbol") -'("overunderset" "Accent symbol" "Symbol below" "Symbol") +'("overunderset" "Symbol above" "Symbol below" "Accent symbol") '("dfrac" 2) '("tfrac" 2) '("binom" 2) I think the definition in amsmath.sty supports my idea: \newcommand{\overunderset}[3]{\binrel@{#3}% \binrel@@{\mathop{\kern\z@#3}\limits^{#1}_{#2}}} (The order of the arguments is different from \overset and \underset... I wonder why amsmath developer chose such confusing order.) Regards, Ikumi Keita ___ auctex-devel mailing list auctex-devel@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-devel
[AUCTeX-commit] GNU AUCTeX branch, master, updated. 39463a695dd0f91ad72204949a74e79279621cb4
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via 39463a695dd0f91ad72204949a74e79279621cb4 (commit) from 22f10dfe29e8310701da583e3847367ea5d7fb8c (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit 39463a695dd0f91ad72204949a74e79279621cb4 Author: Ikumi Keita Date: Sun Oct 20 00:32:26 2019 +0900 ; * preview.el.in (preview-gs-flag-error): Fix typo in comment. --- Summary of changes: preview.el.in | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) hooks/post-receive -- GNU AUCTeX ___ auctex-commit mailing list auctex-com...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-commit
[AUCTeX-diffs] GNU AUCTeX branch, master, updated. 39463a695dd0f91ad72204949a74e79279621cb4
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via 39463a695dd0f91ad72204949a74e79279621cb4 (commit) from 22f10dfe29e8310701da583e3847367ea5d7fb8c (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit 39463a695dd0f91ad72204949a74e79279621cb4 Author: Ikumi Keita Date: Sun Oct 20 00:32:26 2019 +0900 ; * preview.el.in (preview-gs-flag-error): Fix typo in comment. diff --git a/preview.el.in b/preview.el.in index 6699387..89b767a 100644 --- a/preview.el.in +++ b/preview.el.in @@ -1262,7 +1262,7 @@ Try \\[ps-run-start] \\[ps-run-buffer] and \ (defun preview-gs-flag-error (ov err) "Make an eps error flag in overlay OV for ERR string." ;; N.B. Although this code shows command line of gs invocation and - ;; error together via mouse popup menu, they are not necessarilly + ;; error together via mouse popup menu, they are not necessarily ;; associated with each other. There is a case that the command ;; line is for "[...].prv/tmpXX/pr1-2.png" while the error is ;; raised for "[...].prv/tmpXX/pr1-1.png". (c.f. bug#37719) --- Summary of changes: preview.el.in | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) hooks/post-receive -- GNU AUCTeX ___ auctex-diffs mailing list auctex-di...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-diffs
[AUCTeX-diffs] GNU AUCTeX branch, master, updated. 01dc048dedcb5e13152c9439612c817671aa560e
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via 01dc048dedcb5e13152c9439612c817671aa560e (commit) from cc52bce708a77654e1b89048cb8dce484a5e4e88 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit 01dc048dedcb5e13152c9439612c817671aa560e Author: Ikumi Keita Date: Tue Oct 15 21:49:05 2019 +0900 Change default for new Ghostscript * preview.el.in (preview-pdf-color-adjust-method): Change the default value to t and adjust doc string. * doc/preview-problems.texi (No images are displayed with gs 9.27 and earlier): Adjust its contents in line with the above change and change the title from "... newer". diff --git a/doc/preview-problems.texi b/doc/preview-problems.texi index e10fdd7..264e14c 100644 --- a/doc/preview-problems.texi +++ b/doc/preview-problems.texi @@ -19,7 +19,7 @@ newer versions of the problematic software or by simple patches. * Too small bounding boxes:: * x-symbol interoperation:: * Middle-clicks paste instead of toggling:: -* No images are displayed with gs 9.27 and newer:: +* No images are displayed with gs 9.27 and earlier:: @end menu If you find something not mentioned here, please send a bug report using @@ -139,8 +139,8 @@ search with @kbd{@key{RET}} before toggling and resume with @kbd{C-s C-s} or similar afterwards. Since previews over the current match will auto-open, anyway, this should not be much of a problem in practice. -@node No images are displayed with gs 9.27 and newer -@section No images are displayed with gs 9.27 and newer +@node No images are displayed with gs 9.27 and earlier +@section No images are displayed with gs 9.27 and earlier @previewlatex{} tries to adjust the foreground and background colors of generated images to those of Emacs. Unfortunately, incompatible changes @@ -148,22 +148,21 @@ introduced in Ghostscript 9.27 breaks the traditional method partially, and @previewlatex{} can display no images under certain circumstances. A new method implemented alternatively works only with Ghostscript > -9.27, so is disabled by default. If you are using those new -Ghostscript, customize the option -@code{preview-pdf-adjust-color-method}. +9.27. If you are using Ghostscript 9.27 or earlier, customize the +option @code{preview-pdf-adjust-color-method}. @defopt preview-pdf-adjust-color-method Method to adjust colors of images generated from @acronym{PDF}. It is not consulted when the @LaTeX{} command produces @acronym{DVI} files. -When the option is @code{t}, @previewlatex{} adjusts the FG and BG colors -of the generated images by the new method. This method requires that -Ghostscript has working @code{DELAYBIND} feature, thus is invalid with -gs 9.27 (and possibly < 9.27). +When the option is @code{t} (default), @previewlatex{} adjusts the FG +and BG colors of the generated images by the new method. This method +requires that Ghostscript has working @code{DELAYBIND} feature, thus is +invalid with gs 9.27 (and possibly < 9.27). -When it is @code{compatible} (default), @previewlatex{} uses traditional -method. This option is provided for backward compatibility with older -gs. See the below explanation for detail. +When it is @code{compatible}, @previewlatex{} uses traditional method. +This option is provided for backward compatibility with older gs. See +the below explanation for detail. When @code{nil}, no adjustment is done and ``black on white'' image is generated regardless of Emacs color. This is provided for fallback for @@ -191,6 +190,6 @@ written in the image although they may not match with your Emacs color well. @end enumerate -The default value will be changed to @code{t} after Ghostscript 9.28 is -released. +The default value used to be @code{compatible} for short period before +Ghostscript 9.50 was released but now is @code{t}. @end defopt diff --git a/preview.el.in b/preview.el.in index 03e1d9b..6699387 100644 --- a/preview.el.in +++ b/preview.el.in @@ -602,7 +602,7 @@ tag in the mode line." (setq preview-error-condition nil compilation-in-progress (delq process compilation-in-progress) -(defcustom preview-pdf-color-adjust-method 'compatible +(defcustom preview-pdf-color-adjust-method t "Method to adjust colors of images generated from PDF. It is not consulted when the latex command produces DVI files. @@ -638,10 +638,7 @@ that case. - Choose the value nil, which forces plain \"black on white\" appearance for the generated image. You can at least read what are written in the image although they may not match
[AUCTeX-commit] GNU AUCTeX branch, master, updated. 01dc048dedcb5e13152c9439612c817671aa560e
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via 01dc048dedcb5e13152c9439612c817671aa560e (commit) from cc52bce708a77654e1b89048cb8dce484a5e4e88 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit 01dc048dedcb5e13152c9439612c817671aa560e Author: Ikumi Keita Date: Tue Oct 15 21:49:05 2019 +0900 Change default for new Ghostscript * preview.el.in (preview-pdf-color-adjust-method): Change the default value to t and adjust doc string. * doc/preview-problems.texi (No images are displayed with gs 9.27 and earlier): Adjust its contents in line with the above change and change the title from "... newer". --- Summary of changes: doc/preview-problems.texi | 29 ++--- preview.el.in | 7 ++- 2 files changed, 16 insertions(+), 20 deletions(-) hooks/post-receive -- GNU AUCTeX ___ auctex-commit mailing list auctex-com...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-commit
Re: [AUCTeX-devel] TeXLive 2019 update broke preview-latex (was Re: Typeset LaTeX files with non-ascii file name in TeXLive 2018)
Hi Jean, >>>>> jfbu writes: > Hi Keita > The latest LaTeX \input allows spaces in filenames. We see that preview-latex > invocation is with added spaces at both ends of the filename. It can > be confirmed with quick testing that \input{ abc.tex } used to work > but now does not. Ah, thank you very much for your analysis. Now I see the true reason. > Do you remember why the extra spaces added for preview-latex invocation? If I remember correctly, these spaces are added for w32. I'll look into this and try to construct other invocation format valid for all OSes. Best regards, Ikumi Keita ___ auctex-devel mailing list auctex-devel@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-devel
Re: [AUCTeX-devel] TeXLive 2019 update broke preview-latex (was Re: Typeset LaTeX files with non-ascii file name in TeXLive 2018)
Hi all, 1. Examination and observation >>>>> Ikumi Keita writes: > It seems that the latest TeXLive 2019 update broke preview-latex. I > quote a failure log of the latest pdflatex and a successful log of the > pdflatex with the latest update reverted, at the last of this message. I removed "\detokenize" from preview.el locally and confirmed that preview-latex doesn't fail under the latest TeXLive 2019: , | Running `Preview-LaTeX' on `abc' with ``pdflatex -file-line-error "\nonstopmode\nofiles\PassOptionsToPackage{active,tightpage,auctex}{preview}\AtBeginDocument{\ifx\ifPreview\undefined\RequirePackage[displaymath,floats,graphics,textmath,sections,footnotes]{preview}[2004/11/05]\fi}" "\input" abc.tex'' | This is pdfTeX, Version 3.14159265-2.6-1.40.20 (TeX Live 2019) (preloaded format=pdflatex) | restricted \write18 enabled. | entering extended mode | LaTeX2e <2019-10-01> | | [...] | | Output written on abc.pdf (1 page, 15960 bytes). | Transcript written on abc.log. | | TeX Output exited as expected with code 1 at Mon Oct 7 15:41:40 | Running `Preview-PDF2DSC' with ``pdf2dsc abc.pdf abc.prv/tmpE34oe0/preview.dsc'' | | [...] | | Preview-Ghostscript finished at Mon Oct 7 15:41:43 ` I confirmed that preamble caching is also fine without \detokenize in .ini code of /AUCTEXINPUT. > Is this common for others? If so, I suspect that the change in the > internal of latex invalidated "\detokenize" that preview-latex uses to > input files with non-ascii names successfully. However, this suspicion was not totally correct. It seems that "\detokenize" is harmless under normal typeset (not with preview-latex), even with a non-ascii file name: , | Running `LaTeX' on `prim?rias' with ``pdflatex -file-line-error -interaction=nonstopmode "\pdfoutput=0 " "\input" \\detokenize\{\ prim\?rias.tex\ \}'' | This is pdfTeX, Version 3.14159265-2.6-1.40.20 (TeX Live 2019) (preloaded format=pdflatex) | restricted \write18 enabled. | entering extended mode | LaTeX2e <2019-10-01> | | [...] | | Output written on prim?rias.dvi (1 page, 340 bytes). | Transcript written on prim?rias.log. ` I can view the generated dvi file by xdvi without any trouble. Thus I infer that it is something in the preamble code "\nonstopmode ... {preview}[2004/11/05]\fi}" in preview-latex that conflicts with \detokenize. I have no idea whether this is a bug of the latest LaTeX or not. -- 2. What to do I think that we have basically two ways to choose from. (1) Just remove \detokenize from preview.el This makes preview-latex fail with non-ascii file name under TL2018 and early TL2019 (and probably under MikTeX and W32TeX within some interval between 2018 and 2019). We can live with it by answering to reporters of the errors caused by this change with "Update your TeX distribution to the latest version." (2) Introduce a new customize option so that the user can choose whether to use \detokenize in preview-latex or not What do you think about this? Regards, Ikumi Keita ___ auctex-devel mailing list auctex-devel@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-devel
Re: [AUCTeX-devel] TeXLive 2019 update broke preview-latex (was Re: Typeset LaTeX files with non-ascii file name in TeXLive 2018)
>>>>> Ikumi Keita writes: >> Do you remember why the extra spaces added for preview-latex invocation? > If I remember correctly, these spaces are added for w32. I'll look into > this and try to construct other invocation format valid for all OSes. It seems that it's just enough to remove braces around the argument of \input. I think the attached small patch fixes the problem. I'll check whether the regression test passes, and push the fix with suitable commit message soon. Regards, Ikumi Keita diff --git a/preview.el.in b/preview.el.in index e107d5e9..a5c77973 100644 --- a/preview.el.in +++ b/preview.el.in @@ -2305,7 +2305,7 @@ list of LaTeX commands is inserted just before \\begin{document}." (defcustom preview-LaTeX-command '("%`%l \"\\nonstopmode\\nofiles\ \\PassOptionsToPackage{" ("," . preview-required-option-list) "}{preview}\ \\AtBeginDocument{\\ifx\\ifPreview\\undefined" -preview-default-preamble "\\fi}\"%' \"{\\detokenize{\" %t \"}}\"") +preview-default-preamble "\\fi}\"%' \"\\detokenize{\" %t \"}\"") ;; Since TeXLive 2018, the default encoding for LaTeX files has been ;; changed to UTF-8 if used with classic TeX or pdfTeX. I.e., ;; \usepackage[utf8]{inputenc} is enabled by default in (pdf)latex. @@ -3381,7 +3381,7 @@ This is passed through `preview-do-replacements'." ;; Discard all other options. '(("\\`\\([^ ]+\\)\ \\(?: +\\(?:\\(--?kanji[= ][^ ]+\\)\\|-\\(?:[^ \\\"]\\|.\\|\"[^\"]*\"\\)*\\)\\)*.*\ - \"input\" \"{detokenize{\" \\(.*\\) \"}}\"\\'" + \"input\" \"detokenize{\" \\(.*\\)\"} \"\\'" . ("\\1 \\2 -interaction=nonstopmode -file-line-error " preview-format-name " \"/AUCTEXINPUT{\" \\3 \"}\""))) ;; See the ini file code below in `preview-cache-preamble' for the ___ auctex-devel mailing list auctex-devel@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-devel
[AUCTeX-commit] GNU AUCTeX branch, master, updated. eafb11477c08531b823de387f7bdf0b5f8b4a4dd
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via eafb11477c08531b823de387f7bdf0b5f8b4a4dd (commit) from 7cd329f06daf166fb5fb6a6e3959935c786c91a9 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit eafb11477c08531b823de387f7bdf0b5f8b4a4dd Author: Ikumi Keita Date: Mon Oct 7 22:47:10 2019 +0900 Fix preview-latex for TeXLive 2019 update * preview.el.in (preview-LaTeX-command, preview-undump-replacements): Remove the brace around the argument of \input. --- Summary of changes: preview.el.in | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) hooks/post-receive -- GNU AUCTeX ___ auctex-commit mailing list auctex-com...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-commit
[AUCTeX-diffs] GNU AUCTeX branch, master, updated. eafb11477c08531b823de387f7bdf0b5f8b4a4dd
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via eafb11477c08531b823de387f7bdf0b5f8b4a4dd (commit) from 7cd329f06daf166fb5fb6a6e3959935c786c91a9 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit eafb11477c08531b823de387f7bdf0b5f8b4a4dd Author: Ikumi Keita Date: Mon Oct 7 22:47:10 2019 +0900 Fix preview-latex for TeXLive 2019 update * preview.el.in (preview-LaTeX-command, preview-undump-replacements): Remove the brace around the argument of \input. diff --git a/preview.el.in b/preview.el.in index e107d5e..a5c7797 100644 --- a/preview.el.in +++ b/preview.el.in @@ -2305,7 +2305,7 @@ list of LaTeX commands is inserted just before \\begin{document}." (defcustom preview-LaTeX-command '("%`%l \"\\nonstopmode\\nofiles\ \\PassOptionsToPackage{" ("," . preview-required-option-list) "}{preview}\ \\AtBeginDocument{\\ifx\\ifPreview\\undefined" -preview-default-preamble "\\fi}\"%' \"{\\detokenize{\" %t \"}}\"") +preview-default-preamble "\\fi}\"%' \"\\detokenize{\" %t \"}\"") ;; Since TeXLive 2018, the default encoding for LaTeX files has been ;; changed to UTF-8 if used with classic TeX or pdfTeX. I.e., ;; \usepackage[utf8]{inputenc} is enabled by default in (pdf)latex. @@ -3381,7 +3381,7 @@ This is passed through `preview-do-replacements'." ;; Discard all other options. '(("\\`\\([^ ]+\\)\ \\(?: +\\(?:\\(--?kanji[= ][^ ]+\\)\\|-\\(?:[^ \\\"]\\|.\\|\"[^\"]*\"\\)*\\)\\)*.*\ - \"input\" \"{detokenize{\" \\(.*\\) \"}}\"\\'" + \"input\" \"detokenize{\" \\(.*\\)\"} \"\\'" . ("\\1 \\2 -interaction=nonstopmode -file-line-error " preview-format-name " \"/AUCTEXINPUT{\" \\3 \"}\""))) ;; See the ini file code below in `preview-cache-preamble' for the --- Summary of changes: preview.el.in | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) hooks/post-receive -- GNU AUCTeX ___ auctex-diffs mailing list auctex-di...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-diffs
[AUCTeX-diffs] GNU AUCTeX branch, master, updated. 65607e58baeedd85d31c7f1db2e7d1ee25f5031b
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via 65607e58baeedd85d31c7f1db2e7d1ee25f5031b (commit) from eafb11477c08531b823de387f7bdf0b5f8b4a4dd (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit 65607e58baeedd85d31c7f1db2e7d1ee25f5031b Author: Ikumi Keita Date: Tue Oct 8 03:55:21 2019 +0900 * preview.el.in (preview-undump-replacements): Fix last commit. diff --git a/preview.el.in b/preview.el.in index a5c7797..148f05b 100644 --- a/preview.el.in +++ b/preview.el.in @@ -3381,7 +3381,7 @@ This is passed through `preview-do-replacements'." ;; Discard all other options. '(("\\`\\([^ ]+\\)\ \\(?: +\\(?:\\(--?kanji[= ][^ ]+\\)\\|-\\(?:[^ \\\"]\\|.\\|\"[^\"]*\"\\)*\\)\\)*.*\ - \"input\" \"detokenize{\" \\(.*\\)\"} \"\\'" + \"input\" \"detokenize{\" \\(.*\\) \"}\"\\'" . ("\\1 \\2 -interaction=nonstopmode -file-line-error " preview-format-name " \"/AUCTEXINPUT{\" \\3 \"}\""))) ;; See the ini file code below in `preview-cache-preamble' for the --- Summary of changes: preview.el.in | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) hooks/post-receive -- GNU AUCTeX ___ auctex-diffs mailing list auctex-di...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-diffs
[AUCTeX-commit] GNU AUCTeX branch, master, updated. 65607e58baeedd85d31c7f1db2e7d1ee25f5031b
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via 65607e58baeedd85d31c7f1db2e7d1ee25f5031b (commit) from eafb11477c08531b823de387f7bdf0b5f8b4a4dd (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit 65607e58baeedd85d31c7f1db2e7d1ee25f5031b Author: Ikumi Keita Date: Tue Oct 8 03:55:21 2019 +0900 * preview.el.in (preview-undump-replacements): Fix last commit. --- Summary of changes: preview.el.in | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) hooks/post-receive -- GNU AUCTeX ___ auctex-commit mailing list auctex-com...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-commit
Re: [AUCTeX-devel] TeXLive 2019 update broke preview-latex (was Re: Typeset LaTeX files with non-ascii file name in TeXLive 2018)
Hi Jean, thank you for detailed explanation. >>>>> jfbu writes: > Le 07/10/2019 à 22:14, jfbu a écrit : > if /AUCTEXINPUT starts using \input{\detokenize{##1}} then > its invocation in preview-undump-replacements() should not > add extra leading and trailing spaces else we will have same problem > i.e. the line > preview-format-name " \"/AUCTEXINPUT{\" \\3 \"}\""))) > in preview.el > should, if I understand correctly the syntax be > preview-format-name " \"/AUCTEXINPUT{\"\\3\"}\""))) > (the kept \" pair if for LaTeX earlier than 2019-10-01 to > support spaces in filenames \input{"a b c.tex"} works > but \input{a b c.tex} works only since LaTeX 2019-10-01} > But perhaps there is Windows issue with \" ?? Yeah. If I drop the spaces around the file name of "\detokenize{" %t "}" in `preview-LaTeX-command' thus turning it into "\detokenize{"%t"}" , w32 shell (cmd.exe) does not process the quotes as expected when the file name contains space: , | Running `Preview-LaTeX' on `file with space' with ``pdflatex -file-line-error "\nonstopmode\nofiles\PassOptionsToPackage{active,tightpage,auctex}{preview}\AtBeginDocument{\ifx\ifPreview\undefined\RequirePackage[displaymath,floats,graphics,textmath,sections,footnotes]{preview}[2004/11/05]\fi}" "\input" "\detokenize{"^"\^"file with space.tex\^"^""}"'' | ! Unbalanced quotes in argument space.tex"} | | TeX Output exited as expected with code 1 at Tue Oct 8 13:18:06 | LaTeX: LaTeX found no preview images ` I personally don't think it's worth working on for file name beginning with a space now. Regards, Ikumi Keita ___ auctex-devel mailing list auctex-devel@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-devel
Re: [AUCTeX-devel] TeXLive 2019 update broke preview-latex (was Re: Typeset LaTeX files with non-ascii file name in TeXLive 2018)
>>>>> jfbu writes: > Dropping all extras related to LaTeX preview package this is something like > $ pdflatex "\input" "\detokenize{" \"\ test\ e\́e\́e\́\ .tex\" "}" > where I added non-ascii in file name for good measure > I had forgotten mainly the \"...\". > and this succeeds on my mac os x. > The rest of the toolchain (using ghostscript for image snippets) > seems to work fine. Fine. Glad to hear that :-). > However syntax highlighting in the LaTeX-mode buffer with filename > starting with space is off Indeed! It's strange but I have no idea why. > Sorry about possibly causing delays No problem. Regards, Ikumi Keita ___ auctex-devel mailing list auctex-devel@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-devel
[AUCTeX-devel] TeXLive 2019 update broke preview-latex (was Re: Typeset LaTeX files with non-ascii file name in TeXLive 2018)
Hi all, It seems that the latest TeXLive 2019 update broke preview-latex. I quote a failure log of the latest pdflatex and a successful log of the pdflatex with the latest update reverted, at the last of this message. At first, I thought that it is only related with non-ascii file names, but it turned out to be not so. Just a plain file name of "abc.tex" fails with the same error. Is this common for others? If so, I suspect that the change in the internal of latex invalidated "\detokenize" that preview-latex uses to input files with non-ascii names successfully. I haven't tested other engines (xelatex and lualatex). I'm afraid that we have to delay the release of AUCTeX 12.2, due to this problem. Regards, Ikumi Keita ,[ fail with LaTeX 2019-10-01 ] | Running `Preview-LaTeX' on `prim?rias' with ``pdflatex -file-line-error "\nonstopmode\nofiles\PassOptionsToPackage{active,tightpage,auctex}{preview}\AtBeginDocument{\ifx\ifPreview\undefined\RequirePackage[displaymath,floats,graphics,textmath,sections,footnotes]{preview}[2004/11/05]\fi}" "\input" "{\detokenize{" prim\?rias.tex "}}"'' | This is pdfTeX, Version 3.14159265-2.6-1.40.20 (TeX Live 2019) (preloaded format=pdflatex) | restricted \write18 enabled. | entering extended mode | LaTeX2e <2019-10-01> | | No auxiliary output files. | | | ! LaTeX Error: File ` prim?rias.tex ' not found. | | Type X to quit or to proceed, | or enter new name. (Default extension: tex ) | | Enter file name: | ! Emergency stop. | | | <*> ...]\fi} \input {\detokenize{ prim?rias.tex }} | ^^M | ! ==> Fatal error occurred, no output PDF file produced! | Transcript written on texput.log. | | TeX Output exited as expected with code 1 at Sun Oct 6 23:47:00 | LaTeX: LaTeX found no preview images ` ,[ success with LaTeX 2018-12-01 ] | Running `Preview-LaTeX' on `prim?rias' with ``pdflatex -file-line-error "\nonstopmode\nofiles\PassOptionsToPackage{active,tightpage,auctex}{preview}\AtBeginDocument{\ifx\ifPreview\undefined\RequirePackage[displaymath,floats,graphics,textmath,sections,footnotes]{preview}[2004/11/05]\fi}" "\input" "{\detokenize{" prim\?rias.tex "}}"'' | This is pdfTeX, Version 3.14159265-2.6-1.40.20 (TeX Live 2019) (preloaded format=pdflatex) | restricted \write18 enabled. | entering extended mode | LaTeX2e <2018-12-01> | | No auxiliary output files. | | (./prim?rias.tex (/usr/local/texlive/2019/texmf-dist/tex/latex/base/article.cls | Document Class: article 2018/09/03 v1.4i Standard LaTeX document class | (/usr/local/texlive/2019/texmf-dist/tex/latex/base/size10.clo)) | No file prim?rias.aux. | (/usr/local/texlive/texmf-local/tex/latex/preview/preview.sty (/usr/local/texlive/2019/texmf-dist/tex/generic/luatex85/luatex85.sty) (/usr/local/texlive/texmf-local/tex/latex/preview/prtightpage.def) (/usr/local/texlive/texmf-local/tex/latex/preview/prauctex.def | No auxiliary output files. | | | (/usr/local/texlive/texmf-local/tex/latex/preview/prauctex.cfg)) (/usr/local/texlive/texmf-local/tex/latex/preview/prfootnotes.def) | Preview: Fontsize 10pt | Preview: PDFoutput 1 | ) | ./prim?rias.tex:5: Preview: Snippet 1 started. | <-><-> | | l.5 \section | {primárias} | Preview: Tightpage -32891 -32891 32891 32891 | ./prim?rias.tex:5: Preview: Snippet 1 ended.(655359+183500x22609920). | <-><-> | | l.5 \section{primárias} | | [1{/usr/local/texlive/2019/texmf-var/fonts/map/pdftex/updmap/pdftex.map}] | ./prim?rias.tex:6: Preview: Snippet 2 started. | <-><-> | | l.6 \( | abc\) | ./prim?rias.tex:6: Preview: Snippet 2 ended.(455111+0x911285). | <-><-> | | l.6 \(abc\) | | [2] ) | (see the transcript file for additional information) | Output written on prim?rias.pdf (2 pages, 17419 bytes). | Transcript written on prim?rias.log. | | TeX Output exited as expected with code 1 at Sun Oct 6 21:36:06 | Running `Preview-PDF2DSC' with ``pdf2dsc prim\?rias.pdf prim\?rias.prv/tmpzoF6qL/preview.dsc'' | | Preview-PDF2DSC finished at Sun Oct 6 21:36:09 | Running `Preview-Ghostscript' with ``/usr/local/texlive/2019/bin/amd64-freebsd/rungs -dOutputFile\=\(prim\?rias.prv/tmpzoF6qL/pr1-\%d.png\) -q -dDELAYSAFER -dNOPAUSE -DNOPLATFONTS -dPrinted -dTextAlphaBits\=4 -dGraphicsAlphaBits\=4 -sDEVICE\=png16m -r108.606x108.587'' | | Preview-Ghostscript finished at Sun Oct 6 21:36:10 ` ___ auctex-devel mailing list auctex-devel@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-devel
Re: Makefile minor issues
Hi all, >>>>> Ikumi Keita writes: >> I noticed some minor issues about Makefiles of AUCTeX. >> (A) Makefile in latex/ subdirectory >> (B) Clean targets in Makefiles >> I think that it is straightforward to sort out (B), but (A) is not so >> trivial. What do you think about them? > I made two patches for (A) and (B). I think it's OK to apply (B), so > will install in the git repo. Does anyone have objection to (A), which > effectively phases out dvi format and uses pdf instead of it? I committed the proposed patch for (A). Feel free to revert it if you find something wrong about it. Regards, Ikumi Keita
[AUCTeX-commit] GNU AUCTeX branch, master, updated. 2ceda21f6a0d25f5d0b5aef2edcbac9d52b17150
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via 2ceda21f6a0d25f5d0b5aef2edcbac9d52b17150 (commit) from 6c25ffeeb2a4d4f711431b7512cc702746613fbf (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit 2ceda21f6a0d25f5d0b5aef2edcbac9d52b17150 Author: Ikumi Keita Date: Sat Dec 14 21:47:58 2019 +0900 Use pdf rather than dvi for preview package document * latex/Makefile.in: Generate preview.pdf with full contents and use it instead of preview.dvi. --- Summary of changes: latex/Makefile.in | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) hooks/post-receive -- GNU AUCTeX ___ auctex-commit mailing list auctex-com...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-commit
[AUCTeX-diffs] GNU AUCTeX branch, master, updated. 2ceda21f6a0d25f5d0b5aef2edcbac9d52b17150
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via 2ceda21f6a0d25f5d0b5aef2edcbac9d52b17150 (commit) from 6c25ffeeb2a4d4f711431b7512cc702746613fbf (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit 2ceda21f6a0d25f5d0b5aef2edcbac9d52b17150 Author: Ikumi Keita Date: Sat Dec 14 21:47:58 2019 +0900 Use pdf rather than dvi for preview package document * latex/Makefile.in: Generate preview.pdf with full contents and use it instead of preview.dvi. diff --git a/latex/Makefile.in b/latex/Makefile.in index f342b6b..ab63647 100644 --- a/latex/Makefile.in +++ b/latex/Makefile.in @@ -44,7 +44,7 @@ MKINSTALLDIRS = ../mkinstalldirs .PHONY: all install install-texmf install-texmf-doc uninstall clean distclean -all: $(TEXMFGEN) preview.drv preview-mk.ins preview.dvi preview.pdf +all: $(TEXMFGEN) preview.drv preview-mk.ins preview.pdf install: install-texmf install-texmf-doc @@ -63,12 +63,12 @@ install-texmf: $(TEXMFGEN) sleep 2; \ fi -install-texmf-doc: preview.dvi +install-texmf-doc: preview.pdf $(MKINSTALLDIRS) $(DESTDIR)$(previewdocdir) - $(INSTALL_DATA) preview.dvi $(DESTDIR)$(previewdocdir) + $(INSTALL_DATA) preview.pdf $(DESTDIR)$(previewdocdir) uninstall: - rm -rf $(DESTDIR)$(previewdocdir)/preview.dvi $(DESTDIR)$(previewtexmfdir) + rm -rf $(DESTDIR)$(previewdocdir)/preview.dvi $(DESTDIR)$(previewdocdir)/preview.pdf $(DESTDIR)$(previewtexmfdir) preview-mk.ins: preview.dtx bootstrap.ins $(TEX) '\nonstopmode \input bootstrap.ins' @@ -88,9 +88,9 @@ preview.dvi: preview.drv preview.dtx preview.sty $(LATEX) '\nonstopmode \input preview.drv' preview.pdf: preview.drv preview.dtx preview.sty - $(PDFLATEX) '\nonstopmode \AtBeginDocument{\OnlyDescription}\input{preview.drv}' - $(PDFLATEX) '\nonstopmode \AtBeginDocument{\OnlyDescription}\input{preview.drv}' - $(PDFLATEX) '\nonstopmode \AtBeginDocument{\OnlyDescription}\input{preview.drv}' + $(PDFLATEX) '\nonstopmode \input{preview.drv}' + $(PDFLATEX) '\nonstopmode \input{preview.drv}' + $(PDFLATEX) '\nonstopmode \input{preview.drv}' clean: rm -f *~ *.aux *.dvi *.drv *.log preview.pdf --- Summary of changes: latex/Makefile.in | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) hooks/post-receive -- GNU AUCTeX ___ auctex-diffs mailing list auctex-di...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-diffs
Re: LaTeX 2020-02-02 release
Hi Arash and all, >>>>> Arash Esbati writes: > This change means we have to add 2 entries to `LaTeX-font-list' for > \textsw and \textulc. I suggest the keys `C-w' and `C-l', i.e.: > (?\C-l "\\textulc{" "}") > (?\C-w "\\textsw{" "}") It might be good for menu-oriented users to have those new entries in `LaTeX-mode-menu' as well, whereas it might be preferable to avoid commands not used so often in order to keep the menu concise. I'm neutral between these two options. What do you think about it? Regards, Ikumi Keita
Re: defcustoms in styles
>>>>> Arash Esbati writes: > For the start, is there any particular reason for shortvrb.el containing > the variable? I don't see one. The way I see it `LaTeX-shortvrb-chars' > can be moved to tex-style.el. I don't see, either. I think it's OK to move it to tex-style.el. By the way, following code in `TeX-add-style-hook' part seems a bit strange to me: -- (when (and LaTeX-shortvrb-chars (fboundp 'font-latex-set-syntactic-keywords) (eq TeX-install-font-lock 'font-latex-setup)) (font-latex-add-to-syntax-alist [...] -- I think this `fboundp' test is a mistake of (fboundp 'font-latex-add-to-syntax-alist) . And if we modify the code in this way, `declare-function' about `font-latex-add-to-syntax-alist' in the former half of this file can be omitted. Regards, Ikumi Keita
Re: LaTeX 2020-02-02 release
Hi Arash, >>>>> Arash Esbati writes: > Thanks for your response. I had a look at `LaTeX-mode-menu' reg. adding > the new commands but honestly, I wasn't sure where to put them. I > couldn't see the idea behind the sorting of the commands in that menu > which looks like this: It seems that there is no particular reason in that order. > Would it make sense to rework the menu entirely and sort the commands > alphabetically after the attribute as described in fntguide.pdf (page > 6), i.e., family, series and shape or something like that? Yeah, it's very likely as you say. At least I think so. Regards, Ikumi Keita
Re: LaTeX 2020-02-02 release
>>>>> Arash Esbati writes: > Ok, here the next incarnation. I moved textnormal at the end, this for > me the `misc group'. WDYT? Looks nice :-) Cheers, Ikumi Keita
Re: LaTeX 2020-02-02 release
>>>>> Arash Esbati writes: > Thanks, that makes me feel better and not so alone in that corner :-) :-) > Ok, does a setup like this make more sense? How about bringing up "Emphasize" to the top of those commands? It has a bit different meaning among them, and I suppose it is used very frequently. Bye, Ikumi Keita
[AUCTeX-commit] GNU AUCTeX branch, master, updated. f2bac9dcde12d939a5c1267fdba02c2a869bd406
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via f2bac9dcde12d939a5c1267fdba02c2a869bd406 (commit) from 4c1bb12d1e036fc2c639b7d8c7661df344f7f8d3 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit f2bac9dcde12d939a5c1267fdba02c2a869bd406 Author: Ikumi Keita Date: Sat Dec 21 03:30:04 2019 +0900 Improve environment insertion (bug#35284) * latex.el (LaTeX-insert-environment): Place the point and the mark at appropriate place. * tests/latex/latex-test.el (LaTeX-insert-environment-with-active-region): New test. --- Summary of changes: latex.el | 88 +++ tests/latex/latex-test.el | 218 +- 2 files changed, 286 insertions(+), 20 deletions(-) hooks/post-receive -- GNU AUCTeX ___ auctex-commit mailing list auctex-com...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-commit
[AUCTeX-diffs] GNU AUCTeX branch, master, updated. f2bac9dcde12d939a5c1267fdba02c2a869bd406
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via f2bac9dcde12d939a5c1267fdba02c2a869bd406 (commit) from 4c1bb12d1e036fc2c639b7d8c7661df344f7f8d3 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit f2bac9dcde12d939a5c1267fdba02c2a869bd406 Author: Ikumi Keita Date: Sat Dec 21 03:30:04 2019 +0900 Improve environment insertion (bug#35284) * latex.el (LaTeX-insert-environment): Place the point and the mark at appropriate place. * tests/latex/latex-test.el (LaTeX-insert-environment-with-active-region): New test. diff --git a/latex.el b/latex.el index aaa4a1f..f57fcfb 100644 --- a/latex.el +++ b/latex.el @@ -668,41 +668,91 @@ environment just inserted, the buffer position just before (concat "^\\([ \t]*" TeX-comment-start-regexp "+\\)+[ \t]*")) (setq prefix (match-string 0 ;; What to do with the line containing point. -(cond ((save-excursion (beginning-of-line) +;; - Open a new empty line for later insertion of "\begin{foo}" and +;; put the point there. +;; - If there were at first any non-whitespace texts between the +;; point and EOL, send them into their new own line with possible +;; comment prefix. +(cond (;; When the entire line consists of whitespaces except + ;; possible prefix... + (save-excursion (beginning-of-line) (looking-at (concat prefix "[ \t]*$"))) + ;; ...make the line empty and put the point there. (delete-region (match-beginning 0) (match-end 0))) - ((TeX-looking-at-backward (concat "^" prefix "[ \t]*") + (;; When there are only whitespaces except possible prefix + ;; between the point and BOL (including the case the point + ;; is at BOL)... + (TeX-looking-at-backward (if prefix + (concat "^\\(" prefix "\\)?[ \t]*") + "^[ \t]*") (line-beginning-position)) + ;; ...in this case, we have non-whitespace texts between + ;; the point and EOL, so send the entire line into a new + ;; next line and put the point on the empty line just + ;; created. (beginning-of-line) (newline) (beginning-of-line 0)) - ((bolp) + (;; In all other cases... + t + ;; ...insert a new empty line after deleting all + ;; whitespaces around the point, put the point there... (delete-horizontal-space) - (newline) - (beginning-of-line 0)) - (t - (delete-horizontal-space) - (newline 2) - (when prefix (insert prefix)) - (beginning-of-line 0))) + (if (eolp) + (newline) +;; ...and if there were at first any non-whitespace texts +;; between (the original position of) the point and EOL, +;; send them into a new next line with possible comment +;; prefix. +(newline 2) +(when prefix (insert prefix)) +(beginning-of-line 0 ;; What to do with the line containing mark. +;; If there is active region... (when active-mark + ;; - Open a new empty line for later insertion of "\end{foo}" + ;; and put the mark there. + ;; - If there were at first any non-whitespace texts between the + ;; mark and EOL, pass them over the empty line and put them on + ;; their own line with possible comment prefix. (save-excursion (goto-char (mark)) - (cond ((save-excursion (beginning-of-line) - (or (looking-at (concat prefix "[ \t]*$")) - (looking-at "[ \t]*$"))) + (cond (;; When the entire line consists of whitespaces except + ;; possible prefix... + (save-excursion (beginning-of-line) + (looking-at + (if prefix + (concat "\\(" prefix "\\)?[ \t]*$") + "[ \t]*$"))) + ;; ...make the line empty and put the mark there. (delete-region (match-beginning 0) (match-end 0))) - ((TeX-looking-at-backward (concat "^" prefix "[ \t]*") + (;; When there are only whitespac
[AUCTeX-diffs] GNU AUCTeX branch, master, updated. 1d72a5ceec85f507a252ccc41f9a5abda966e110
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via 1d72a5ceec85f507a252ccc41f9a5abda966e110 (commit) via 89798122cb96d78a8b56178d46a0fb40ba85342a (commit) via 7565d7c8d9e8bb78bdd3635824979a7bb78b2e07 (commit) via 68247ff4680eabcbad11853447880f5a77735112 (commit) from 8cd9db38c6eba5066dd02cf9166e51160f93abe9 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit 1d72a5ceec85f507a252ccc41f9a5abda966e110 Author: Ikumi Keita Date: Sun Jan 5 21:15:02 2020 +0900 Remove compatibility code for older emacsen According to NEWS.22, `font-lock-extra-managed-props' is available in all supported emacsen. * font-latex.el (font-latex-script,font-latex-script-char): Remove `boundp' test for `font-lock-extra-managed-props'. (font-latex-unfontify-region): Don't play with `invisible' text property because it is now handled by font-lock thanks to `font-lock-extra-managed-props'. diff --git a/font-latex.el b/font-latex.el index 0e767c7..81e52c6 100644 --- a/font-latex.el +++ b/font-latex.el @@ -1375,8 +1375,7 @@ If optional argument is non-nil, print status messages." (if (and (eq (car-safe prop) 'raise) (null (cddr prop))) (put-text-property beg next 'display nil)) - (setq beg next))) -(remove-text-properties start end '(invisible + (setq beg next) (defun font-latex-after-hacking-local-variables () "Refresh fontification if required by updates of file-local variables. @@ -2054,27 +2053,15 @@ END marks boundaries for searching for quotation ends." (while (eq (char-before pos) ?\\) (setq pos (1- pos) odd (not odd))) odd))) -;; Adding other text properties than `face' is supported by -;; `font-lock-apply-highlight' in CVS Emacsen since 2001-10-28. -;; With the introduction of this feature the variable -;; `font-lock-extra-managed-props' was introduced and serves here -;; for feature checking. -(let ((extra-props-flag (boundp 'font-lock-extra-managed-props))) - (if (eq (char-after pos) ?_) - (if extra-props-flag - (font-latex--get-script-props pos :sub) - 'font-latex-subscript-face) - (if extra-props-flag - (font-latex--get-script-props pos :super) - 'font-latex-superscript-face) +(if (eq (char-after pos) ?_) + (font-latex--get-script-props pos :sub) + (font-latex--get-script-props pos :super (defun font-latex-script-char (pos) "Return face and display spec for subscript and superscript character at POS." - (if (boundp 'font-lock-extra-managed-props) - `(face font-latex-script-char-face -,@(when (eq font-latex-fontify-script 'invisible) -'(invisible t))) -'font-latex-script-char-face)) + `(face font-latex-script-char-face +,@(when (eq font-latex-fontify-script 'invisible) +'(invisible t ;;; docTeX commit 89798122cb96d78a8b56178d46a0fb40ba85342a Author: Ikumi Keita Date: Sun Jan 5 20:03:11 2020 +0900 Don't use obsolete variable According to NEWS.20, `font-lock' no longer supports `font-lock-comment-start-regexp'. * font-latex.el (defvar): Remove `defvar' for `font-lock-comment-start-regexp'. (font-latex-setup): Don't include `font-lock-comment-start-regexp' in `font-lock-defaults'. diff --git a/font-latex.el b/font-latex.el index de7d5ea..0e767c7 100644 --- a/font-latex.el +++ b/font-latex.el @@ -1221,9 +1221,6 @@ have changed." ;;; Setup -(defvar font-lock-comment-start-regexp nil - "Regexp to match the start of a comment.") - (defvar font-latex-extend-region-functions nil "List of functions extending the region for multiline constructs. @@ -1281,8 +1278,7 @@ triggers Font Lock to recognize the change." `((font-latex-keywords font-latex-keywords-1 font-latex-keywords-2) nil nil ,font-latex-syntax-alist nil)) (variables -'((font-lock-comment-start-regexp . "%") - (font-lock-mark-block-function . mark-paragraph) +'((font-lock-mark-block-function . mark-paragraph) (font-lock-fontify-region-function . font-latex-fontify-region) (font-lock-unfontify-region-function commit 7565d7c8d9e8bb78bdd3635824979a7bb78b2e07 Author: Ikumi Keita Date: Sun Jan 5 19:44:13 2020 +0900 Remove compatibility code for older emacsen Since `font-lock-multiline' is available in all supported
[AUCTeX-diffs] GNU AUCTeX branch, master, updated. 8cd9db38c6eba5066dd02cf9166e51160f93abe9
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via 8cd9db38c6eba5066dd02cf9166e51160f93abe9 (commit) from f700ec0f766d6f2b5a1bd375e1e49294de625c6d (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit 8cd9db38c6eba5066dd02cf9166e51160f93abe9 Author: Ikumi Keita Date: Fri Jan 3 19:51:20 2020 +0900 Add changelog for my previous bug fix * doc/changes.texi (News in 12.3): Add a new entry for bug fix of `LaTeX-insert-environment' (C-c C-e). diff --git a/doc/changes.texi b/doc/changes.texi index 47e1a06..ae0b84a 100644 --- a/doc/changes.texi +++ b/doc/changes.texi @@ -1,5 +1,5 @@ @c This is part of the AUCTeX manual. -@c Copyright (C) 1994-2002, 2004-2010, 2012-2019 Free Software +@c Copyright (C) 1994-2002, 2004-2010, 2012-2020 Free Software @c Foundation, Inc. @c See file auctex.texi for copying conditions. @include macros.texi @@ -8,6 +8,16 @@ @end ifset +@heading News in 12.3 +@itemize @bullet +@item +Insertion of environments in @LaTeX{} documents (i.e. @kbd{C-c C-e}) was +improved. The former code had a few bugs, which sometimes resulted in +either spurious empty line or spurious comment prefix, or both, +especially when the region is active. Those bugs are now fixed. + +@end itemize + @heading News in 12.2 @itemize @bullet --- Summary of changes: doc/changes.texi | 12 +++- 1 file changed, 11 insertions(+), 1 deletion(-) hooks/post-receive -- GNU AUCTeX ___ auctex-diffs mailing list auctex-di...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-diffs
[AUCTeX-commit] GNU AUCTeX branch, master, updated. 8cd9db38c6eba5066dd02cf9166e51160f93abe9
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via 8cd9db38c6eba5066dd02cf9166e51160f93abe9 (commit) from f700ec0f766d6f2b5a1bd375e1e49294de625c6d (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit 8cd9db38c6eba5066dd02cf9166e51160f93abe9 Author: Ikumi Keita Date: Fri Jan 3 19:51:20 2020 +0900 Add changelog for my previous bug fix * doc/changes.texi (News in 12.3): Add a new entry for bug fix of `LaTeX-insert-environment' (C-c C-e). --- Summary of changes: doc/changes.texi | 12 +++- 1 file changed, 11 insertions(+), 1 deletion(-) hooks/post-receive -- GNU AUCTeX ___ auctex-commit mailing list auctex-com...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-commit
[AUCTeX-commit] GNU AUCTeX branch, master, updated. 1d72a5ceec85f507a252ccc41f9a5abda966e110
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via 1d72a5ceec85f507a252ccc41f9a5abda966e110 (commit) via 89798122cb96d78a8b56178d46a0fb40ba85342a (commit) via 7565d7c8d9e8bb78bdd3635824979a7bb78b2e07 (commit) via 68247ff4680eabcbad11853447880f5a77735112 (commit) from 8cd9db38c6eba5066dd02cf9166e51160f93abe9 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit 1d72a5ceec85f507a252ccc41f9a5abda966e110 Author: Ikumi Keita Date: Sun Jan 5 21:15:02 2020 +0900 Remove compatibility code for older emacsen According to NEWS.22, `font-lock-extra-managed-props' is available in all supported emacsen. * font-latex.el (font-latex-script,font-latex-script-char): Remove `boundp' test for `font-lock-extra-managed-props'. (font-latex-unfontify-region): Don't play with `invisible' text property because it is now handled by font-lock thanks to `font-lock-extra-managed-props'. commit 89798122cb96d78a8b56178d46a0fb40ba85342a Author: Ikumi Keita Date: Sun Jan 5 20:03:11 2020 +0900 Don't use obsolete variable According to NEWS.20, `font-lock' no longer supports `font-lock-comment-start-regexp'. * font-latex.el (defvar): Remove `defvar' for `font-lock-comment-start-regexp'. (font-latex-setup): Don't include `font-lock-comment-start-regexp' in `font-lock-defaults'. commit 7565d7c8d9e8bb78bdd3635824979a7bb78b2e07 Author: Ikumi Keita Date: Sun Jan 5 19:44:13 2020 +0900 Remove compatibility code for older emacsen Since `font-lock-multiline' is available in all supported emacsen, we no longer need `font-latex-multiline'. * font-latex.el (font-latex-setup): Remove `boundp' check. (font-latex-unfontify-region): Delete unnecessary code. (font-lock-after-change-function): Delete obsolete advice. (font-latex-put-multiline-property-maybe): Remove. (font-latex-match-command-with-arguments): (font-latex-match-command-in-braces): (font-latex-match-math-env): (font-latex-match-math-envII): (font-latex-match-quotation): Remove call on `font-latex-put-multiline-property-maybe'. commit 68247ff4680eabcbad11853447880f5a77735112 Author: Ikumi Keita Date: Sun Jan 5 19:33:33 2020 +0900 Delete obsolete comment * font-latex.el: Delete obsolete comment. `font-latex-do-multi-line' was removed in 2007. --- Summary of changes: font-latex.el | 97 +-- 1 file changed, 14 insertions(+), 83 deletions(-) hooks/post-receive -- GNU AUCTeX ___ auctex-commit mailing list auctex-com...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-commit
[AUCTeX-diffs] GNU AUCTeX branch, master, updated. 0d3393d0b9c97bb12dafd7b52c1dac2fae86765c
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via 0d3393d0b9c97bb12dafd7b52c1dac2fae86765c (commit) from 0dee720196bb24fcaed46d92ed9383530d53b0e1 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit 0d3393d0b9c97bb12dafd7b52c1dac2fae86765c Author: Ikumi Keita Date: Sun Apr 5 19:33:00 2020 +0900 * preview.el.in (preview-gs-open): Use wrapper function. diff --git a/preview.el.in b/preview.el.in index 89b767a..8094d85 100644 --- a/preview.el.in +++ b/preview.el.in @@ -787,7 +787,7 @@ stopped{handleerror quit}if \ ;; heuristic here. (with-current-buffer TeX-command-buffer (and TeX-PDF-mode - (not TeX-PDF-from-DVI))) + (not (TeX-PDF-from-DVI (preview-gs-queue-empty) (preview-parse-messages (or setup #'preview-gs-dvips-process-setup --- Summary of changes: preview.el.in | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) hooks/post-receive -- GNU AUCTeX ___ auctex-diffs mailing list auctex-di...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-diffs
[AUCTeX-commit] GNU AUCTeX branch, master, updated. 0d3393d0b9c97bb12dafd7b52c1dac2fae86765c
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via 0d3393d0b9c97bb12dafd7b52c1dac2fae86765c (commit) from 0dee720196bb24fcaed46d92ed9383530d53b0e1 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit 0d3393d0b9c97bb12dafd7b52c1dac2fae86765c Author: Ikumi Keita Date: Sun Apr 5 19:33:00 2020 +0900 * preview.el.in (preview-gs-open): Use wrapper function. --- Summary of changes: preview.el.in | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) hooks/post-receive -- GNU AUCTeX ___ auctex-commit mailing list auctex-com...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-commit
[AUCTeX-diffs] GNU AUCTeX branch, master, updated. c31faf7a78d6b4a12ad24c0c2d8c2e2dfedfd53c
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via c31faf7a78d6b4a12ad24c0c2d8c2e2dfedfd53c (commit) from 6e99536f4113bb1c52cb2f237f4c3ff9e05e5bf8 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit c31faf7a78d6b4a12ad24c0c2d8c2e2dfedfd53c Author: Ikumi Keita Date: Sun May 17 18:01:58 2020 +0900 ; Cosmetic change * font-latex.el (font-latex-extend-region-backwards-command-with-args): Make docsring coherent with similar functions. (font-latex-set-syntactic-keywords): (font-latex-setup): Replace backquotes with normal quotes where possible. diff --git a/font-latex.el b/font-latex.el index 9e11384..6210967 100644 --- a/font-latex.el +++ b/font-latex.el @@ -1052,7 +1052,7 @@ have changed." (setq font-latex-doctex-syntactic-keywords (append font-latex-syntactic-keywords ;; For docTeX comment-in-doc. - `(("\\(\\^\\)\\^A" (1 (font-latex-doctex-^^A))) + '(("\\(\\^\\)\\^A" (1 (font-latex-doctex-^^A))) ;;; Syntactic fontification @@ -1267,7 +1267,7 @@ triggers Font Lock to recognize the change." `((font-latex-keywords font-latex-keywords-1 font-latex-keywords-2) nil nil ,font-latex-syntax-alist nil)) (variables -`((font-lock-mark-block-function . mark-paragraph) +'((font-lock-mark-block-function . mark-paragraph) (font-lock-unfontify-region-function . font-latex-unfontify-region) (font-lock-extend-region-functions @@ -1580,7 +1580,7 @@ Returns nil if none of KEYWORDS is found." (defvar font-lock-end) (defun font-latex-extend-region-backwards-command-with-args () - "Return position to extend region backwards for commands with args." + "Extend region backwards for commands with args." (save-excursion (goto-char font-lock-end) (catch 'extend --- Summary of changes: font-latex.el | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) hooks/post-receive -- GNU AUCTeX ___ auctex-diffs mailing list auctex-di...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-diffs
[AUCTeX-commit] GNU AUCTeX branch, master, updated. c31faf7a78d6b4a12ad24c0c2d8c2e2dfedfd53c
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via c31faf7a78d6b4a12ad24c0c2d8c2e2dfedfd53c (commit) from 6e99536f4113bb1c52cb2f237f4c3ff9e05e5bf8 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit c31faf7a78d6b4a12ad24c0c2d8c2e2dfedfd53c Author: Ikumi Keita Date: Sun May 17 18:01:58 2020 +0900 ; Cosmetic change * font-latex.el (font-latex-extend-region-backwards-command-with-args): Make docsring coherent with similar functions. (font-latex-set-syntactic-keywords): (font-latex-setup): Replace backquotes with normal quotes where possible. --- Summary of changes: font-latex.el | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) hooks/post-receive -- GNU AUCTeX ___ auctex-commit mailing list auctex-com...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-commit
[AUCTeX-diffs] GNU AUCTeX branch, master, updated. 324422673f512569b7b8310ebfaa4152eb3438ba
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via 324422673f512569b7b8310ebfaa4152eb3438ba (commit) from a12c790ff9cc739bbf74eb4ba69ad655978a6080 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit 324422673f512569b7b8310ebfaa4152eb3438ba Author: Ikumi Keita Date: Sat May 9 20:31:26 2020 +0900 Update all mode lines * tex-buf.el (TeX-command-sentinel): Supply argument t for `force-mode-line-update' and discard useless `with-current-buffer'. * preview.el.in: Replace all `(set-buffer-modified-p (buffer-modified-p))' with `(force-mode-line-update)' and remove `sit-for' calls accompanying them. diff --git a/preview.el.in b/preview.el.in index 8094d85..6163db1 100644 --- a/preview.el.in +++ b/preview.el.in @@ -753,7 +753,7 @@ Gets the usual PROCESS and STRING parameters, see (setq mode-name "Preview-Ghostscript") (push process compilation-in-progress) (TeX-command-mode-line process) - (set-buffer-modified-p (buffer-modified-p)) + (force-mode-line-update) process))) (defun preview-gs-open ( setup) @@ -3189,8 +3189,7 @@ and `preview-colors' are set as given." (sit-for 0) process) (setq mode-line-process ": run") - (set-buffer-modified-p (buffer-modified-p)) - (sit-for 0) ; redisplay + (force-mode-line-update) (call-process TeX-shell nil (current-buffer) nil TeX-shell-command-option command @@ -3233,8 +3232,7 @@ If FAST is set, do a fast conversion." (sit-for 0) process) (setq mode-line-process ": run") - (set-buffer-modified-p (buffer-modified-p)) - (sit-for 0) ; redisplay + (force-mode-line-update) (call-process TeX-shell nil (current-buffer) nil TeX-shell-command-option command @@ -3277,8 +3275,7 @@ If FAST is set, do a fast conversion." (sit-for 0) process) (setq mode-line-process ": run") - (set-buffer-modified-p (buffer-modified-p)) - (sit-for 0) ; redisplay + (force-mode-line-update) (call-process TeX-shell nil (current-buffer) nil TeX-shell-command-option command diff --git a/tex-buf.el b/tex-buf.el index 7a4e0da..ff6be7e 100644 --- a/tex-buf.el +++ b/tex-buf.el @@ -1482,11 +1482,8 @@ reasons. Use `TeX-run-function' instead." (delete-process process)) ;; Force mode line redisplay soon - ;; Do this in the command buffer so that "Next Error" item - ;; will appear in the menu bar just after compilation. - ;; (bug#38058) - (with-current-buffer TeX-command-buffer -(force-mode-line-update) + ;; Do this in all buffers (bug#38058 and bug#40965) + (force-mode-line-update t (setq compilation-in-progress (delq process compilation-in-progress))) --- Summary of changes: preview.el.in | 11 --- tex-buf.el| 7 ++- 2 files changed, 6 insertions(+), 12 deletions(-) hooks/post-receive -- GNU AUCTeX ___ auctex-diffs mailing list auctex-di...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-diffs
[AUCTeX-commit] GNU AUCTeX branch, master, updated. 324422673f512569b7b8310ebfaa4152eb3438ba
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via 324422673f512569b7b8310ebfaa4152eb3438ba (commit) from a12c790ff9cc739bbf74eb4ba69ad655978a6080 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit 324422673f512569b7b8310ebfaa4152eb3438ba Author: Ikumi Keita Date: Sat May 9 20:31:26 2020 +0900 Update all mode lines * tex-buf.el (TeX-command-sentinel): Supply argument t for `force-mode-line-update' and discard useless `with-current-buffer'. * preview.el.in: Replace all `(set-buffer-modified-p (buffer-modified-p))' with `(force-mode-line-update)' and remove `sit-for' calls accompanying them. --- Summary of changes: preview.el.in | 11 --- tex-buf.el| 7 ++- 2 files changed, 6 insertions(+), 12 deletions(-) hooks/post-receive -- GNU AUCTeX ___ auctex-commit mailing list auctex-com...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-commit
[AUCTeX-commit] GNU AUCTeX branch, master, updated. edc7e0ecc050803618e65d99b10957bb1ad383b3
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via edc7e0ecc050803618e65d99b10957bb1ad383b3 (commit) from 324422673f512569b7b8310ebfaa4152eb3438ba (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit edc7e0ecc050803618e65d99b10957bb1ad383b3 Author: Ikumi Keita Date: Sat May 9 20:55:08 2020 +0900 ; * preview.el.in (): Update copyright year. --- Summary of changes: preview.el.in | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) hooks/post-receive -- GNU AUCTeX ___ auctex-commit mailing list auctex-com...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-commit
[AUCTeX-diffs] GNU AUCTeX branch, master, updated. edc7e0ecc050803618e65d99b10957bb1ad383b3
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via edc7e0ecc050803618e65d99b10957bb1ad383b3 (commit) from 324422673f512569b7b8310ebfaa4152eb3438ba (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit edc7e0ecc050803618e65d99b10957bb1ad383b3 Author: Ikumi Keita Date: Sat May 9 20:55:08 2020 +0900 ; * preview.el.in (): Update copyright year. diff --git a/preview.el.in b/preview.el.in index 6163db1..1b3ed93 100644 --- a/preview.el.in +++ b/preview.el.in @@ -1,7 +1,7 @@ ;;; preview.el --- embed preview LaTeX images in source buffer ;; Copyright (C) 2001-2006, 2010-2015, -;; 2017-2019 Free Software Foundation, Inc. +;; 2017-2020 Free Software Foundation, Inc. ;; Author: David Kastrup ;; Keywords: tex, wp, convenience --- Summary of changes: preview.el.in | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) hooks/post-receive -- GNU AUCTeX ___ auctex-diffs mailing list auctex-di...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-diffs
Re: Onwards to lexical binding (attempt 1)
>>>>> Tassilo Horn writes: >> Or it can just be >> (or TeX-exit-mark >> (make-marker)) >> for simplicity. > Right, just go ahead. ;-) Done. Bye, Ikumi Keita
[AUCTeX-commit] GNU AUCTeX branch, lexical-binding-attempt-1, updated. caf52d762545178268f5448c464e58cffaf5d5d2
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, lexical-binding-attempt-1 has been updated via caf52d762545178268f5448c464e58cffaf5d5d2 (commit) from 6fcbaf5ea64b76553ad29437d969c93a2a006cd9 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit caf52d762545178268f5448c464e58cffaf5d5d2 Author: Ikumi Keita Date: Sun Sep 6 00:18:33 2020 +0900 Arrange test condition * latex.el (LaTeX-env-args): Just test nil or not since `TeX-exit-mark' is now always bound. --- Summary of changes: latex.el | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) hooks/post-receive -- GNU AUCTeX ___ auctex-commit mailing list auctex-com...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-commit
[AUCTeX-diffs] GNU AUCTeX branch, lexical-binding-attempt-1, updated. caf52d762545178268f5448c464e58cffaf5d5d2
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, lexical-binding-attempt-1 has been updated via caf52d762545178268f5448c464e58cffaf5d5d2 (commit) from 6fcbaf5ea64b76553ad29437d969c93a2a006cd9 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit caf52d762545178268f5448c464e58cffaf5d5d2 Author: Ikumi Keita Date: Sun Sep 6 00:18:33 2020 +0900 Arrange test condition * latex.el (LaTeX-env-args): Just test nil or not since `TeX-exit-mark' is now always bound. diff --git a/latex.el b/latex.el index 2dc00ba..f74c76d 100644 --- a/latex.el +++ b/latex.el @@ -1316,9 +1316,8 @@ Just like array and tabular." (save-excursion (LaTeX-find-matching-begin) (end-of-line) -(let ((TeX-exit-mark (if (boundp 'TeX-exit-mark) -TeX-exit-mark - (make-marker +(let ((TeX-exit-mark (or TeX-exit-mark +(make-marker (TeX-parse-arguments args (defun LaTeX-env-label-as-keyval (_optional keyword keyvals environment) --- Summary of changes: latex.el | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) hooks/post-receive -- GNU AUCTeX ___ auctex-diffs mailing list auctex-di...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-diffs
Re: Onwards to lexical binding (attempt 1)
>>>>> Ikumi Keita writes: > By the way, I think this part should be improved: > -- > modified latex.el > @@ -1316,9 +1316,9 @@ Just like array and tabular." >(save-excursion > (LaTeX-find-matching-begin) > (end-of-line) > -(let ((exit-mark (if (boundp 'exit-mark) > - exit-mark > -(make-marker > +(let ((TeX-exit-mark (if (boundp 'TeX-exit-mark) > + TeX-exit-mark > +(make-marker >(TeX-parse-arguments args > -- > Shouldn't this boundp test be markerp because TeX-exit-mark is now bound > always? Or it can just be (or TeX-exit-mark (make-marker)) for simplicity. Bye, Ikumi Keita
Re: Onwards to lexical binding (attempt 1)
Hi Tassilo, >>>>> Tassilo Horn writes: > I just did so (-> TeX-exit-mark), and also with last-optional-rejected > (-> TeX-last-optional-rejected). Inserting a parbox works again. :-) Now it works for me, too, thanks! By the way, I think this part should be improved: -- modified latex.el @@ -1316,9 +1316,9 @@ Just like array and tabular." (save-excursion (LaTeX-find-matching-begin) (end-of-line) -(let ((exit-mark (if (boundp 'exit-mark) -exit-mark - (make-marker +(let ((TeX-exit-mark (if (boundp 'TeX-exit-mark) +TeX-exit-mark + (make-marker (TeX-parse-arguments args -- Shouldn't this boundp test be markerp because TeX-exit-mark is now bound always? Regards, Ikumi Keita
Re: Onwards to lexical binding (attempt 1)
Hi Tassilo, >>>>> Tassilo Horn writes: > I've done that now, too. I still can run LaTeX and call the Evince > viewer (TeX-evince-sync-view and TeX-pdf-tools-sync-view don't rely on > `file' anymore but use `TeX-active-master' as suggested by Al Haji-Ali), > so it seems I haven't broken everything [but possibly something I didn't > test]... I tried it and got the following error when I did C-c C-m parbox RET RET: Debugger entered--Lisp error: (void-variable exit-mark) TeX-argument-insert("" nil) TeX-arg-string(nil "Width") TeX-parse-argument(nil "Width") TeX-parse-arguments(([TeX-arg-tb] ["Height"] [TeX-arg-tb "Inner position"] "Width" t)) TeX-parse-macro("parbox" ([TeX-arg-tb] ["Height"] [TeX-arg-tb "Inner position"] "Width" t)) TeX-insert-macro("parbox") funcall-interactively(TeX-insert-macro "parbox") call-interactively(TeX-insert-macro nil nil) command-execute(TeX-insert-macro) We still have to deal with the variable `exit-mark'. Regards, Ikumi Keita
Re: Onwards to lexical binding (attempt 1)
>>>>> Ikumi Keita writes: > However, it seems that the first run of preview-latex on any region > fails. It must be my conceptual patch that is guilty for that. I haven't > looked into the actual codes yet, but my impression is that some kind of > initialization is required to fix this... Understood. Preview-latex (and presumably some other parts of AUCTeX) calls TeX-command-expand directly, which bypasses setting TeX-current-process-region-p. Thus TeX-active-master doesn't work as expected. >>> I think that the second argument of TeX-command-expand is no longer >>> necessary, thanks to the fine utility TeX-active-master. So this thought of mine wasn't correct. How about the attached patch? It reverts TeX--master-or-region-file-with-extra-quotes to the former form and uses the second argument of TeX-command-expand again. Regards, Ikumi Keita diff --git a/tex-buf.el b/tex-buf.el index 975c6fc9..49b53c81 100644 --- a/tex-buf.el +++ b/tex-buf.el @@ -551,15 +551,6 @@ without further expansion." "%")) (or list (TeX-expand-list))) pat (regexp-opt (mapcar #'car list))) -;; `TeX-command-expand' is called with `file-fn' argument being one -;; of `TeX-master-file', `TeX-region-file' and -;; `TeX-active-master'. The return value of these functions -;; sometimes needs suitable "decorations" for an argument for -;; underlying shell or latex executable, or both, when the -;; relavant file-fn name involves some special characters such as -;; space and multibyte characters. Hence embed that function in a -;; template prepared for that purpose. -(setq file-fn #'TeX--master-or-region-file-with-extra-quotes) (while (setq TeX-expand-pos (string-match pat TeX-expand-command TeX-expand-pos)) (setq string (match-string 0 TeX-expand-command) entry (assoc string list) @@ -567,9 +558,9 @@ without further expansion." arguments (cdr (cdr entry)) ;Remaining elements string (save-match-data (cond - ((memq expansion (list 'TeX-active-master - #'TeX-active-master)) - (let ((res (apply file-fn arguments))) + ((eq expansion #'TeX-active-master) + (let ((res (apply #'TeX--master-or-region-file-with-extra-quotes + file-fn arguments))) ;; Advance past the file name in order to ;; prevent expanding any substring of it. (setq TeX-expand-pos @@ -587,20 +578,29 @@ without further expansion." TeX-expand-command)) (defun TeX--master-or-region-file-with-extra-quotes -( extension nondirectory ask extra) - "Return the current master or region file name with quote for shell. -I.e. it encloses the file name with space within quotes `\"' -first when \" \\input\" is supplemented (indicated by dynamically -binded variable `TeX-command-text' having string value.) It also -encloses the file name within \\detokenize{} when the following -three conditions are met: - 1. compiling with standard (pdf)LaTeX or upLaTeX - 2. \" \\input\" is supplemented - 3. EXTRA is non-nil (default when expanding \"%T\") - -Helper function of `TeX-command-expand'." +(file-fn extension nondirectory ask extra) + "Return file name with quote for shell. +Helper function of `TeX-command-expand'. + +This is a kind of template. How to use: +Fix, by `apply-partially', the first argument FILE-FN as one of +the three functions `TeX-master-file', `TeX-region-file' or +`TeX-active-master'. Then the result is just a wrapper for that +function suitable in `TeX-command-expand'. + +As a wrapper described above, it passes EXTENSION, NONDIRECTORY +and ASK to the \"bare\" function as-is, and arranges the returned +file name for use with command shell. I.e. it encloses the file +name with space within quotes `\"' first when \" \\input\" is +supplemented (indicated by dynamically binded variable +`TeX-command-text' having string value.) It also encloses the +file name within \\detokenize{} when the following three +conditions are met: +1. compiling with standard (pdf)LaTeX or upLaTeX +2. \" \\input\" is supplemented +3. EXTRA is non-nil (default when expanding \"%T\")" (shell-quote-argument - (let* ((raw (funcall #'TeX-active-master extension nondirectory ask)) + (let* ((raw (funcall file-fn extension nondirectory ask)) ;; String `TeX-command-text' means that the file name is ;; given through \input command. (quote-for-space (if (and (stringp TeX-command-text)
Re: Onwards to lexical binding (attempt 1)
Hi Tassilo, >>>>> Tassilo Horn writes: > Yeah, but preview-latex can bind that itself. I mean, it has separate > commands for preview document and preview region and then passes either > TeX-master-file or TeX-region-file. So if it binds > TeX-current-process-region-p itself and TeX-active-master is used > implicitly, it should just work. OK, I'm fine with either way, provided that other parts of AUCTeX which needs binding TeX-current-process-region-p, if any, are fixed as well. > Here's the patch which was empty yesterday where I actually do that. Thanks. I'll test it later, probably tommorow. Regards, Ikumi Keita
Re: Onwards to lexical binding (attempt 1)
Hi Tassilo, >>>>> Tassilo Horn writes: >>> Here's the patch which was empty yesterday where I actually do that. >> >> Thanks. I'll test it later, probably tommorow. > Great, thanks! It doesn't work for region preview. After Setting region on circ.tex and C-c C-p C-r, I get error and in process buffer I find Running `Preview-PDF2DSC' with ``pdf2dsc circ.pdf circ.prv/tmpSYCDuG/preview.dsc'' . Thus preview-latex provides "master" file name "circ.pdf" rather than "_region_.pdf" for pdf2dsc for this case. I think `preview-region' should bind TeX-current-process-region-p permanently by `setq', not `let', so that TeX-active-master returns intended file name even in process sentinels. Regards, Ikumi Keita
Re: Onwards to lexical binding (attempt 1)
Hi Tassilo, >>>>> Tassilo Horn writes: >> I think `preview-region' should bind TeX-current-process-region-p >> permanently by `setq', not `let', so that TeX-active-master returns >> intended file name even in process sentinels. > Indeed, below is a patch which does that. It works for me as far as I > have tested. If more testing doesn't reveal any blocker, I'm slightly > leaning towards doing that change. IMHO, it makes the code a bit > easier to understand. Oops, by similar reason, `preview-document' should bind TeX-current-process-region-p to nil. Otherwise C-c C-p C-d just after region preview doesn't work. > I noticed some other (unrelated) strange things: > 1. My preview.el was terribly outdated. It seems it's not re-generated > from preview.el.in on every "make" invocation. I needed to delete it to > be re-generated... Indeed. I guess that Makefile is written in such a way... > 2. The first preview invocation errors out with: > [1{/var/lib/texmf/fonts/map/pdftex/updmap/pdftex.map}] !name(2.tex) > !offset(47) ) > (see the transcript file for additional > information) > Output written on _region_.pdf (1 page, 23275 bytes). > SyncTeX written on _region_.synctex.gz. > Transcript written on _region_.log. > TeX Output exited as expected with code 1 at Fri Sep 11 09:28:23 > Running `Preview-DviPS' with ``dvips -Pwww _region_.dvi -o > _region_.prv/tmpLKkFSa/preview.ps'' > Parser: End of Preview snippet 1 unexpected > Preview-DviPS killed at Fri Sep 11 09:28:23 > LaTeX: End of Preview snippet 1 unexpected > Trying again, it works and previews are displayed. That also happens on > master. Hmm, strange. While pdflatex produces PDF file, preview-latex invokes dvips. It doesn't occur for me. Perhaps customization on preview-latex related options has some inconsistency. Regards, Ikumi Keita
Re: Onwards to lexical binding (attempt 1)
Hi Tassilo, >>>>> Tassilo Horn writes: > Right. preview-generate-preview used to do that, but that seems to be > to late. So in the new version, I setq TeX-current-process-region-p > before the calls to that and removed its region-p argument. New patch > at the end of the mail. Thanks, now it functions well. It seems ready to be committed in the lexical-binding-attempt-1 branch. Regards, Ikumi Keita
[AUCTeX-commit] GNU AUCTeX branch, lexical-binding-attempt-1, updated. 3268e848c11d57cca9c6d763573769a816663146
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, lexical-binding-attempt-1 has been updated via 3268e848c11d57cca9c6d763573769a816663146 (commit) from 801b5d466c4badc037aca073b338c930828004b3 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit 3268e848c11d57cca9c6d763573769a816663146 Author: Ikumi Keita Date: Sat Sep 12 01:44:18 2020 +0900 ; Trivial cleanups * tex-buf.el (TeX-command-expand): Omit one `let'. (TeX--master-or-region-file-with-extra-quotes): Omit spurious `funcall'. Add descriptions about arguments in doc-string. * tex.el (declare-function): Fix arglist for updated `TeX-command-expand'. --- Summary of changes: tex-buf.el | 63 +++--- tex.el | 2 +- 2 files changed, 32 insertions(+), 33 deletions(-) hooks/post-receive -- GNU AUCTeX ___ auctex-commit mailing list auctex-com...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-commit
[AUCTeX-diffs] GNU AUCTeX branch, lexical-binding-attempt-1, updated. 3268e848c11d57cca9c6d763573769a816663146
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, lexical-binding-attempt-1 has been updated via 3268e848c11d57cca9c6d763573769a816663146 (commit) from 801b5d466c4badc037aca073b338c930828004b3 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit 3268e848c11d57cca9c6d763573769a816663146 Author: Ikumi Keita Date: Sat Sep 12 01:44:18 2020 +0900 ; Trivial cleanups * tex-buf.el (TeX-command-expand): Omit one `let'. (TeX--master-or-region-file-with-extra-quotes): Omit spurious `funcall'. Add descriptions about arguments in doc-string. * tex.el (declare-function): Fix arglist for updated `TeX-command-expand'. diff --git a/tex-buf.el b/tex-buf.el index b9d9085..134bf36 100644 --- a/tex-buf.el +++ b/tex-buf.el @@ -550,48 +550,47 @@ without further expansion." "%")) (or list (TeX-expand-list))) pat (regexp-opt (mapcar #'car list))) -(let ((file-fn #'TeX--master-or-region-file-with-extra-quotes)) - (while (setq TeX-expand-pos (string-match pat TeX-expand-command TeX-expand-pos)) -(setq string (match-string 0 TeX-expand-command) - entry (assoc string list) - expansion (car (cdr entry)) ;Second element - arguments (cdr (cdr entry)) ;Remaining elements - string (save-match-data - (cond -((memq expansion (list 'TeX-active-master - #'TeX-active-master)) - (let ((res (apply file-fn arguments))) - ;; Advance past the file name in order to - ;; prevent expanding any substring of it. - (setq TeX-expand-pos - (+ TeX-expand-pos (length res))) - res)) -((functionp expansion) - (apply expansion arguments)) - ((boundp expansion) - (apply (symbol-value expansion) arguments)) - (t -(error "Nonexpansion %s" expansion) -(if (stringp string) - (setq TeX-expand-command - (replace-match string t t TeX-expand-command - TeX-expand-command))) +(while (setq TeX-expand-pos (string-match pat TeX-expand-command TeX-expand-pos)) + (setq string (match-string 0 TeX-expand-command) + entry (assoc string list) + expansion (car (cdr entry)) ;Second element + arguments (cdr (cdr entry)) ;Remaining elements + string (save-match-data +(cond + ((eq expansion #'TeX-active-master) + (let ((res (apply #'TeX--master-or-region-file-with-extra-quotes arguments))) + ;; Advance past the file name in order to + ;; prevent expanding any substring of it. + (setq TeX-expand-pos + (+ TeX-expand-pos (length res))) + res)) + ((functionp expansion) + (apply expansion arguments)) + ((boundp expansion) + (apply (symbol-value expansion) arguments)) + (t + (error "Nonexpansion %s" expansion) + (if (stringp string) + (setq TeX-expand-command + (replace-match string t t TeX-expand-command +TeX-expand-command)) (defun TeX--master-or-region-file-with-extra-quotes ( extension nondirectory ask extra) "Return the current master or region file name with quote for shell. -I.e. it encloses the file name with space within quotes `\"' -first when \" \\input\" is supplemented (indicated by dynamically -binded variable `TeX-command-text' having string value.) It also -encloses the file name within \\detokenize{} when the following -three conditions are met: +Pass arguments EXTENSION NONDIRECTORY ASK to `TeX-active-master'. +If the returned file name contains space, enclose it within +quotes `\"' when \" \\input\" is supplemented (indicated by +dynamically bound variable `TeX-command-text' having string +value.) Also enclose the file name within \\detokenize{} when +the following three conditions are met: 1. compiling with standard (pdf)LaTeX or upLaTeX 2. \" \\input\" is supplemented 3. EXTRA is non-nil (default when expanding \"%T\") Helper func
Re: Onwards to lexical binding (attempt 1)
Hi Tassilo, >>>>> Tassilo Horn writes: > I worked upon your patch and committed that. Should work again. > TeX-file-fn is now gone and one should simply use `TeX-active-master' in > TeX-expand-list{,-builtin} where formally `file' (and then > `TeX-file-fn') has been used. Works fine for normal typeset. However, it seems that the first run of preview-latex on any region fails. It must be my conceptual patch that is guilty for that. I haven't looked into the actual codes yet, but my impression is that some kind of initialization is required to fix this... >> I think that the second argument of TeX-command-expand is no longer >> necessary, thanks to the fine utility TeX-active-master. > Indeed. Here is an untested patch which does that (all tests still > pass). The attachment is empty:-) > Since it also affects preview-latex which I did not test, I haven't > committed it yet. If it works, preview.el.in could also be changed to > that preview-gs-file is no cons of (TeX-active-master . file) where > the car is unused anymore. Agreed. Regards, Ikumi Keita
Re: Onwards to lexical binding (attempt 1)
Hi Tassilo, I noticed a regression in TeX-command-expand. When the file name contains spaces, "%o" is expanded to raw file name without suitable quotes for shell. How about the attached patch? It also make "%o" expansion skip the obtained file name. Evaluating (TeX-command-expand "%o") no longer falls into infinite loop even when the file name itself contains "%o". Regards, Ikumi Keita 0001-Make-o-compatible-again-with-file-name-with-spaces.patch.gz Description: fix "%o" expansion
[AUCTeX-diffs] GNU AUCTeX branch, lexical-binding-attempt-1, updated. d91a08d905fa80b5ff04deb77672dc9d38a2fc70
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, lexical-binding-attempt-1 has been updated via d91a08d905fa80b5ff04deb77672dc9d38a2fc70 (commit) from 3268e848c11d57cca9c6d763573769a816663146 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit d91a08d905fa80b5ff04deb77672dc9d38a2fc70 Author: Ikumi Keita Date: Sat Sep 12 14:52:35 2020 +0900 Make "%o" compatible again with file name with spaces * tex-buf.el (TeX-active-master-with-quotes): Rename from `TeX--master-or-region-file-with-extra-quotes' and make public. Incorporate adjust of `TeX-expand-pos' from `TeX-command-expand'. (TeX-command-expand): Simplify. * tex.el (TeX-expand-list-builtin): Use renamed function instead of `TeX-active-master'. * tests/tex/command-expansion.el (TeX-command-expand-skip-file-name): Include "%o" in test. (TeX-command-expand-file-name-with-spaces): New test. (TeX-command-detokenize): Add case where `TeX-master' has space. diff --git a/tests/tex/command-expansion.el b/tests/tex/command-expansion.el index bb5c7d4..3b2779d 100644 --- a/tests/tex/command-expansion.el +++ b/tests/tex/command-expansion.el @@ -88,13 +88,20 @@ ;; Skip on w32 because the quoting style of `shell-quote-argument' ;; is different. (skip-unless (not (eq system-type 'windows-nt))) - (should (string= - (let ((major-mode 'latex-mode) -(TeX-engine 'default) -(TeX-master "/tmp/abc") -(TeX-command-extra-options " \"\\foo\"")) -(TeX-command-expand "%`%(extraopts)%' %T")) - " \"\\foo\" \"\\input\" detokenize\\{\\ abc.tex\\ \\}"))) + (let ((major-mode 'latex-mode) + (TeX-engine 'default) + (TeX-command-extra-options " \"\\foo\"") + TeX-master) + +(setq TeX-master "/tmp/abc") +(should (string= +(TeX-command-expand "%`%(extraopts)%' %T") +" \"\\foo\" \"\\input\" detokenize\\{\\ abc.tex\\ \\}")) + +(setq TeX-master "/tmp/abc def") +(should (string= +(TeX-command-expand "%`%(extraopts)%' %T") +" \"\\foo\" \"\\input\" detokenize\\{\\ \\\"abc\\ def.tex\\\"\\ \\}" (ert-deftest TeX-command-expand-skip-file-name () "Check whether file name is not subject to further expansion. @@ -105,6 +112,7 @@ See <https://lists.gnu.org/r/bug-auctex/2014-08/msg00012.html>." ;; is different. (skip-unless (not (eq system-type 'windows-nt))) (let ((TeX-master "abc-def") + (TeX-output-extension "pdf") (TeX-expand-list '(("-" (lambda () ":") (should (string= (TeX-command-expand "%s") @@ -121,12 +129,9 @@ See <https://lists.gnu.org/r/bug-auctex/2014-08/msg00012.html>." (should (string= (TeX-command-expand "%f") (TeX-master-file "ps" t))) -;; The expander of "%o" does not yet cater for this possible endless -;; loop. -;; (should (string= -;; (TeX-command-expand "%o") -;; (TeX-master-file "pdf" t))) -)) +(should (string= +(TeX-command-expand "%o") +(TeX-master-file "pdf" t) (ert-deftest TeX-command-expand-active-master () "Test whether `TeX-active-master' is valid argument for `TeX-command-expand'." @@ -144,4 +149,27 @@ See <https://lists.gnu.org/r/bug-auctex/2014-08/msg00012.html>." (TeX-command-expand "%s") TeX-region +(ert-deftest TeX-command-expand-file-name-with-spaces () + "Test whether file name with spaces is quoted correctly." + (let ((TeX-master "abc def") + (TeX-output-extension "pdf")) +(should (string= +(TeX-command-expand "%s") +(shell-quote-argument TeX-master))) +(should (string= +(TeX-command-expand "%t") +(shell-quote-argument (TeX-master-file "tex" t +(should (string= +(TeX-command-expand "%T") +(shell-quote-argument (TeX-master-file "tex" t +(should (string= +(TeX-command-expand "%d") +(shell-quote-argument (TeX-master-file "dvi" t +(should (string= +
[AUCTeX-commit] GNU AUCTeX branch, lexical-binding-attempt-1, updated. d91a08d905fa80b5ff04deb77672dc9d38a2fc70
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, lexical-binding-attempt-1 has been updated via d91a08d905fa80b5ff04deb77672dc9d38a2fc70 (commit) from 3268e848c11d57cca9c6d763573769a816663146 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit d91a08d905fa80b5ff04deb77672dc9d38a2fc70 Author: Ikumi Keita Date: Sat Sep 12 14:52:35 2020 +0900 Make "%o" compatible again with file name with spaces * tex-buf.el (TeX-active-master-with-quotes): Rename from `TeX--master-or-region-file-with-extra-quotes' and make public. Incorporate adjust of `TeX-expand-pos' from `TeX-command-expand'. (TeX-command-expand): Simplify. * tex.el (TeX-expand-list-builtin): Use renamed function instead of `TeX-active-master'. * tests/tex/command-expansion.el (TeX-command-expand-skip-file-name): Include "%o" in test. (TeX-command-expand-file-name-with-spaces): New test. (TeX-command-detokenize): Add case where `TeX-master' has space. --- Summary of changes: tests/tex/command-expansion.el | 54 + tex-buf.el | 68 ++ tex.el | 25 3 files changed, 88 insertions(+), 59 deletions(-) hooks/post-receive -- GNU AUCTeX ___ auctex-commit mailing list auctex-com...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-commit
Re: Onwards to lexical binding (attempt 1)
>>>>> Tassilo Horn writes: > Very good! I've also had the same idea just last night in order to get > that ugly 'TeX-active-master special case out of `TeX-expand-command'. > Please commit. Done.:-) Cheers, Ikumi Keita
[AUCTeX-diffs] GNU AUCTeX branch, lexical-binding-attempt-1, updated. 813e5478bb8d8ded55179fd927346f99d13f5ff7
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, lexical-binding-attempt-1 has been updated via 813e5478bb8d8ded55179fd927346f99d13f5ff7 (commit) from d91a08d905fa80b5ff04deb77672dc9d38a2fc70 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit 813e5478bb8d8ded55179fd927346f99d13f5ff7 Author: Ikumi Keita Date: Sat Sep 12 17:35:48 2020 +0900 Simplify nested `let's * tex-buf.el (TeX-active-master-with-quotes): Simplify nested `let's into singe one. diff --git a/tex-buf.el b/tex-buf.el index 9b04a07..bb5e275 100644 --- a/tex-buf.el +++ b/tex-buf.el @@ -585,30 +585,30 @@ infinite loop in `TeX-command-expand'. Helper function of `TeX-command-expand'. Use only within entries in `TeX-expand-list-builtin' and `TeX-expand-list'." - (let ((res -(shell-quote-argument - (let* ((raw (TeX-active-master extension nondirectory ask)) -;; String `TeX-command-text' means that the file name is -;; given through \input command. -(quote-for-space (if (and (stringp TeX-command-text) - (string-match " " raw)) - "\"" ""))) - (format -(if (and extra - (stringp TeX-command-text) - (memq major-mode '(latex-mode doctex-mode)) - (memq TeX-engine '(default uptex))) -;; Since TeXLive 2018, the default encoding for LaTeX -;; files has been changed to UTF-8 if used with -;; classic TeX or pdfTeX. I.e., -;; \usepackage[utf8]{inputenc} is enabled by default -;; in (pdf)latex. -;; c.f. LaTeX News issue 28 -;; Due to this change, \detokenize is required to -;; recognize non-ascii characters in the file name -;; when \input precedes. -"\\detokenize{ %s }" "%s") -(concat quote-for-space raw quote-for-space)) + (let* ((raw (TeX-active-master extension nondirectory ask)) +;; String `TeX-command-text' means that the file name is +;; given through \input command. +(quote-for-space (if (and (stringp TeX-command-text) + (string-match " " raw)) + "\"" "")) +(res + (shell-quote-argument + (format + (if (and extra +(stringp TeX-command-text) +(memq major-mode '(latex-mode doctex-mode)) +(memq TeX-engine '(default uptex))) + ;; Since TeXLive 2018, the default encoding for LaTeX + ;; files has been changed to UTF-8 if used with + ;; classic TeX or pdfTeX. I.e., + ;; \usepackage[utf8]{inputenc} is enabled by default + ;; in (pdf)latex. + ;; c.f. LaTeX News issue 28 + ;; Due to this change, \detokenize is required to + ;; recognize non-ascii characters in the file name + ;; when \input precedes. + "\\detokenize{ %s }" "%s") + (concat quote-for-space raw quote-for-space) ;; Advance past the file name in order to ;; prevent expanding any substring of it. (setq TeX-expand-pos --- Summary of changes: tex-buf.el | 48 1 file changed, 24 insertions(+), 24 deletions(-) hooks/post-receive -- GNU AUCTeX ___ auctex-diffs mailing list auctex-di...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-diffs
[AUCTeX-commit] GNU AUCTeX branch, lexical-binding-attempt-1, updated. 813e5478bb8d8ded55179fd927346f99d13f5ff7
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, lexical-binding-attempt-1 has been updated via 813e5478bb8d8ded55179fd927346f99d13f5ff7 (commit) from d91a08d905fa80b5ff04deb77672dc9d38a2fc70 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit 813e5478bb8d8ded55179fd927346f99d13f5ff7 Author: Ikumi Keita Date: Sat Sep 12 17:35:48 2020 +0900 Simplify nested `let's * tex-buf.el (TeX-active-master-with-quotes): Simplify nested `let's into singe one. --- Summary of changes: tex-buf.el | 48 1 file changed, 24 insertions(+), 24 deletions(-) hooks/post-receive -- GNU AUCTeX ___ auctex-commit mailing list auctex-com...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-commit
Re: Patching graphicx.el
Hi Arash, >>>>> Arash Esbati writes: > The way I see it is due to how `LaTeX-arg-includegraphics' inserts the > name of the graphic. This way, it is not possible to use > `LaTeX-arg-includegraphics' in other styles and the result is also odd. > I suppose the following patch to graphicx.el and AUCTeX-ify > `LaTeX-arg-includegraphics'. Any comment welcome. The attached patch somehow doesn't apply for my local repository, but I looks reasonable anyway. Regards, Ikumi Keita
[AUCTeX-commit] GNU AUCTeX branch, lexical-binding-attempt-1, updated. 860f46c45f4d105f24d4063ba6d83124970fdbea
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, lexical-binding-attempt-1 has been updated via 860f46c45f4d105f24d4063ba6d83124970fdbea (commit) from 284d1be0967665170073d7574ca5bab7c9f59025 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit 860f46c45f4d105f24d4063ba6d83124970fdbea Author: Ikumi Keita Date: Sun Oct 18 17:03:37 2020 +0900 ; * style/graphicx.el: Fix Ataka-san's name and email. --- Summary of changes: style/graphicx.el | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) hooks/post-receive -- GNU AUCTeX ___ auctex-commit mailing list auctex-com...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-commit
[AUCTeX-diffs] GNU AUCTeX branch, lexical-binding-attempt-1, updated. 860f46c45f4d105f24d4063ba6d83124970fdbea
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, lexical-binding-attempt-1 has been updated via 860f46c45f4d105f24d4063ba6d83124970fdbea (commit) from 284d1be0967665170073d7574ca5bab7c9f59025 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit 860f46c45f4d105f24d4063ba6d83124970fdbea Author: Ikumi Keita Date: Sun Oct 18 17:03:37 2020 +0900 ; * style/graphicx.el: Fix Ataka-san's name and email. diff --git a/style/graphicx.el b/style/graphicx.el index be918bd..cfcba21 100644 --- a/style/graphicx.el +++ b/style/graphicx.el @@ -28,7 +28,7 @@ ;; Acknowledgements ;; Dr. Thomas Baumann ;; David Kastrup -;; Masayuki Akata +;; Masayuki Ataka ;;; Code: --- Summary of changes: style/graphicx.el | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) hooks/post-receive -- GNU AUCTeX ___ auctex-diffs mailing list auctex-di...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-diffs
Re: Patching graphicx.el
Hi Arash, >>>>> Arash Esbati writes: > Thanks for trying and my apologies that the change doesn't apply, I > can't tell why. No problem. > I just pushed that change to lexical-binding-attempt-1 branch. Do want > to update your local repo and give it a roll? I tried it and comfirmed that it works well. Thanks! Best, Ikumi Keita
Re: Output filenames and the inconsistency of the `file` variable
Hi Al, >>>>> Al Haji-Ali writes: > To start with, I think the first change is to ensure that the variable > `file` (or the lexically-scoped alternative) is not used as a string. > Since this is an internal variable, I think user config is not an > issue. In my humble opinion, it's a bit dangerous to presume so. In general, a user can have an entry in TeX-view-program-list like ("XYZ" my-TeX-view-open-XYZ) where the second symbol is one's own elisp function. If that function assumes that the variable `file' has string value, your proposal would cause error. Thus I think the problem is more delicate than its appearance. How about using functionp test or stringp test on the value of `file' for the changes in TeX-pdf-tools-sync-view and TeX-evince-sync-view-1? That would be rather safe way although there still remains doubled identity, function name and string, of the variable `file'... Another possible idea is to stop relying on dynamic scope variable `file' and just to call TeX-active-master in TeX-pdf-tools-sync-view and TeX-evince-sync-view-1. My understanding is that TeX-active-master serves for such purposes for most cases. Regards, Ikumi Keita
Re: Output filenames and the inconsistency of the `file` variable
Hi Al, >>>>> Al Haji-Ali writes: > However, I noticed that in functions in `tex.el` like > `TeX-pdf-tools-sync-view` and `TeX-evince-sync-view-1` the filename of the > output file is constructed from the `file` variable along with > `(TeX-output-extension)` instead of calling the filename function (like > `TeX-master-file`) with `(TeX-output-extension)` as an argument. Is there a > reason for this? I think that is just a historical reason. > Would changing that to calling the appropriate filename function be > acceptable in my contribution? Probably yes, provided that your contribution is careful enough not to break compatibility with existing codes including users' customization. > Also, this might be related, I noticed that the variable `file` is used in > dynamic scope with two different meanings: sometimes as a function to get > the filename, other times as the filename itself (after being overwritten > on tex-buf.el:534). I am wondering if there is a reason for having the same > variable name for two different functionalities. I think this is another historical reason. > I think having two dynamically-scoped variables (something like `file` > and `file-name` for example) would be more useful and less confusing. > This would make it possible to call the correct filename function with > the output extension in the functions I mentioned above. Recently, there were some discussions in this list about moving to lexical scope: https://lists.gnu.org/archive/html/auctex-devel/2020-08/msg8.html So the situation with respect to this dynamically-scoped variables may be much different in future AUCTeX. Regards, Ikumi Keita
Re: Putting tex-math-face in tex group
Hi Arash, >>>>> Arash Esbati writes: > tex-math-face defined in tex-font.el doesn't specify a custom group. > Any objections if I apply this patch in order to pacify the compiler? I think that the patch itself is harmless. At the same time, I feel that we should overhaul tex-font.el if possible. The code of tex-font.el is basically the same with the font-lock-related contents of tex-mode.el (standard tex mode of emacs), and seems to be much outdated. So there are little points to have such own contents in tex-font.el. In my opinion, we can replace most of the contents of tex-font.el with just (require 'tex-mode) and leave only `tex-font-setup' with the slightest change to follow the recent font-lock setup in tex-mode.el. Regards, Ikumi Keita
[AUCTeX-commit] GNU AUCTeX branch, master, updated. 087b0300012d14cc86dec946d6c225cdb06460a1
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via 087b0300012d14cc86dec946d6c225cdb06460a1 (commit) from ef1f6708812606f66b5a562bb2be754412baa74f (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit 087b0300012d14cc86dec946d6c225cdb06460a1 Author: Ikumi Keita Date: Fri Aug 28 15:32:53 2020 +0900 Simplify mode line for emacs 27 * tex-buf.el: Don't add an entry in `minor-mode-alist' for `compilation-in-progress'. In emacs 27, compile.el adds a similar entry in `mode-line-modes'. --- Summary of changes: tex-buf.el | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) hooks/post-receive -- GNU AUCTeX ___ auctex-commit mailing list auctex-com...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-commit
[AUCTeX-diffs] GNU AUCTeX branch, master, updated. 087b0300012d14cc86dec946d6c225cdb06460a1
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via 087b0300012d14cc86dec946d6c225cdb06460a1 (commit) from ef1f6708812606f66b5a562bb2be754412baa74f (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit 087b0300012d14cc86dec946d6c225cdb06460a1 Author: Ikumi Keita Date: Fri Aug 28 15:32:53 2020 +0900 Simplify mode line for emacs 27 * tex-buf.el: Don't add an entry in `minor-mode-alist' for `compilation-in-progress'. In emacs 27, compile.el adds a similar entry in `mode-line-modes'. diff --git a/tex-buf.el b/tex-buf.el index ff6be7e..510722c 100644 --- a/tex-buf.el +++ b/tex-buf.el @@ -1883,9 +1883,11 @@ variable is nil." (defvar compilation-in-progress nil "List of compilation processes now running.") -(or (assq 'compilation-in-progress minor-mode-alist) -(setq minor-mode-alist (cons '(compilation-in-progress " Compiling") -minor-mode-alist))) +;; COMPATIBILITY for emacs < 27 +(if (< emacs-major-version 27) +(or (assq 'compilation-in-progress minor-mode-alist) + (setq minor-mode-alist (cons '(compilation-in-progress " Compiling") +minor-mode-alist (defun TeX-process-get-variable (name symbol default) "Return the value in the process buffer for NAME of SYMBOL. --- Summary of changes: tex-buf.el | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) hooks/post-receive -- GNU AUCTeX ___ auctex-diffs mailing list auctex-di...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-diffs
Re: tex.el compiler warnings
Hi Arash and Tassilo, >>>>> Tassilo Horn writes: > So please keep all warnings about references to free, unprefixed > variables. Arash, you recently committed some changes using `with-no-warnings' to suppress the warnings about free, unprefixed variables, didn't you? It seems that it should be discussed what to do with this policy and those commits. Regards, Ikumi Keita
Re: Putting tex-math-face in tex group
>>>>> Arash Esbati writes: > Indeed, the code in tex-font.el is really outdated. And that's the > reason I thought I just silence the compiler without touching it. Ah I see. Then I'm fine with your proposal. I agree that it isn't worth taking care of tex-font.el. Regards, Ikumi Keita
Re: Output filenames and the inconsistency of the `file` variable
Hi Al, Your proposition sounds very much like the one proposed in this list a while ago. See the thread beginning with https://lists.gnu.org/archive/html/auctex-devel/2020-06/msg00031.html . I'm cc'ing to Tohiko Looka, the original poster of the above thread. The thread seems to be dormant now though Colin and Tassilo expressed their interests in that idea. How about discussing among those looking forward to this idea, to refine and incubate full-fledged implementation ready to incorporate into AUCTeX? (I personally am not much inclined to that idea of using separate build directory, but cooperation between the folks with similar spirits in this list is of course welcome :-) Best regards, Ikumi Keita >>>>> Al Haji-Ali writes: > For the past 3 months, I have been using the `--output-directory` > option of tex compilation engines (like pdflatex) to save all output > and temporary files in an "auctex/build" sub-directory next to my tex > files. > I found this arrangement to be very convenient. For version control, I > simply gitignore the "auctex" directory (which also includes the > temporary `el` files in my setup) instead of having to ignore > extensions like pdf. Also, having multiple (independent) tex files in > the same directory did not clutter my source files with temporary > ones. > The way I implemented this is by: > 1. Passing the `--output-directory` option to pdflatex through > "TeX-command-extra-options" > 2. Advising TeX-master-file to return a path in the output directory > instead of the current directory whenever an output extension (or > those in `*-clean-*-suffixes`) is passed as an argument. > 3. This is not relevant to AUCTeX, but I also implemented two > functions that can be executed in a tex file: one to copy the output > file from the build directory to another one and another function to > attach the pdf (from the build directory) in an email. > This worked surprisingly well! However, because AUCTeX has these lines > of code where the path is constructed using a string concatenation > rather than calling `TeX-active-master`, this meant that I had to > patch some functions to ensure everything worked as intended > (including viewing the pdf and tex-syncing it). > I plan to propose the whole setup as a contribution to AUCTeX, but I > wanted to first make sure of the least disruptive way to add this > functionality natively. The required modification of TeX-master-file > is not disruptive, since it can be controlled with a flag. So the only > potentially disruptive changes are the ones I outlined in my previous > email.
Re: Output filenames and the inconsistency of the `file` variable
Hi Al, >>>>> Al Haji-Ali writes: > The documentation of `TeX-view-program-list` does not mention `file` > as a variable. So from this perspective, IMO it is justifiable to > normalize the treatment of this variable. I don't have strong objection about this perspective. > With that being said, I understand that this could break some user > configs that use undocumented variables. For historical reason (again), AUCTeX has been permissive for such abuse of dynamic scope variables in users' customization because it directs to use undeclared dynamic scope variables in, e.g., hooks in LaTeX-section-hook and TeX-arg-* functions used in TeX-parse-macro. So in general we have to anticipate that some users use undocumented variable `file' in their costom functions when we try to be very conservative. I admit it's unfortunate but I suppose such care wouldn't be dispensible. > Although I would argue that changing the name of the variable (as the > thread you linked earlier suggests) would also break these configs. Yes. That must be the reason why Tassilo is very cautious in the thread about carrying out such changes. When AUCTeX finally move to lexical scope, it should be clearly advertised to users that those dynamically-scoped variabales are no longer valid and AUCTeX should be careful so that the cost of rewriting such code to conform to lexical scoping would be reasonably low. >> How about using functionp test or stringp test on the value of `file' >> for the changes in TeX-pdf-tools-sync-view and TeX-evince-sync-view-1? >> That would be rather safe way although there still remains doubled >> identity, function name and string, of the variable `file'... >> > I am not sure how to do this using `functionp` and `stringp`. The > problem is the value of `file` is overwritten in `TeX-command` before > the program is called so in `TeX-pdf-tools-sync-view` and > `TeX-evince-sync-view-1` the function value of `file` is inaccessible. Ah, then my assumption wasn't right. I guessed that you want to call TeX-pdf-tools-sync-view or TeX-evince-sync-view-1 with `file' bound to function name such as TeX-master-file or TeX-region-file, so I presumed that it would be a low-cost hack to make those functions work with whichever value of the variable `file', a function name or a string. > One way to not break any past behavior would be to introduce a new > dynamically-scoped variable `file-fn` that always has the function > value. Although I am not sure if introducing yet another dynamically > scoped variable is a good solution long-term. Agreed. >> Another possible idea is to stop relying on dynamic scope variable >> `file' and just to call TeX-active-master in TeX-pdf-tools-sync-view and >> TeX-evince-sync-view-1. My understanding is that TeX-active-master >> serves for such purposes for most cases. >> > This is what I thought at first. However, according to comments in > tex.el, `file` can be `TeX-master-file', `TeX-region-file' or > `TeX-active-master'. TeX-active-master just calls TeX-master-file or TeX-region-file depending on the value of TeX-current-process-region-p, which is set early stage in TeX-command. So TeX-active-master would usually be a drop-in replacement for TeX-master-file and TeX-region-file as long as it is called inside TeX-command. Best, Ikumi Keita
[AUCTeX-commit] GNU AUCTeX branch, master, updated. b06ca0a48e56b7e006be02a7a845054d600538c5
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via b06ca0a48e56b7e006be02a7a845054d600538c5 (commit) from 07b79fca8bf7114981605748735594d7b1eff0a5 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit b06ca0a48e56b7e006be02a7a845054d600538c5 Author: Ikumi Keita Date: Tue Aug 4 21:03:17 2020 +0900 Fix document * doc/auctex.texi (Fontification of math): Delete incorrect description. --- Summary of changes: doc/auctex.texi | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) hooks/post-receive -- GNU AUCTeX ___ auctex-commit mailing list auctex-com...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-commit
[AUCTeX-diffs] GNU AUCTeX branch, master, updated. b06ca0a48e56b7e006be02a7a845054d600538c5
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via b06ca0a48e56b7e006be02a7a845054d600538c5 (commit) from 07b79fca8bf7114981605748735594d7b1eff0a5 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit b06ca0a48e56b7e006be02a7a845054d600538c5 Author: Ikumi Keita Date: Tue Aug 4 21:03:17 2020 +0900 Fix document * doc/auctex.texi (Fontification of math): Delete incorrect description. diff --git a/doc/auctex.texi b/doc/auctex.texi index 7eb606a..b19b60d 100644 --- a/doc/auctex.texi +++ b/doc/auctex.texi @@ -2270,9 +2270,7 @@ like @samp{\(...\)} or @samp{\[...\]} is built-in and not customizable. Support for other math macros and environments can be adapted by customizing the variables @code{font-latex-match-math-command-keywords} and @code{texmathp-tex-commands} respectively. It is no longer -recommended to customize @code{font-latex-math-environments}, which -is initialized according to @code{texmathp-tex-commands} and -@code{texmathp-tex-commands-default} by default. +recommended to customize @code{font-latex-math-environments}. To convert your customization in @code{font-latex-math-environments} into @code{texmathp-tex-commands}, please register your own math --- Summary of changes: doc/auctex.texi | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) hooks/post-receive -- GNU AUCTeX ___ auctex-diffs mailing list auctex-di...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-diffs
[AUCTeX-diffs] GNU AUCTeX branch, lexical-binding-attempt-1, updated. 8bfa0ba6121c53dfd93921d728241a08f15015fc
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, lexical-binding-attempt-1 has been updated via 8bfa0ba6121c53dfd93921d728241a08f15015fc (commit) from bacdf14aa2a99732034bb7c869f3643222c8222d (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit 8bfa0ba6121c53dfd93921d728241a08f15015fc Author: Ikumi Keita Date: Tue Sep 22 22:01:45 2020 +0900 Use declared dynamic variable in tex-jp * tex-jp.el (TeX-expand-list-builtin): Replace former dynamic var `pos' with `TeX-expand-pos'. (): Enable lexical binding. Move `coding' tag to the File Local Variable section at EOF. Update copyright year. diff --git a/tex-jp.el b/tex-jp.el index dff1b11..41ea4c5 100644 --- a/tex-jp.el +++ b/tex-jp.el @@ -1,6 +1,6 @@ -;;; tex-jp.el --- Support for Japanese TeX. -*- coding: iso-2022-jp-unix; -*- +;;; tex-jp.el --- Support for Japanese TeX. -*- lexical-binding: t; -*- -;; Copyright (C) 1999, 2001-2008, 2012-2013, 2016-2018 +;; Copyright (C) 1999, 2001-2008, 2012-2013, 2016-2018, 2020 ;; Free Software Foundation, Inc. ;; Author: KOBAYASHI Shinji , @@ -148,7 +148,8 @@ systems are determined by their values regardless of the kanji option." (let ((result (format "-d %s" dicname))) ;; Advance past the file name in order to ;; prevent expanding any substring of it. - (setq pos (+ pos (length result))) + (setq TeX-expand-pos + (+ TeX-expand-pos (length result))) result) "" ;; pxdvi $B$H(B %(o?)xdvi $B$NE,@Z$JJ}$rA*Br$9$k!#(B @@ -912,4 +913,8 @@ save size (provide 'tex-jp) +;; Local Variables: +;; coding: iso-2022-jp-unix +;; End: + ;;; tex-jp.el ends here --- Summary of changes: tex-jp.el | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-) hooks/post-receive -- GNU AUCTeX ___ auctex-diffs mailing list auctex-di...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-diffs
[AUCTeX-commit] GNU AUCTeX branch, lexical-binding-attempt-1, updated. 8bfa0ba6121c53dfd93921d728241a08f15015fc
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, lexical-binding-attempt-1 has been updated via 8bfa0ba6121c53dfd93921d728241a08f15015fc (commit) from bacdf14aa2a99732034bb7c869f3643222c8222d (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit 8bfa0ba6121c53dfd93921d728241a08f15015fc Author: Ikumi Keita Date: Tue Sep 22 22:01:45 2020 +0900 Use declared dynamic variable in tex-jp * tex-jp.el (TeX-expand-list-builtin): Replace former dynamic var `pos' with `TeX-expand-pos'. (): Enable lexical binding. Move `coding' tag to the File Local Variable section at EOF. Update copyright year. --- Summary of changes: tex-jp.el | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-) hooks/post-receive -- GNU AUCTeX ___ auctex-commit mailing list auctex-com...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-commit
[AUCTeX-commit] GNU AUCTeX branch, lexical-binding-attempt-1, updated. 021d822f0db5aa67dcc3d5de3e992f2952d24f95
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, lexical-binding-attempt-1 has been updated via 021d822f0db5aa67dcc3d5de3e992f2952d24f95 (commit) from 487816398131a47a4b8a79bf9d86aae0b2b970a0 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit 021d822f0db5aa67dcc3d5de3e992f2952d24f95 Author: Ikumi Keita Date: Sun Oct 4 02:34:31 2020 +0900 ; * tests/tex/command-expansion.el: Update copyright year. --- Summary of changes: tests/tex/command-expansion.el | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) hooks/post-receive -- GNU AUCTeX ___ auctex-commit mailing list auctex-com...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-commit
[AUCTeX-diffs] GNU AUCTeX branch, lexical-binding-attempt-1, updated. 021d822f0db5aa67dcc3d5de3e992f2952d24f95
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, lexical-binding-attempt-1 has been updated via 021d822f0db5aa67dcc3d5de3e992f2952d24f95 (commit) from 487816398131a47a4b8a79bf9d86aae0b2b970a0 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit 021d822f0db5aa67dcc3d5de3e992f2952d24f95 Author: Ikumi Keita Date: Sun Oct 4 02:34:31 2020 +0900 ; * tests/tex/command-expansion.el: Update copyright year. diff --git a/tests/tex/command-expansion.el b/tests/tex/command-expansion.el index 3b2779d..0bf661a 100644 --- a/tests/tex/command-expansion.el +++ b/tests/tex/command-expansion.el @@ -1,6 +1,6 @@ ;;; command-expansion.el --- tests for TeX command expansion -;; Copyright (C) 2014, 2018 Free Software Foundation, Inc. +;; Copyright (C) 2014, 2018, 2020 Free Software Foundation, Inc. ;; This file is part of AUCTeX. --- Summary of changes: tests/tex/command-expansion.el | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) hooks/post-receive -- GNU AUCTeX ___ auctex-diffs mailing list auctex-di...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-diffs
[AUCTeX-commit] GNU AUCTeX branch, master, updated. 2f1369fc0115addb375df0064460178705ff658c
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via 2f1369fc0115addb375df0064460178705ff658c (commit) from d27f6a12d89a094bf5fd8f45f5ef7a16a9a9b29f (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit 2f1369fc0115addb375df0064460178705ff658c Author: Ikumi Keita Date: Thu May 28 14:52:01 2020 +0900 Fix false negative of texmathp (bug#41559) * texmathp.el (texmathp-compile): Fix `texmathp-onoff-regexp' to allow switch to begin at (point-min). Use `regexp-opt' instead of `mapconcat'+`regexp-quote'. (texmathp): Use `>=' instead of `>' so that match is updated even when arg-on or sw-on begins at (point-min). * tests/latex/texmathp-test.el: New test. --- Summary of changes: .../path-expansion.el => latex/texmathp-test.el} | 42 +++--- texmathp.el| 17 - 2 files changed, 38 insertions(+), 21 deletions(-) copy tests/{tex/path-expansion.el => latex/texmathp-test.el} (55%) hooks/post-receive -- GNU AUCTeX ___ auctex-commit mailing list auctex-com...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-commit
Re: TeX-remove-style-hook is local
Hi Stefan, >>>>> Stefan Monnier writes: > My understanding is that the intention is to add the function on the > global part of the hook, which is what the patch below does. > If the intention is to add it buffer-locally, then the code will need to > be restructured since the `add-hook` currently runs only when loading > AUCTeX and not when enabling the mode. I agree, by the following reasons, that it's appropriate to delete the `make-variable-buffer-local' as you proposed: Since AUCTeX run `TeX-auto-add-type' only once per an information type, `add-hook' in this macro should operate on the global value of `TeX-remove-style-hook'. There is another instance of `add-hook' on `TeX-remove-style-hook' in the form (add-hook 'TeX-remove-style-hook (lambda () (setq LaTeX-largest-level nil))) in latex.el at its top level. I also think that this `add-hook' should operate on the global value of `TeX-remove-style-hook'. Thus it looks apparently wrong to apply `make-variable-buffer-local' to it. How do others on auctex-dev list think? > AFAICT, `TeX-auto-add-type` is a macro that's run exclusively at the > top-level of some of AUCTeX's files (context.el, tex.el, and latex.el), This isn't correct actually, though it isn't important under the current context. Many files among style/*.el add their own information types by this macro. Regards, Ikumi Keita
[AUCTeX-diffs] GNU AUCTeX branch, master, updated. 2f1369fc0115addb375df0064460178705ff658c
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via 2f1369fc0115addb375df0064460178705ff658c (commit) from d27f6a12d89a094bf5fd8f45f5ef7a16a9a9b29f (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit 2f1369fc0115addb375df0064460178705ff658c Author: Ikumi Keita Date: Thu May 28 14:52:01 2020 +0900 Fix false negative of texmathp (bug#41559) * texmathp.el (texmathp-compile): Fix `texmathp-onoff-regexp' to allow switch to begin at (point-min). Use `regexp-opt' instead of `mapconcat'+`regexp-quote'. (texmathp): Use `>=' instead of `>' so that match is updated even when arg-on or sw-on begins at (point-min). * tests/latex/texmathp-test.el: New test. diff --git a/tests/latex/texmathp-test.el b/tests/latex/texmathp-test.el new file mode 100644 index 000..a1313b9 --- /dev/null +++ b/tests/latex/texmathp-test.el @@ -0,0 +1,51 @@ +;;; texmathp-test.el --- tests for texmathp + +;; Copyright (C) 2020 Free Software Foundation, Inc. + +;; This file is part of AUCTeX. + +;; AUCTeX is free software; you can redistribute it and/or modify it +;; under the terms of the GNU General Public License as published by +;; the Free Software Foundation; either version 3, or (at your option) +;; any later version. + +;; AUCTeX is distributed in the hope that it will be useful, but +;; WITHOUT ANY WARRANTY; without even the implied warranty of +;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU +;; General Public License for more details. + +;; You should have received a copy of the GNU General Public License +;; along with AUCTeX; see the file COPYING. If not, write to the Free +;; Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA +;; 02110-1301, USA. + +;;; Code: + +(require 'ert) +(require 'latex) +(require 'texmathp) + +;; bug#41559 +(ert-deftest texmathp-bob () + "Test math expressions beginning at BOB are identified correctly." + (should (with-temp-buffer + (insert "\\(") + (LaTeX-mode) + (texmathp))) + + (should (with-temp-buffer + (insert "\\[") + (LaTeX-mode) + (texmathp))) + + (should (with-temp-buffer + (insert "\\ensuremath{") + (LaTeX-mode) + (texmathp))) + + (should (with-temp-buffer + (insert "$") + (LaTeX-mode) + (texmathp + +;;; texmathp-test.el ends here diff --git a/texmathp.el b/texmathp.el index ba5aba3..723623b 100644 --- a/texmathp.el +++ b/texmathp.el @@ -185,13 +185,11 @@ customize (customize calls it when setting the variable)." ((memq type '(sw-toggle)) 'togglers))) (set var (cons (car entry) (symbol-value var (setq texmathp-onoff-regexp - (concat "[^]\\(" - (mapconcat 'regexp-quote switches "\\|") - "\\)") + (concat "\\(?:[^]\\|\\`\\)" + (regexp-opt switches t)) texmathp-toggle-regexp - (concat "\\([^\\$]\\|\\`\\)\\(" - (mapconcat 'regexp-quote togglers "\\|") - "\\)" + (concat "\\([^\\$]\\|\\`\\)" + (regexp-opt togglers t) (defcustom texmathp-tex-commands nil "List of environments and macros influencing (La)TeX math mode. @@ -287,14 +285,17 @@ See the variable `texmathp-tex-commands' about which commands are checked." ;; Select the nearer match (and env-match (setq match env-match)) -(and mac-match (> (cdr mac-match) (cdr match)) (setq match mac-match)) +;; Use `>=' instead of `>' in case called inside \ensuremath{..} +;; beginning just at (point-min). +(and mac-match (>= (cdr mac-match) (cdr match)) (setq match mac-match)) (setq math-on (memq (nth 1 (assoc (car match) texmathp-tex-commands1)) '(env-on arg-on))) ;; Check for switches (and (not math-on) (setq sw-match (texmathp-match-switch bound)) -(> (cdr sw-match) (cdr match)) +;; Use `>=' instead of `>' by similar reason as above. (bug#41559) +(>= (cdr sw-match) (cdr match)) (eq (nth 1 (assoc (car sw-match) texmathp-tex-commands1)) 'sw-on) (setq match sw-match math-on t)) --- Summary of changes: .../path-expansion.el => latex/texmathp-test.el} | 42 +++--- texmathp.el
[AUCTeX-diffs] GNU AUCTeX branch, master, updated. d27f6a12d89a094bf5fd8f45f5ef7a16a9a9b29f
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via d27f6a12d89a094bf5fd8f45f5ef7a16a9a9b29f (commit) from a9fd06c699ec010ca5f894be03c7f448e581b686 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit d27f6a12d89a094bf5fd8f45f5ef7a16a9a9b29f Author: Ikumi Keita Date: Mon May 25 20:28:17 2020 +0900 Fix regression of unfontify-region * font-latex.el (font-latex-unfontify-region): Restore `invisible' as removed property. My commit on Jan 7 was partially wrong. It kept "^" and "_" invisible even when commented out if `font-latex-fontify-script' is `invisible'. diff --git a/font-latex.el b/font-latex.el index 6210967..91137ba 100644 --- a/font-latex.el +++ b/font-latex.el @@ -1328,7 +1328,7 @@ If SYNTACTIC-KWS is non-nil, also update (defun font-latex-unfontify-region (beg end ignored) "Unfontify region from BEG to END." (font-lock-default-unfontify-region beg end) - (remove-text-properties beg end '(script-level)) + (remove-list-of-text-properties beg end '(script-level invisible)) (while (< beg end) (let ((next (next-single-property-change beg 'display nil end)) (prop (get-text-property beg 'display))) --- Summary of changes: font-latex.el | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) hooks/post-receive -- GNU AUCTeX ___ auctex-diffs mailing list auctex-di...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-diffs
[AUCTeX-commit] GNU AUCTeX branch, master, updated. d27f6a12d89a094bf5fd8f45f5ef7a16a9a9b29f
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via d27f6a12d89a094bf5fd8f45f5ef7a16a9a9b29f (commit) from a9fd06c699ec010ca5f894be03c7f448e581b686 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit d27f6a12d89a094bf5fd8f45f5ef7a16a9a9b29f Author: Ikumi Keita Date: Mon May 25 20:28:17 2020 +0900 Fix regression of unfontify-region * font-latex.el (font-latex-unfontify-region): Restore `invisible' as removed property. My commit on Jan 7 was partially wrong. It kept "^" and "_" invisible even when commented out if `font-latex-fontify-script' is `invisible'. --- Summary of changes: font-latex.el | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) hooks/post-receive -- GNU AUCTeX ___ auctex-commit mailing list auctex-com...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-commit
[AUCTeX-commit] GNU AUCTeX branch, master, updated. fc77abfcee426c3b3e0ced2f0cad02a385bfcb60
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via fc77abfcee426c3b3e0ced2f0cad02a385bfcb60 (commit) from e410168c79e942fd00fd4bb4b922ca3d70c1e09c (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit fc77abfcee426c3b3e0ced2f0cad02a385bfcb60 Author: Ikumi Keita Date: Mon Jun 1 00:29:06 2020 +0900 Simplify regexp in texmathp.el * texmathp.el (texmathp-compile): Simplify regexp. In [^...] backslash and dollar have no special meaning so they can (should) be raw. --- Summary of changes: texmathp.el | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) hooks/post-receive -- GNU AUCTeX ___ auctex-commit mailing list auctex-com...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-commit
[AUCTeX-diffs] GNU AUCTeX branch, master, updated. fc77abfcee426c3b3e0ced2f0cad02a385bfcb60
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via fc77abfcee426c3b3e0ced2f0cad02a385bfcb60 (commit) from e410168c79e942fd00fd4bb4b922ca3d70c1e09c (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit fc77abfcee426c3b3e0ced2f0cad02a385bfcb60 Author: Ikumi Keita Date: Mon Jun 1 00:29:06 2020 +0900 Simplify regexp in texmathp.el * texmathp.el (texmathp-compile): Simplify regexp. In [^...] backslash and dollar have no special meaning so they can (should) be raw. diff --git a/texmathp.el b/texmathp.el index 723623b..3beac2e 100644 --- a/texmathp.el +++ b/texmathp.el @@ -185,10 +185,10 @@ customize (customize calls it when setting the variable)." ((memq type '(sw-toggle)) 'togglers))) (set var (cons (car entry) (symbol-value var (setq texmathp-onoff-regexp - (concat "\\(?:[^]\\|\\`\\)" + (concat "\\(?:[^\\]\\|\\`\\)" (regexp-opt switches t)) texmathp-toggle-regexp - (concat "\\([^\\$]\\|\\`\\)" + (concat "\\([^\\$]\\|\\`\\)" (regexp-opt togglers t) (defcustom texmathp-tex-commands nil --- Summary of changes: texmathp.el | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) hooks/post-receive -- GNU AUCTeX ___ auctex-diffs mailing list auctex-di...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-diffs
[AUCTeX-commit] GNU AUCTeX branch, master, updated. 6654955216a42936b87f76dc346aad829b1d52fb
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via 6654955216a42936b87f76dc346aad829b1d52fb (commit) from fc77abfcee426c3b3e0ced2f0cad02a385bfcb60 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit 6654955216a42936b87f76dc346aad829b1d52fb Author: Ikumi Keita Date: Wed Jun 3 01:44:32 2020 +0900 Use search-based fontification for $...$ (bug#33139) It turned out that using "text quotes" syntax for "$" in `font-lock-syntax-table' sometimes leads to scrambled fontification about in-line math $...$ (bug#33139). That's because conflicting results of `syntax-ppss' are compiled into syntax-ppss cache as explained in: https://lists.gnu.org/archive/html/emacs-devel/2020-05/msg02725.html We, Keita, Tassilo and Arash discussed this issue and decided to use search-based fontification for $...$ since syntactic fontification for $...$ doesn't seem prospective with regard to this bug. * font-latex.el (font-latex-match-dollar-math): (font-latex-find-dollar-math,font-latex-extend-region-backwards-dollar-math): New functions to do search-based fontification for $...$. Fontification facility for $$...$$ is merged into them. (font-latex-make-user-keywords,font-latex-setup): Use new functions. (font-latex-syntax-alist): Don't change syntax of "$" during font lock. (font-latex-syntactic-face-function): Simplify. --- Summary of changes: font-latex.el | 131 +- 1 file changed, 111 insertions(+), 20 deletions(-) hooks/post-receive -- GNU AUCTeX ___ auctex-commit mailing list auctex-com...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-commit
[AUCTeX-diffs] GNU AUCTeX branch, master, updated. 6654955216a42936b87f76dc346aad829b1d52fb
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via 6654955216a42936b87f76dc346aad829b1d52fb (commit) from fc77abfcee426c3b3e0ced2f0cad02a385bfcb60 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit 6654955216a42936b87f76dc346aad829b1d52fb Author: Ikumi Keita Date: Wed Jun 3 01:44:32 2020 +0900 Use search-based fontification for $...$ (bug#33139) It turned out that using "text quotes" syntax for "$" in `font-lock-syntax-table' sometimes leads to scrambled fontification about in-line math $...$ (bug#33139). That's because conflicting results of `syntax-ppss' are compiled into syntax-ppss cache as explained in: https://lists.gnu.org/archive/html/emacs-devel/2020-05/msg02725.html We, Keita, Tassilo and Arash discussed this issue and decided to use search-based fontification for $...$ since syntactic fontification for $...$ doesn't seem prospective with regard to this bug. * font-latex.el (font-latex-match-dollar-math): (font-latex-find-dollar-math,font-latex-extend-region-backwards-dollar-math): New functions to do search-based fontification for $...$. Fontification facility for $$...$$ is merged into them. (font-latex-make-user-keywords,font-latex-setup): Use new functions. (font-latex-syntax-alist): Don't change syntax of "$" during font lock. (font-latex-syntactic-face-function): Simplify. diff --git a/font-latex.el b/font-latex.el index 91137ba..a707616 100644 --- a/font-latex.el +++ b/font-latex.el @@ -833,7 +833,7 @@ Generated by `font-latex-make-user-keywords'."))) ;; Add the "fixed" matchers and highlighters. (dolist (item '(("\\(^\\|[^\\]\\)\\(&+\\)" 2 'font-latex-warning-face) -("\\$\\$\\([^$]+\\)\\$\\$" 1 'font-latex-math-face) +(font-latex-match-dollar-math 0 'font-latex-math-face keep) (font-latex-match-quotation (0 'font-latex-string-face append) (1 'font-latex-warning-face)) @@ -1057,24 +1057,10 @@ have changed." ;;; Syntactic fontification -;; Copy and adaptation of `tex-font-lock-syntactic-face-function' in -;; `tex-mode.el' of CVS Emacs (March 2004) (defun font-latex-syntactic-face-function (state) - (let ((char (nth 3 state))) -(cond - ((not char) 'font-lock-comment-face) - ((eq char ?$) 'font-latex-math-face) - (t - (when (characterp char) - ;; This is a \verb?...? construct. Let's find the end and mark it. - (save-excursion - (skip-chars-forward (string ?^ char)) ;; Use `end' ? - (when (eq (char-syntax (preceding-char)) ?/) - (put-text-property (1- (point)) (point) 'syntax-table '(1))) - (unless (eobp) - (put-text-property (point) (1+ (point)) 'syntax-table '(7) - 'font-latex-verbatim-face - + (if (nth 3 state) + 'font-latex-verbatim-face +'font-lock-comment-face)) ;;; Faces @@ -1225,7 +1211,7 @@ have changed." (defvar font-latex-syntax-alist ;; Use word syntax for @ because we use \> for matching macros and ;; we don't want \foo@bar to be found if we search for \foo. - '((?\( . ".") (?\) . ".") (?$ . "\"") (?@ . "w")) + '((?\( . ".") (?\) . ".") (?@ . "w")) "List of specifiers for the syntax alist of `font-lock-defaults'.") (defun font-latex-add-to-syntax-alist (list) @@ -1277,7 +1263,8 @@ triggers Font Lock to recognize the change." font-latex-extend-region-backwards-command-in-braces font-latex-extend-region-backwards-quotation font-latex-extend-region-backwards-math-env - font-latex-extend-region-backwards-math-envII) + font-latex-extend-region-backwards-math-envII + font-latex-extend-region-backwards-dollar-math) (syntax-propertize-function . font-latex-syntax-propertize-function) (syntax-propertize-extend-region-functions @@ -1819,6 +1806,110 @@ The \\begin{equation} incl. arguments in the same line and (setq font-lock-beg (point)) (throw 'extend t)) +(defun font-latex-match-dollar-math (limit) + "Match inline math $...$ or display math $$...$$ before LIMIT." + (if (font-latex-find-dollar-math limit) + ;; Found "$" which starts $...$ or $$...$$. + (let ((beg (point)) + ;; Go inside the math expression. + (num (skip-chars-forward "$" limit))) + (if (<
[AUCTeX-diffs] GNU AUCTeX branch, lexical-binding-attempt-1, deleted. 860f46c45f4d105f24d4063ba6d83124970fdbea
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, lexical-binding-attempt-1 has been deleted was 860f46c45f4d105f24d4063ba6d83124970fdbea --- 860f46c45f4d105f24d4063ba6d83124970fdbea ; * style/graphicx.el: Fix Ataka-san's name and email. --- hooks/post-receive -- GNU AUCTeX ___ auctex-diffs mailing list auctex-di...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-diffs
[AUCTeX-diffs] GNU AUCTeX branch, master, updated. 5bffc2458ee9dfc846c1d98679cc9653534aac4e
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via 5bffc2458ee9dfc846c1d98679cc9653534aac4e (commit) from b42e932a4f7778f4cecc9a8354466461ae1e02b9 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit 5bffc2458ee9dfc846c1d98679cc9653534aac4e Author: Ikumi Keita Date: Sat Oct 24 17:13:10 2020 +0900 ; * font-latex.el (font-latex-update-math-env): Fix docstring. diff --git a/font-latex.el b/font-latex.el index f8edf91..6c1110f 100644 --- a/font-latex.el +++ b/font-latex.el @@ -1788,7 +1788,6 @@ Set by `font-latex--update-math-env' and used in (defun font-latex-update-math-env () "Update regexp to search for math environments. -Helper function for style files such as amsmath.el. Extract environments marked as `env-on' in `texmathp-tex-commands1' except starred variants. Then build `font-latex--match-math-envII-regexp' from them, appending the --- Summary of changes: font-latex.el | 1 - 1 file changed, 1 deletion(-) hooks/post-receive -- GNU AUCTeX ___ auctex-diffs mailing list auctex-di...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-diffs
Re: Re: Slash at the end of directory (was Re: oddity with TeX-style-private ?)
>>>>> "Tassilo Horn" writes: >> > IMHO, `TeX-strip-extension' and `TeX-parse-path' should better test with >> > (member dir (mapcar #'file-name-as-directory TeX-style-global)) etc. It >> > shouldn't be the user's job to ensure that directory names are written >> > with a trailing slash. >> >> OK, then how about the following patch. > LGTM, thanks! Pushed. Oops, sorry, I overlooked that new commits had come in. A non fast-forward merge commit was produced on savannah. Not good... Regards, Ikumi Keita
Re: Slash at the end of directory (was Re: oddity with TeX-style-private ?)
Hi Tassilo, >>>>> Tassilo Horn writes: > IMHO, `TeX-strip-extension' and `TeX-parse-path' should better test with > (member dir (mapcar #'file-name-as-directory TeX-style-global)) etc. It > shouldn't be the user's job to ensure that directory names are written > with a trailing slash. OK, then how about the following patch? Regards, Ikumi Keita diff --git a/tex-site.el.in b/tex-site.el.in index fd28869d..12287d51 100644 --- a/tex-site.el.in +++ b/tex-site.el.in @@ -75,7 +75,6 @@ TeX-auto-* (automatically generated lisp).") (defcustom TeX-auto-global @lispautodir@ "*Directory containing automatically generated information. -Must end with a directory separator. For storing automatic extracted information about the TeX macros shared by all users of a site." diff --git a/tex.el b/tex.el index 43cc8d11..d722587e 100644 --- a/tex.el +++ b/tex.el @@ -2612,6 +2612,9 @@ If REGEXP is nil, or \"\", an error will occur." (TeX-split-string (if (string-match ";" value) ";" ":") value))) + (global (append '("/" "\\") + (mapcar #'file-name-as-directory + TeX-macro-global))) entry answers) (while entries @@ -2622,7 +2625,7 @@ If REGEXP is nil, or \"\", an error will occur." (substring entry 0 (match-beginning 0)) entry))) (or (not (file-name-absolute-p entry)) - (member entry (append '("/" "\\") TeX-macro-global)) + (member entry global) (setq answers (cons entry answers answers)) @@ -4483,8 +4486,9 @@ If EXTENSIONS is not specified or nil, the value of (defun TeX-strip-extension ( string extensions nodir nostrip) "Return STRING without any trailing extension in EXTENSIONS. If NODIR is t, also remove directory part of STRING. -If NODIR is `path', remove directory part of STRING if it is equal to -the current directory, `TeX-macro-private' or `TeX-macro-global'. +If NODIR is `path', remove directory part of STRING if it is +equal to the current directory or is a member of +`TeX-macro-private' or `TeX-macro-global'. If NOSTRIP is set, do not remove extension after all. STRING defaults to the name of the current buffer. EXTENSIONS defaults to `TeX-file-extensions'." @@ -4502,8 +4506,8 @@ EXTENSIONS defaults to `TeX-file-extensions'." (dir (expand-file-name (or (file-name-directory strip) "./" (if (or (eq nodir t) (string-equal dir (expand-file-name "./")) - (member dir TeX-macro-global) - (member dir TeX-macro-private)) + (member dir (mapcar #'file-name-as-directory TeX-macro-global)) + (member dir (mapcar #'file-name-as-directory TeX-macro-private))) (file-name-nondirectory strip) strip)))
[AUCTeX-diffs] GNU AUCTeX branch, master, updated. 7288b930150d6d64ad3cae2a38c98adb784bdc04
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via 7288b930150d6d64ad3cae2a38c98adb784bdc04 (commit) via 59c40b0140651ba8764b78bcafcdd523741cfcc4 (commit) from a9c37b7ecd2353093fbecc84561fcace1ac8be80 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit 7288b930150d6d64ad3cae2a38c98adb784bdc04 Merge: a9c37b7 59c40b0 Author: Ikumi Keita Date: Mon Oct 26 01:15:47 2020 +0900 Merge branch 'my-fix' into master commit 59c40b0140651ba8764b78bcafcdd523741cfcc4 Author: Ikumi Keita Date: Sun Oct 25 16:02:14 2020 +0900 Ensure the trailing slash for directory * tex.el (TeX-parse-path, TeX-strip-extension): Apply `file-name-as-directory' for `TeX-macro-global' and `TeX-macro-private' in case their entries lack the trailing slash. * tex-site.el.in (TeX-auto-global): Delete obsolete sentence. diff --git a/tex-site.el.in b/tex-site.el.in index fd28869..12287d5 100644 --- a/tex-site.el.in +++ b/tex-site.el.in @@ -75,7 +75,6 @@ TeX-auto-* (automatically generated lisp).") (defcustom TeX-auto-global @lispautodir@ "*Directory containing automatically generated information. -Must end with a directory separator. For storing automatic extracted information about the TeX macros shared by all users of a site." diff --git a/tex.el b/tex.el index 43cc8d1..d722587 100644 --- a/tex.el +++ b/tex.el @@ -2612,6 +2612,9 @@ If REGEXP is nil, or \"\", an error will occur." (TeX-split-string (if (string-match ";" value) ";" ":") value))) +(global (append '("/" "\\") +(mapcar #'file-name-as-directory +TeX-macro-global))) entry answers) (while entries @@ -2622,7 +2625,7 @@ If REGEXP is nil, or \"\", an error will occur." (substring entry 0 (match-beginning 0)) entry))) (or (not (file-name-absolute-p entry)) - (member entry (append '("/" "\\") TeX-macro-global)) + (member entry global) (setq answers (cons entry answers answers)) @@ -4483,8 +4486,9 @@ If EXTENSIONS is not specified or nil, the value of (defun TeX-strip-extension ( string extensions nodir nostrip) "Return STRING without any trailing extension in EXTENSIONS. If NODIR is t, also remove directory part of STRING. -If NODIR is `path', remove directory part of STRING if it is equal to -the current directory, `TeX-macro-private' or `TeX-macro-global'. +If NODIR is `path', remove directory part of STRING if it is +equal to the current directory or is a member of +`TeX-macro-private' or `TeX-macro-global'. If NOSTRIP is set, do not remove extension after all. STRING defaults to the name of the current buffer. EXTENSIONS defaults to `TeX-file-extensions'." @@ -4502,8 +4506,8 @@ EXTENSIONS defaults to `TeX-file-extensions'." (dir (expand-file-name (or (file-name-directory strip) "./" (if (or (eq nodir t) (string-equal dir (expand-file-name "./")) - (member dir TeX-macro-global) - (member dir TeX-macro-private)) + (member dir (mapcar #'file-name-as-directory TeX-macro-global)) + (member dir (mapcar #'file-name-as-directory TeX-macro-private))) (file-name-nondirectory strip) strip))) --- Summary of changes: tex-site.el.in | 1 - tex.el | 14 +- 2 files changed, 9 insertions(+), 6 deletions(-) hooks/post-receive -- GNU AUCTeX ___ auctex-diffs mailing list auctex-di...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-diffs
[AUCTeX-commit] GNU AUCTeX branch, master, updated. 7288b930150d6d64ad3cae2a38c98adb784bdc04
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "GNU AUCTeX". The branch, master has been updated via 7288b930150d6d64ad3cae2a38c98adb784bdc04 (commit) via 59c40b0140651ba8764b78bcafcdd523741cfcc4 (commit) from a9c37b7ecd2353093fbecc84561fcace1ac8be80 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log - commit 7288b930150d6d64ad3cae2a38c98adb784bdc04 Merge: a9c37b7 59c40b0 Author: Ikumi Keita Date: Mon Oct 26 01:15:47 2020 +0900 Merge branch 'my-fix' into master commit 59c40b0140651ba8764b78bcafcdd523741cfcc4 Author: Ikumi Keita Date: Sun Oct 25 16:02:14 2020 +0900 Ensure the trailing slash for directory * tex.el (TeX-parse-path, TeX-strip-extension): Apply `file-name-as-directory' for `TeX-macro-global' and `TeX-macro-private' in case their entries lack the trailing slash. * tex-site.el.in (TeX-auto-global): Delete obsolete sentence. --- Summary of changes: tex-site.el.in | 1 - tex.el | 14 +- 2 files changed, 9 insertions(+), 6 deletions(-) hooks/post-receive -- GNU AUCTeX ___ auctex-commit mailing list auctex-com...@gnu.org https://lists.gnu.org/mailman/listinfo/auctex-commit
Re: locate-library for lisp files in style directory?
Hi Uwe, >>>>> Uwe Brauer writes: > That puzzled me for a long time, how can I check, if actually a file in > auctex style directory is used? > locate-library seems not to work since there is actually no path set to > that directory. Well the TeX-style-*whatever* variable is set but > locate-library does not work for these. > So how can I figure out whether or which style file is used? With respect to "whether", you can obtain approximate result by (locate-library "STYLE-NAME" nil TeX-style-path) . The above code shows only the first match. AUCTeX loads _all_ style files found in TeX-style-path. If you really want to know all of them, make a new function by modifying TeX-load-style. Regards, Ikumi Keita
Slash at the end of directory (was Re: oddity with TeX-style-private ?)
>>>>> Tassilo Horn writes: > Uwe Brauer writes: >>>>> "CB" == Colin Baxter writes: CB> However, the default setting for this variable, given in CB> 'tex-site.el' is "/usr/local/var/auctex", which does not end in '/'. Uwe> Right, this also crossed my mind and I tried it out. Uwe> The '/' did not make any difference. I was also wondering. Tassilo> Probably a relict of ancient times where paths were build using Tassilo> `concat' rather than `expand-file-name' and friends, I don't Tassilo> know... The commit [1] removed most of the occurences of "Must end with a directory separator." The one discussed here must be just a remnant of them. I think that statement can be deleted safely. According to the words from David in [2], it seems that the removal was done for the sake of xemacs. By the way, I think that file-name-as-directory should be re-enabled for TeX-macro-{global,private}, considering that we no longer support xemacs, because it seems that TeX-strip-extension and TeX-parse-path assume that those variables have list of strings which end with trailing slashes. TeX-strip-extension: (let* (... (dir (expand-file-name (or (file-name-directory strip) "./" ... (member dir TeX-macro-global) (member dir TeX-macro-private)) TeX-parse-path: (setq entry (file-name-as-directory (if (string-match "/?/?\\'" entry) (substring entry 0 (match-beginning 0)) entry))) ... (member entry (append '("/" "\\") TeX-macro-global)) Regards, Ikumi Keita [1] https://git.savannah.gnu.org/cgit/auctex.git/commit/?id=4ac002797d84dcb57ee4ec10c7e120ad71bf0c69 [2] https://debbugs.gnu.org/cgi/bugreport.cgi?bug=31150 "XEmacs used to do things differently (and wasn't tolerant about it as opposed to Emacs if I remember correctly) and thus many packages used that behavior."
Re: Auctex Commit
[Cc: auctex-devel] Hi Daniel, thanks for your report. >>>>> Daniel Fleischer writes: > Hi, I'm using Auctex, mainly using CDLatex and Org-mode. > In June, you made some commits to texmathp.el, namely removed the AMS > environment (put them in style/amsmath.el). You added the comment: > "Support for AMS-LaTeX and packages mathtools, empheq and breqn is added as > well if `TeX-parse-self' option is enabled." > I don't understand how to get texmathp to recognize the math environment in > Orgmode. For example it doesn't recognize: > \begin{align} > \end{align} > to be a mathematical environment, so it doesn't run > `org-cdlatex-underscore-caret` correctly, for example. > Thanks for the help, > Daniel Fleischer I'm sorry, I didn't care about usage of texmathp.el outside AUCTeX. I'd like to address this issue soon. For temporal workaround, you can add math environments like align to `texmathp-tex-commands'. The story behind the scene was: 1. Font lock (syntax highlighting) facility of AUCTeX changed to use texmathp.el intensively in order to address a bug in highlighting of math expression. 2. Now font lock facility checks for all math environments registered in texmathp. Thus we want to have lightweight list of math environments for efficiency since font lock operation is called at almost every key strokes. 3. So we moved extra math environments from texmathp.el into respective style files. To developers in auctex-devel lists: I think that the simplest solution is to restore extra math environments back into texmathp.el again. This means that font lock operation checks for all math environments defined in AMS-LaTeX, mathtools, empheq and breqn even for vanilla LaTeX documents, but I hope we can live with that. Comments or suggestions? Regards, Ikumi Keita
Re: Auctex Commit
Hi all, >>>>> Daniel Fleischer writes: > Thank you very much for the quick response. > I kept wondering if I should go to CDLatex (as a gateway to latex on orgmode) > or AUCTex. > There's an issue on CDLatex about that > https://github.com/cdominik/cdlatex/issues/17 > <https://github.com/cdominik/cdlatex/issues/17>. > Anyway, I guess it's a question of efficiency vs. compatibility with > downstream packages. Tough call, it's up to the community. > In the meantime I went back to 12.2, before the commits of this year. > Thanks again, > Daniel Fleischer I just pushed the fix into the git repository. I have my summer vacation from tomorrow and have poor internet connectivity during it. So please feel free to revert that commit if it causes some trouble I'm not aware of. Cheers, Ikumi Keita