unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Re: bug#23329: 11.89; Unloading tex-site doesn't restore original functionality
       [not found] <18668.1461248711@localhost>
@ 2016-04-21 16:22 ` Mosè Giordano
  2016-04-22  6:49   ` Ikumi Keita
  0 siblings, 1 reply; 4+ messages in thread
From: Mosè Giordano @ 2016-04-21 16:22 UTC (permalink / raw)
  To: Ikumi Keita; +Cc: 23329, emacs-devel

Dear Emacs developers,

it has been reported that unloading AUCTeX (M-x unload-feature RET
tex-site RET) doesn't work in Emacs 24:
https://debbugs.gnu.org/cgi/bugreport.cgi?bug=23329  I tested on
fencepost and noted that it works with Emacs 22 and 23, instead.

tex-site.el defines `tex-site-unload-hook' which looks into
`after-load-alist' to remove AUCTeX settings (it looks for
`TeX-modes-set' function), but in Emacs 24 `after-load-alist' seems to
have changed and there is no reference to `TeX-modes-set' anymore.  Do
you have any idea how to unload AUCTeX in Emacs 24?

Thanks,
Mosè


2016-04-21 16:25 GMT+02:00 Ikumi Keita <ikumi@ikumi.que.jp>:
>
> Remember to cover the basics, that is, what you expected to happen and
> what in fact did happen.
>
> Be sure to consult the FAQ section in the manual before submitting
> a bug report.  In addition check if the bug is reproducable with an
> up-to-date version of AUCTeX.  So please upgrade to the version
> available from http://www.gnu.org/software/auctex/ if your
> installation is older than the one available from the web site.
>
> If the bug is triggered by a specific (La)TeX file, you should try
> to produce a minimal sample file showing the problem and include it
> in your report.
>
> Your report will be posted for the auctex package at the GNU bug
> tracker.  Visit http://debbugs.gnu.org/cgi/pkgreport.cgi?pkg=auctex
> to browse existing AUCTeX bugs.
> ------------------------------------------------------------------------
>
> Unloading tex-site doesn't restore the functionality of the original
> tex-mode.
>
> [How to reproduce]
> 1. Start a new emacs session.
> 2. M-x unload-feature RET tex-site RET
> 3. C-x C-f /tmp/abc.tex RET
> 4. I expect abc.tex is opened with the original, emacs built-in
>    tex-mode. However, emacs reports that "File mode specification error:
>    (void-function TeX-tex-mode)"
>
> Even if I replace ~/.emacs/init.el with a file of only a line
> (load "auctex.el" nil t t)
> , nothing changes.
>
> If I set `debug-on-error' to t before the item 3 of the above list, the
> following backtrace is displayed.  Apparently overriding advice still
> remains.
> vvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvv
> Debugger entered--Lisp error: (void-function TeX-tex-mode)
>   TeX-tex-mode()
>   apply(TeX-tex-mode nil)
>   tex-mode()
>   set-auto-mode-0(tex-mode nil)
>   set-auto-mode()
>   normal-mode(t)
>   after-find-file(t t)
>   find-file-noselect-1(#<buffer abc.tex> "/tmp/abc.tex" nil nil "/tmp/abc.tex" nil)
>   find-file-noselect("/tmp/abc.tex" nil nil t)
>   find-file("/tmp/abc.tex" t)
>   call-interactively(find-file nil nil)
>   command-execute(find-file)
> ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
>
> Emacs  : GNU Emacs 24.5.1 (x86_64-unknown-freebsd10.3, GTK+ Version 3.18.8)
>  of 2016-04-08 on freebsd.vmware
> Package: 11.89
>
> current state:
> ==============
> (setq
>  AUCTeX-date "2015-11-13"
>  window-system 'x
>  LaTeX-version "2e"
>  TeX-style-path '("/usr/local/var/auctex"
>                   "/usr/local/share/emacs/site-lisp/auctex/style"
>                   "/home/keita/.emacs.d/auctex/auto"
>                   "/home/keita/.emacs.d/auctex/style" "auto" "style")
>  TeX-auto-save nil
>  TeX-parse-self t
>  TeX-master t
>  TeX-command-list '(("jTeX" "%(PDF)jtex %`%S%(PDFout)%(mode)%' %t" TeX-run-TeX
>                      nil (plain-tex-mode) :help "Run NTT jTeX")
>                     ("jLaTeX" "%(PDF)jlatex %`%S%(PDFout)%(mode)%' %t"
>                      TeX-run-TeX nil (latex-mode) :help "Run NTT jLaTeX")
>                     ("pTeX" "%(PDF)ptex %(kanjiopt)%`%S%(PDFout)%(mode)%' %t"
>                      TeX-run-TeX nil (plain-tex-mode) :help "Run ASCII pTeX")
>                     ("pLaTeX"
>                      "%(PDF)platex %(kanjiopt)%`%S%(PDFout)%(mode)%' %t"
>                      TeX-run-TeX nil (latex-mode) :help "Run ASCII pLaTeX")
>                     ("Mendex" "mendex %(mendexkopt)%s" TeX-run-command nil t
>                      :help "Create index file with mendex")
>                     ("jBibTeX" "jbibtex %s" TeX-run-BibTeX nil t :help
>                      "Run jBibTeX")
>                     ("pBibTeX" "pbibtex %(kanjiopt)%s" TeX-run-BibTeX nil t
>                      :help "Run pBibTeX")
>                     ("-" "" ignore nil t)
>                     ("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 t :help
>                      "Run BibTeX")
>                     ("Biber" "biber %s" TeX-run-Biber nil t :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 t :help
>                      "Generate PostScript file")
>                     ("Dvips" "%(o?)dvips %d -o %f " TeX-run-dvips nil t :help
>                      "Convert DVI file to PostScript")
>                     ("Dvipdfmx" "dvipdfmx %d" TeX-run-dvipdfmx nil t :help
>                      "Convert DVI file to PDF with dvipdfmx")
>                     ("Ps2pdf" "ps2pdf %f" TeX-run-ps2pdf nil t :help
>                      "Convert PostScript file to PDF")
>                     ("Index" "%(makeindex) %s" TeX-run-index nil t :help
>                      "Run makeindex to create index file")
>                     ("Xindy" "texindy %s" TeX-run-command nil t :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")
>                     )
>  )
>
>
>
> _______________________________________________
> bug-auctex mailing list
> bug-auctex@gnu.org
> https://lists.gnu.org/mailman/listinfo/bug-auctex
>



^ permalink raw reply	[flat|nested] 4+ messages in thread

* bug#23329: 11.89; Unloading tex-site doesn't restore original functionality
  2016-04-21 16:22 ` bug#23329: 11.89; Unloading tex-site doesn't restore original functionality Mosè Giordano
@ 2016-04-22  6:49   ` Ikumi Keita
  2016-04-22 11:56     ` Stefan Monnier
  0 siblings, 1 reply; 4+ messages in thread
From: Ikumi Keita @ 2016-04-22  6:49 UTC (permalink / raw)
  To: Mosè Giordano; +Cc: 23329, emacs-devel

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

Hi all,

> tex-site.el defines `tex-site-unload-hook' which looks into
> `after-load-alist' to remove AUCTeX settings (it looks for
> `TeX-modes-set' function), but in Emacs 24 `after-load-alist' seems to
> have changed and there is no reference to `TeX-modes-set' anymore.  Do
> you have any idea how to unload AUCTeX in Emacs 24?

I think I have figured out why `unload-feature' doesn't work well.  The
story is as follows.

If `advice-add' is available, tex-site.el uses `advice-add' on the
original tex modes and doesn't touch `after-load-alist'.

The advices are added through the defcustom of `TeX-modes' when
tex-site.el is loaded, because it has the line
  :set 'TeX-modes-set
.  Thus `tex-site-unload-hook' should remove these advices when
`advice-add' is available.

I confirmed that the attached crude workaround makes unloading tex-site
to work as expected.

Regards,
Ikumi Keita


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: quick workaround --]
[-- Type: text/x-diff, Size: 706 bytes --]

diff -r 26f0a318aedb -r 125674651465 tex-site.el.in
--- a/tex-site.el.in	Tue Jun 03 14:10:34 2014 +0900
+++ b/tex-site.el.in	Fri Apr 22 15:22:33 2016 +0900
@@ -87,6 +87,8 @@
 
 (add-hook 'tex-site-unload-hook
 	  (lambda ()
+	    (if (fboundp 'advice-add)
+		(TeX-modes-set 'TeX-modes nil)
 	    (let ((list after-load-alist))
 	      (while list
 		;; Adapted copy of the definition of `assq-delete-all'
@@ -101,7 +103,7 @@
 			     (eq (car (car tail)) key))
 			(setq alist (delq (car tail) alist)))
 		    (setq tail (cdr tail))))
-		(setq list (cdr list))))
+		(setq list (cdr list)))))
 	    (setq load-path (delq TeX-lisp-directory load-path))))
 
 (defun TeX-modes-set (var value &optional update)



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: bug#23329: 11.89; Unloading tex-site doesn't restore original functionality
  2016-04-22  6:49   ` Ikumi Keita
@ 2016-04-22 11:56     ` Stefan Monnier
  2016-04-23 16:52       ` Mosè Giordano
  0 siblings, 1 reply; 4+ messages in thread
From: Stefan Monnier @ 2016-04-22 11:56 UTC (permalink / raw)
  To: emacs-devel; +Cc: bug-auctex

> I confirmed that the attached crude workaround makes unloading tex-site
> to work as expected.

Actually, can't the tex-site-unload-hook just always call (TeX-modes-set
'TeX-modes nil)?


        Stefan




^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: bug#23329: 11.89; Unloading tex-site doesn't restore original functionality
  2016-04-22 11:56     ` Stefan Monnier
@ 2016-04-23 16:52       ` Mosè Giordano
  0 siblings, 0 replies; 4+ messages in thread
From: Mosè Giordano @ 2016-04-23 16:52 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: Ikumi Keita, 23329-done, emacs-devel

Hi Stefan,

2016-04-22 13:56 GMT+02:00 Stefan Monnier <monnier@iro.umontreal.ca>:
>> I confirmed that the attached crude workaround makes unloading tex-site
>> to work as expected.
>
> Actually, can't the tex-site-unload-hook just always call (TeX-modes-set
> 'TeX-modes nil)?

No, it doesn't work, I just tested on Emacs 22-24.  Only the change
suggested by Ikumi works across all Emacs versions.  I installed his
patch and I'm closing the issue, thanks for your help anyway.

Bye,
Mosè



^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2016-04-23 16:52 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <18668.1461248711@localhost>
2016-04-21 16:22 ` bug#23329: 11.89; Unloading tex-site doesn't restore original functionality Mosè Giordano
2016-04-22  6:49   ` Ikumi Keita
2016-04-22 11:56     ` Stefan Monnier
2016-04-23 16:52       ` Mosè Giordano

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).