Fixes problem with flymake-xml-init.

flymake / emacs-flymake

Continuous syntax checking for Emacs. Fork to add max parallel invocations and other bug fixes.

http://www.illusori.co.uk/projects/emacs-flymake/

GNU General Public License v3.0

184 stars 36 forks source link

Fixes problem with flymake-xml-init. #8

Closed marcinant closed 12 years ago

marcinant commented 12 years ago

Current version throws exception on xmlstarlet 1.3.0 and emacs 24.2.1 (xmlstarlet --version 1.3.0 compiled against libxml2 2.7.8, linked with 20708 compiled against libxslt 1.1.26, linked with 10126)

"Configuration error has occured while running (xmlstarlet val /file_flymake.html). Flymake will be switched OFF"

This patch fixes this and now Flymake works properly with html files in nxml mode.

illusori commented 12 years ago

I've only applied the change to add -e (in https://github.com/illusori/emacs-flymake/commit/d9476016c368d94b0dc9c58f30f05dd1dba1c065) to the arguments, I couldn't see the reasoning behind changing the temp file handling. Let me know if there was a reason and I'll revisit that decision.

Sorry it's taken me so long to find time to look at this, and thanks for the patch. :)