patch to support maven in compile.el

patch to support maven in compile.el

[joakim]

Maven is a popular build tool for Java. Maven is free software.

--- compile.el	2007-07-23 19:38:52.000000000 +0200
+++ /home/joakim/.elisp/compile.el	2007-07-20 15:34:17.000000000 +0200
@@ -167,6 +167,9 @@
      "^[ \t]*\\[[^] \n]+\\][ \t]*\\([^: \n]+\\):\\([0-9]+\\):\\(?:\\([0-9]+\\):[0-9]+:[0-9]+:\\)?\
 \\( warning\\)?" 1 2 3 (4))
 
+    (maven
+     "\\(.*?\\):\\[\\([0-9]+\\),\\([0-9]+\\)\\]" 1 2 3)
+    
     (bash
      "^\\([^: \n\t]+\\): line \\([0-9]+\\):" 1 2)
 
-- 
Joakim Verona

Re: patch to support maven in compile.el

[Stefan Monnier]

> Maven is a popular build tool for Java. Maven is free software.

Can you ask them to change their syntax to the one recommended in the GNU
coding standards, i.e. <file>:<linenb>:<colnb> ?

This proliferation of formats is simply ridiculous.


        Stefan


> --- compile.el	2007-07-23 19:38:52.000000000 +0200
> +++ /home/joakim/.elisp/compile.el	2007-07-20 15:34:17.000000000 +0200
> @@ -167,6 +167,9 @@
>       "^[ \t]*\\[[^] \n]+\\][ \t]*\\([^: \n]+\\):\\([0-9]+\\):\\(?:\\([0-9]+\\):[0-9]+:[0-9]+:\\)?\
>  \\( warning\\)?" 1 2 3 (4))
 
> +    (maven
> +     "\\(.*?\\):\\[\\([0-9]+\\),\\([0-9]+\\)\\]" 1 2 3)
> +    
>      (bash
>       "^\\([^: \n\t]+\\): line \\([0-9]+\\):" 1 2)
 
> -- 
> Joakim Verona


> _______________________________________________
> Emacs-devel mailing list
> Emacs-devel@gnu.org
> http://lists.gnu.org/mailman/listinfo/emacs-devel

Re: patch to support maven in compile.el

[joakim]

Stefan Monnier <monnier@iro.umontreal.ca> writes:

>> Maven is a popular build tool for Java. Maven is free software.
>
> Can you ask them to change their syntax to the one recommended in the GNU
> coding standards, i.e. <file>:<linenb>:<colnb> ?
>
> This proliferation of formats is simply ridiculous.

I can try it. Would it be possible to add the rule in the mean time?


>
>
>         Stefan
>
>
>> --- compile.el	2007-07-23 19:38:52.000000000 +0200
>> +++ /home/joakim/.elisp/compile.el	2007-07-20 15:34:17.000000000 +0200
>> @@ -167,6 +167,9 @@
>>       "^[ \t]*\\[[^] \n]+\\][ \t]*\\([^: \n]+\\):\\([0-9]+\\):\\(?:\\([0-9]+\\):[0-9]+:[0-9]+:\\)?\
>>  \\( warning\\)?" 1 2 3 (4))
>  
>> +    (maven
>> +     "\\(.*?\\):\\[\\([0-9]+\\),\\([0-9]+\\)\\]" 1 2 3)
>> +    
>>      (bash
>>       "^\\([^: \n\t]+\\): line \\([0-9]+\\):" 1 2)
>  
>> -- 
>> Joakim Verona
>
>
>> _______________________________________________
>> Emacs-devel mailing list
>> Emacs-devel@gnu.org
>> http://lists.gnu.org/mailman/listinfo/emacs-devel

-- 
Joakim Verona

Re: patch to support maven in compile.el

[Richard Stallman]

    Can you ask them to change their syntax to the one recommended in the GNU
    coding standards, i.e. <file>:<linenb>:<colnb> ?

    This proliferation of formats is simply ridiculous.

I agree, but in the meantime, we may as well support their format.
Could you please install that patch?

Re: patch to support maven in compile.el

[Stefan Monnier]

>>> Maven is a popular build tool for Java. Maven is free software.
>> Can you ask them to change their syntax to the one recommended in the GNU
>> coding standards, i.e. <file>:<linenb>:<colnb> ?
>> This proliferation of formats is simply ridiculous.
> I can try it. Would it be possible to add the rule in the mean time?

Yes, done,


        Stefan

Re: patch to support maven in compile.el

[Stefan Monnier]

>     Can you ask them to change their syntax to the one recommended in the GNU
>     coding standards, i.e. <file>:<linenb>:<colnb> ?

>     This proliferation of formats is simply ridiculous.

> I agree, but in the meantime, we may as well support their format.
> Could you please install that patch?

Done,


        Stefan