unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: Daniel Colascione <dancol@dancol.org>
To: Filipp Gunbin <fgunbin@fastmail.fm>, Glenn Morris <rgm@gnu.org>
Cc: 18109@debbugs.gnu.org
Subject: bug#18109: 24.4.50; `compilation-error-regexp-alist-alist': wrong regexp for Maven
Date: Sun, 03 Aug 2014 08:12:23 -0700	[thread overview]
Message-ID: <53DE5157.9060909@dancol.org> (raw)
In-Reply-To: <867g2xvf75.fsf@W2139SPB.PlayTeam.ru>

[-- Attachment #1: Type: text/plain, Size: 1112 bytes --]

On 07/28/2014 05:30 AM, Filipp Gunbin wrote:
> Glenn,
> 
> On 26/07/2014 03:22 -0400, Glenn Morris wrote:
> 
>> Please explain why it is wrong, and show an example of what is is
>> supposed to match. The current one in etc/compilation.txt is:
>>
>> * maven 2.0.9
>>
>> symbol: maven
>>
>> FooBar.java:[111,53] no interface expected here
> 
> Oh yes, sorry for the bad report.
> 
> While the original regexp catches these errors:
> 
> D:\cygwin\root\my-project\src\main\java\my\project\controllers\MainController.java:[27,12] error: cannot find symbol
> [ERROR]   symbol:   class SomeService
>   location: class MainController
> 
> it does not catch these, which Maven emits sometimes:
> 
> [ERROR] D:\cygwin\root\my-project\src\main\java\my\project\controllers\MainController.java:[21,1] error: cannot find symbol
> [ERROR]   symbol: class Controller
> 
> My version catches both.
> 
> Also, the original regexp seems to be more complicated than it really
> should be.
> 
> Tested on Maven 2.2.1 and 3.0.4.
> 

Would you please consider converting this regexp to rx form?



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 884 bytes --]

  reply	other threads:[~2014-08-03 15:12 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-07-25 17:33 bug#18109: 24.4.50; `compilation-error-regexp-alist-alist': wrong regexp for Maven Filipp Gunbin
2014-07-26  7:22 ` Glenn Morris
2014-07-28 12:30   ` Filipp Gunbin
2014-08-03 15:12     ` Daniel Colascione [this message]
2020-09-09 11:16     ` Lars Ingebrigtsen
2020-12-03 14:59       ` Lars Ingebrigtsen
2020-12-04 18:11         ` Filipp Gunbin
2020-12-04 19:22 ` Mattias Engdegård
2020-12-05 22:21   ` Filipp Gunbin
2020-12-06  9:32     ` Mattias Engdegård
2020-12-06 14:22       ` Filipp Gunbin
2020-12-06 15:05         ` Mattias Engdegård
2020-12-06 15:25           ` Mattias Engdegård
2020-12-07 10:41           ` Filipp Gunbin
2020-12-07 13:49             ` Mattias Engdegård
2020-12-07 20:07               ` Filipp Gunbin
2020-12-09 18:41                 ` Mattias Engdegård
2020-12-10 13:12                   ` Filipp Gunbin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=53DE5157.9060909@dancol.org \
    --to=dancol@dancol.org \
    --cc=18109@debbugs.gnu.org \
    --cc=fgunbin@fastmail.fm \
    --cc=rgm@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).