31021Re: patch for compiler/tidy.vim
- Feb 1, 2003On Sun, 2 Feb 2003, Doug Kearns wrote:
> On Fri, Jan 31, 2003 at 03:30:48PM -0500, Mark Stosberg wrote:Thanks for the response Doug. This works better in the sense that it
> > Hello,
> > I'm currently using vim 6.1.262 with a version of "tidy" from the new
> > project at http://tidy.sf.net/ that has taken over the original author's
> > work.
> > However, I found that the "compiler/tidy.vim" file distributed with vim
> > was not fully functional-- the errorfmt string was not working.
> > After researching this some, it seemed patching this file was the best
> > solution. Here's the patch, which seems to work for me.
> > The tidy version I'm using is described as:
> > "HTML Tidy for FreeBSD released on 1st January 2003"
> The problem is that there is currently Tidy Classic and Tidy Library and
> neither of these have been released, in a traditional sense in over two
> Apparently, the Compile Farm builds have recently been moved over to the
> library version, which has introduced a new option '--gnu-emacs-file' to
> specify the filename used in the errorformat. If this option is not
> present then '--gnu-emacs' is _silently_ ignored. I'm currently trying
> to find out what this new option's purpose is and will update tidy.vim
> when the 'situation' is clarified. Until then you can change the makeprg
> line to:
> setlocal makeprg=tidy\ -quiet\ -errors\ --gnu-emacs\ yes\ --gnu-emacs-file\ %\ %
causes output to be produced that is almost identical the output format
before. There is one minor difference: There is no space between the last
colon and "Error" or "Warning" as there was before. This patch against
the tidy.vim distributed with Vim illustrates this:
--- tidy.vim.orig Fri Jan 31 15:11:47 2003
+++ tidy.vim Sat Feb 1 12:52:46 2003
@@ -16,9 +16,9 @@
setlocal makeprg=tidy\ -errors\ --gnu-emacs\ yes\ %
- setlocal makeprg=tidy\ -quiet\ -errors\ --gnu-emacs\ yes\ %
+ setlocal makeprg=tidy\ -quiet\ -errors\ --gnu-emacs\ yes\ --gnu-emacs-file\ %\ %
-" sample warning: foo.html:8:1: Warning: inserting missing 'foobar' element
-" sample error: foo.html:9:2: Error: <foobar> is not recognized!
-setlocal errorformat=%f:%l:%c:\ Error:%m,%f:%l:%c:\ Warning:%m,%-G%.%#
+" sample warning: foo.html:8:1:Warning: inserting missing 'foobar' element
+" sample error: foo.html:9:2:Error: <foobar> is not recognized!
Perhaps it should be considered a bug in tidy that the format changed?
- << Previous post in topic Next post in topic >>