GNU bug report logs - #24978
25.1; Bad interaction between nxml-auto-insert-xml-declaration-flag and archive-extract

Previous Next

Full log

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

From: Heinz Rommerskirchen <heinz <at> h-rommerskirchen.de>
To: bug-gnu-emacs <at> gnu.org
Subject: 25.1; Bad interaction between nxml-auto-insert-xml-declaration-flag
 and archive-extract
Date: Mon, 21 Nov 2016 14:41:24 +0100

If the variable nxml-auto-insert-xml-declaration-flag is set, an XML
declaration is inserted into non-empty XML files inside a zip archive.

To reproduce this bug create the valid XML file foo.xml containing
=== start foo.xml ===
<?xml version="1.0" encoding="utf-8"?>
<content>y</content>
=== end foo.xml ===
and pack it into a zip-archive:
$ zip -m bar.zip foo.xml
Now start "emacs -Q", evaluate
"(setq nxml-auto-insert-xml-declaration-flag 't)" in the *scratch*
buffer, visit bar.zip with C-x C-f, and press Enter on the entry
"foo.xml". foo.xml will be opened and an additional XML declaration is
inserted into the buffer rendering the file invalid and marking the
buffer as modified.

The following simple patch solved the problem for me.

=== start patch ====
--- nxml-mode-from-25.1.el	2016-11-20 19:45:43.339427354 +0100
+++ nxml-mode.el	2016-11-20 19:48:26.009625534 +0100
@@ -566,7 +566,8 @@
   ;; hook. The hook also had the drawback of overriding explicit user
   ;; instruction to save as some encoding other than utf-8.
   ;;(add-hook 'write-contents-hooks 'nxml-prepare-to-save)
-  (when (not (and (buffer-file-name) (file-exists-p (buffer-file-name))))
+  (when (and (not (and (buffer-file-name) (file-exists-p (buffer-file-name))))
+             (not archive-superior-buffer))
     (when (and nxml-default-buffer-file-coding-system
 	       (not (local-variable-p 'buffer-file-coding-system)))
       (setq buffer-file-coding-system nxml-default-buffer-file-coding-system))
=== end patch ===

-- 
Dr. Heinrich Rommerskirchen
Prof.-Schmid-Str. 41
82140 Olching
Allemagne

Tel. +49 8142 28787

Email heinz <at> h-rommerskirchen.de

Previous Next

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