Subject: bug#20556: closed (Re: bug#20556: [PATCH] compilation mode highlights Maven compilation warnings as errors)

From: [email-masked] (GNU bug Tracking System)
Date: Mon, 18 May 2015 21:06:04 +0000

Your bug report

#20556: [PATCH] compilation mode highlights Maven compilation warnings as errors

which was filed against the emacs package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to [email-masked].

-- 
20556: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=20556
GNU Bug Tracking System
Contact [email-masked] with problems

Subject: Re: bug#20556: [PATCH] compilation mode highlights Maven compilation warnings as errors
From: Dmitry Gutov <[email-masked]>
To: Paul Pogonyshev <[email-masked]>
Cc: [email-masked]
Date: Tue, 19 May 2015 00:04:55 +0300
List-Id: <debbugs-submit.debbugs.gnu.org>

On 05/18/2015 10:25 AM, Paul Pogonyshev wrote:
> This is an excerpt of Maven 3.1 output on a project, where I
> artificially added some errors to the source code:

Thanks. I've pushed your patch.

Subject: [PATCH] compilation mode highlights Maven compilation warnings as errors
From: Paul Pogonyshev <[email-masked]>
To: [email-masked]
Date: Tue, 12 May 2015 16:04:58 +0200
List-Id: <debbugs-submit.debbugs.gnu.org>

Trivial patch follows. Might be not generic enough, but certainly an
improvement in my case.

Paul

--- /tmp/olBiri_compile.el    2015-05-12 16:02:38.958969722 +0200
+++ lisp/progmodes/compile.el    2015-05-12 16:02:36.838967609 +0200
@@ -216,7 +216,7 @@ of[ \t]+\"?\\([a-zA-Z]?:?[^\":\n]+\\)\"?
     ;; due to matching filenames via \\(.*?\\).  This might be faster.
     (maven
      ;; Maven is a popular free software build tool for Java.
-     "\\([^ \n]\\(?:[^\n :]\\| [^-/\n]\\|:[^
\n]\\)*?\\):\\[\\([0-9]+\\),\\([0-9]+\\)\\] " 1 2 3)
+     "\\(\\[WARNING\\] *\\)?\\([^ \n]\\(?:[^\n :]\\| [^-/\n]\\|:[^
\n]\\)*?\\):\\[\\([0-9]+\\),\\([0-9]+\\)\\] " 2 3 4 (1))

     (jikes-line
      "^ *\\([0-9]+\\)\\.[ \t]+.*\n +\\(<-*>\n\\*\\*\\*
\\(?:Error\\|Warnin\\(g\\)\\)\\)"