GNU bug report logs - #46117
12.3.1; windows unfriendly change

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 46117 in the body.
You can then email your comments to 46117 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Matt Stinson <mrmatt2532 <at> gmail.com>
To: bug-auctex <at> gnu.org
Subject: 12.3.1; windows unfriendly change
Date: Tue, 26 Jan 2021 09:31:51 -0600

[Message part 1 (text/plain, inline)]

The recent change to tex-jp.el (commit:

384c1d2528eba44cd53d8ea42e151c7b4445bafd) appears to have added a

windows unfriendly coding.



When the auctex builds I receive the following error related to tex-jp.el:
"local variables

entry is missing the suffix"



I think rather than

coding: iso-2022-jp-unix

it should be

coding: iso-2022-jp



Emacs  : GNU Emacs 27.1 (build 1, x86_64-w64-mingw32)

 of 2020-08-21

Package: 12.3.1



current state:

==============

(setq

 AUCTeX-date "2020-10-22"

 window-system 'w32

 LaTeX-version "2e"

 TeX-style-path '("~/.emacs.d/auctex"

                  "c:/Users/irfydj/.emacs.d/straight/build/auctex/style"

                  "c:/Users/irfydj/.emacs.d/auctex/auto"

                  "c:/Users/irfydj/.emacs.d/auctex/style" "auto" "style")

 TeX-auto-save t

 TeX-parse-self t

 TeX-master t

 TeX-command-list '(("TeX"

                     "%(PDF)%(tex) %(file-line-error) %`%(extraopts)
%S%(PDFout)%(mode)%' %t"

                     TeX-run-TeX nil

                     (plain-tex-mode ams-tex-mode texinfo-mode) :help

                     "Run plain TeX")

                    ("LaTeX" "%`%l%(mode)%' %T" TeX-run-TeX nil

                     (latex-mode doctex-mode) :help "Run LaTeX")

                    ("Makeinfo" "makeinfo %(extraopts) %t" TeX-run-compile
nil

                     (texinfo-mode) :help "Run Makeinfo with Info output")

                    ("Makeinfo HTML" "makeinfo %(extraopts) --html %t"

                     TeX-run-compile nil (texinfo-mode) :help

                     "Run Makeinfo with HTML output")

                    ("AmSTeX" "amstex %(PDFout) %`%(extraopts) %S%(mode)%'
%t"

                     TeX-run-TeX nil (ams-tex-mode) :help "Run AMSTeX")

                    ("ConTeXt"

                     "%(cntxcom) --once --texutil %(extraopts)
%(execopts)%t"

                     TeX-run-TeX nil (context-mode) :help "Run ConTeXt
once")

                    ("ConTeXt Full" "%(cntxcom) %(extraopts) %(execopts)%t"

                     TeX-run-TeX nil (context-mode) :help

                     "Run ConTeXt until completion")

                    ("BibTeX" "bibtex %s" TeX-run-BibTeX nil

                     (plain-tex-mode latex-mode doctex-mode ams-tex-mode

                      texinfo-mode context-mode)

                     :help "Run BibTeX")

                    ("Biber" "biber %s" TeX-run-Biber nil

                     (plain-tex-mode latex-mode doctex-mode ams-tex-mode

                      texinfo-mode)

                     :help "Run Biber")

                    ("View" "%V" TeX-run-discard-or-function t t :help

                     "Run Viewer")

                    ("Print" "%p" TeX-run-command t t :help "Print the
file")

                    ("Queue" "%q" TeX-run-background nil t :help

                     "View the printer queue" :visible TeX-queue-command)

                    ("File" "%(o?)dvips %d -o %f " TeX-run-dvips t

                     (plain-tex-mode latex-mode doctex-mode ams-tex-mode

                      texinfo-mode)

                     :help "Generate PostScript file")

                    ("Dvips" "%(o?)dvips %d -o %f " TeX-run-dvips nil

                     (plain-tex-mode latex-mode doctex-mode ams-tex-mode

                      texinfo-mode)

                     :help "Convert DVI file to PostScript")

                    ("Dvipdfmx" "dvipdfmx %d" TeX-run-dvipdfmx nil

                     (plain-tex-mode latex-mode doctex-mode ams-tex-mode

                      texinfo-mode)

                     :help "Convert DVI file to PDF with dvipdfmx")

                    ("Ps2pdf" "ps2pdf %f" TeX-run-ps2pdf nil

                     (plain-tex-mode latex-mode doctex-mode ams-tex-mode

                      texinfo-mode)

                     :help "Convert PostScript file to PDF")

                    ("Glossaries" "makeglossaries %s" TeX-run-command nil

                     (plain-tex-mode latex-mode doctex-mode ams-tex-mode

                      texinfo-mode)

                     :help "Run makeglossaries to create glossary\n
file")

                    ("Index" "makeindex %s" TeX-run-index nil

                     (plain-tex-mode latex-mode doctex-mode ams-tex-mode

                      texinfo-mode)

                     :help "Run makeindex to create index file")

                    ("upMendex" "upmendex %s" TeX-run-index t

                     (plain-tex-mode latex-mode doctex-mode ams-tex-mode

                      texinfo-mode)

                     :help "Run upmendex to create index file")

                    ("Xindy" "texindy %s" TeX-run-command nil

                     (plain-tex-mode latex-mode doctex-mode ams-tex-mode

                      texinfo-mode)

                     :help "Run xindy to create index file")

                    ("Check" "lacheck %s" TeX-run-compile nil (latex-mode)

                     :help "Check LaTeX file for correctness")

                    ("ChkTeX" "chktex -v6 %s" TeX-run-compile nil
(latex-mode)

                     :help "Check LaTeX file for common mistakes")

                    ("Spell" "(TeX-ispell-document \"\")" TeX-run-function
nil

                     t :help "Spell-check the document")

                    ("Clean" "TeX-clean" TeX-run-function nil t :help

                     "Delete generated intermediate files")

                    ("Clean All" "(TeX-clean t)" TeX-run-function nil t
:help

                     "Delete generated intermediate and output files")

                    ("Other" "" TeX-run-command t t :help

                     "Run an arbitrary command")

                    )

 )

[Message part 2 (text/html, inline)]

Message #8 received at 46117 <at> debbugs.gnu.org (full text, mbox):

From: Ikumi Keita <ikumi <at> ikumi.que.jp>
To: Matt Stinson <mrmatt2532 <at> gmail.com>
Cc: 46117 <at> debbugs.gnu.org
Subject: Re: bug#46117: 12.3.1; windows unfriendly change
Date: Wed, 27 Jan 2021 16:53:46 +0900

Hi Matt,

>>>>> Matt Stinson <mrmatt2532 <at> gmail.com> writes:
> When the auctex builds I receive the following error related to tex-jp.el:
> "local variables

> entry is missing the suffix"

I assume the following two:
(1) You are using git for windows which enables autocrlf feature.
(2) You built AUCTeX on a local copy of git repository.
(Actually I couldn't reproduce the problem by just copying the files to
my windows 10 and running configure and make with msys. It's only after
a web search with the keywords "local variables entry is missing the
suffix" that I realized what's going on by references such as:
https://github.com/hlissner/doom-emacs/issues/2637
https://github.com/raxod502/straight.el/issues/346
https://github.com/tumashu/helm-posframe/pull/9
https://github.com/ikirill/irony-eldoc/pull/13
)
The autocrlf feature silently converts the eol of the files in the
working directory from LF to CRLF, which ends up with contradicting
coding: tag specified in tex-jp.el.

I think that the proper approach for this issue is to turn off the
autocrlf feature for at least the local AUCTeX repository, but at the
same time think that it isn't bad to make AUCTeX more generous to
loosely configured local git repos. I'll remove eol suffix from coding:
tag.

> The recent change to tex-jp.el (commit:
> 384c1d2528eba44cd53d8ea42e151c7b4445bafd) appears to have added a

> windows unfriendly coding.

> I think rather than

> coding: iso-2022-jp-unix

> it should be

> coding: iso-2022-jp

I don't think it's windows unfriendly. As stated above, the current tag
coding: iso-2022-jp-unix
works fine even on windows if the file is copied as-is, without eol
format conversion. In my opinion, it's more of the problem on the side
of git for windows, with respect to the status of autocrlf feature.

Regards,
Ikumi Keita

Message #15 received at 46117 <at> debbugs.gnu.org (full text, mbox):

From: Matt Stinson <mrmatt2532 <at> gmail.com>
To: Ikumi Keita <ikumi <at> ikumi.que.jp>
Cc: 46117 <at> debbugs.gnu.org
Subject: Re: bug#46117: 12.3.1; windows unfriendly change
Date: Wed, 27 Jan 2021 08:21:50 -0600

[Message part 1 (text/plain, inline)]

Hello Ikumi,

Yes, you are correct in your assumptions regarding my configuration. In
fact, I am using straight.el (2nd github link you found). While I agree
that this issue is not strictly a bug, it is inconvenient for users who
automate the git package pulling process from many repositories (such as
straight.el users). If acceptable, I would request that the relaxed coding
setting is used (without the "-unix"), especially since the autocrlf
setting is very commonly used on windows (default in fact).

To put this case in perspective, I have almost 100 different emacs package
repositories that are automatically pulled down using git and this is
literally the only emacs lisp file among the complete set that has this
issue.

By the way, I have tested on my end and I can confirm that simply removing
the "-unix" from the coding and rebuilding resolves any issue.

Thank you,
Matt

On Wed, Jan 27, 2021 at 1:53 AM Ikumi Keita <ikumi <at> ikumi.que.jp> wrote:

> Hi Matt,
>
> >>>>> Matt Stinson <mrmatt2532 <at> gmail.com> writes:
> > When the auctex builds I receive the following error related to
> tex-jp.el:
> > "local variables
>
> > entry is missing the suffix"
>
> I assume the following two:
> (1) You are using git for windows which enables autocrlf feature.
> (2) You built AUCTeX on a local copy of git repository.
> (Actually I couldn't reproduce the problem by just copying the files to
> my windows 10 and running configure and make with msys. It's only after
> a web search with the keywords "local variables entry is missing the
> suffix" that I realized what's going on by references such as:
> https://github.com/hlissner/doom-emacs/issues/2637
> https://github.com/raxod502/straight.el/issues/346
> https://github.com/tumashu/helm-posframe/pull/9
> https://github.com/ikirill/irony-eldoc/pull/13
> )
> The autocrlf feature silently converts the eol of the files in the
> working directory from LF to CRLF, which ends up with contradicting
> coding: tag specified in tex-jp.el.
>
> I think that the proper approach for this issue is to turn off the
> autocrlf feature for at least the local AUCTeX repository, but at the
> same time think that it isn't bad to make AUCTeX more generous to
> loosely configured local git repos. I'll remove eol suffix from coding:
> tag.
>
> > The recent change to tex-jp.el (commit:
> > 384c1d2528eba44cd53d8ea42e151c7b4445bafd) appears to have added a
>
> > windows unfriendly coding.
>
> > I think rather than
>
> > coding: iso-2022-jp-unix
>
> > it should be
>
> > coding: iso-2022-jp
>
> I don't think it's windows unfriendly. As stated above, the current tag
> coding: iso-2022-jp-unix
> works fine even on windows if the file is copied as-is, without eol
> format conversion. In my opinion, it's more of the problem on the side
> of git for windows, with respect to the status of autocrlf feature.
>
> Regards,
> Ikumi Keita
>

[Message part 2 (text/html, inline)]

Message #18 received at 46117 <at> debbugs.gnu.org (full text, mbox):

From: Ikumi Keita <ikumi <at> ikumi.que.jp>
To: Matt Stinson <mrmatt2532 <at> gmail.com>
Cc: 46117 <at> debbugs.gnu.org
Subject: Re: bug#46117: 12.3.1; windows unfriendly change
Date: Thu, 28 Jan 2021 00:13:55 +0900

>>>>> Matt Stinson <mrmatt2532 <at> gmail.com> writes:
> If acceptable, I would request that the relaxed coding setting is used
> (without the "-unix")

Yeah, I have alreadly done so, hence I closed the bug :-). Try the
latest AUCTeX git repo, please.

Best,
Ikumi Keita

Message #21 received at 46117 <at> debbugs.gnu.org (full text, mbox):

From: Matt Stinson <mrmatt2532 <at> gmail.com>
To: Ikumi Keita <ikumi <at> ikumi.que.jp>
Cc: 46117 <at> debbugs.gnu.org
Subject: Re: bug#46117: 12.3.1; windows unfriendly change
Date: Wed, 27 Jan 2021 10:10:33 -0600

[Message part 1 (text/plain, inline)]

Perfect, thank you!

On Wed, Jan 27, 2021 at 9:13 AM Ikumi Keita <ikumi <at> ikumi.que.jp> wrote:

> >>>>> Matt Stinson <mrmatt2532 <at> gmail.com> writes:
> > If acceptable, I would request that the relaxed coding setting is used
> > (without the "-unix")
>
> Yeah, I have alreadly done so, hence I closed the bug :-). Try the
> latest AUCTeX git repo, please.
>
> Best,
> Ikumi Keita
>

[Message part 2 (text/html, inline)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.