From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.blaine.gmane.org!not-for-mail From: Filipp Gunbin Newsgroups: gmane.emacs.bugs Subject: bug#18109: 24.4.50; `compilation-error-regexp-alist-alist': wrong regexp for Maven Date: Mon, 07 Dec 2020 13:41:09 +0300 Message-ID: References: <5FFB3461-E756-4C09-9BFE-E0F9C840E533@acm.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214"; logging-data="22345"; mail-complaints-to="usenet@ciao.gmane.io" User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/28.0.50 (darwin) Cc: Lars Ingebrigtsen , 18109@debbugs.gnu.org To: Mattias =?UTF-8?Q?Engdeg=C3=A5rd?= Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Mon Dec 07 11:42:36 2020 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane-mx.org Original-Received: from lists.gnu.org ([209.51.188.17]) by ciao.gmane.io with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1kmDyP-0005T4-IC for geb-bug-gnu-emacs@m.gmane-mx.org; Mon, 07 Dec 2020 11:42:29 +0100 Original-Received: from localhost ([::1]:35492 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1kmDyO-0003Ed-Ib for geb-bug-gnu-emacs@m.gmane-mx.org; Mon, 07 Dec 2020 05:42:28 -0500 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]:58314) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1kmDxy-0003EE-Az for bug-gnu-emacs@gnu.org; Mon, 07 Dec 2020 05:42:02 -0500 Original-Received: from debbugs.gnu.org ([209.51.188.43]:40991) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1kmDxy-0000kU-0a for bug-gnu-emacs@gnu.org; Mon, 07 Dec 2020 05:42:02 -0500 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1kmDxx-0000G1-Ur for bug-gnu-emacs@gnu.org; Mon, 07 Dec 2020 05:42:01 -0500 X-Loop: help-debbugs@gnu.org Resent-From: Filipp Gunbin Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Mon, 07 Dec 2020 10:42:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 18109 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: moreinfo Original-Received: via spool by 18109-submit@debbugs.gnu.org id=B18109.1607337709970 (code B ref 18109); Mon, 07 Dec 2020 10:42:01 +0000 Original-Received: (at 18109) by debbugs.gnu.org; 7 Dec 2020 10:41:49 +0000 Original-Received: from localhost ([127.0.0.1]:52537 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1kmDxk-0000Fa-PV for submit@debbugs.gnu.org; Mon, 07 Dec 2020 05:41:48 -0500 Original-Received: from wout5-smtp.messagingengine.com ([64.147.123.21]:46149) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1kmDxi-0000FN-Qe for 18109@debbugs.gnu.org; Mon, 07 Dec 2020 05:41:48 -0500 Original-Received: from compute2.internal (compute2.nyi.internal [10.202.2.42]) by mailout.west.internal (Postfix) with ESMTP id A070A455; Mon, 7 Dec 2020 05:41:40 -0500 (EST) Original-Received: from mailfrontend2 ([10.202.2.163]) by compute2.internal (MEProxy); Mon, 07 Dec 2020 05:41:40 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=fastmail.fm; h= from:to:cc:subject:in-reply-to:references:date:message-id :mime-version:content-type:content-transfer-encoding; s=fm1; bh= YdGMCsQ6IlyYyfsVNtptBGglpFmiogtBQYasLCTJ+z8=; b=YE2fO2C86dLbAkIN l45Qrmo4dhE0Jye4wKJno+VlNCxBM8cJ4pHyURFxjmAMtJNCfkBH+NDPLLmLSJxa PKVebRcJR3/7LgMcFCDFdbT+JhjaIm5EfsfLabztZ4wWqw2sgakTmjGxrze0OOmz Y7Ym3FUMPzgWwKr63DZUHhSBR6XBfSRnJ4rm1lOekLn0+3LZp3dPyJh7kLdlnXwF tKAeC2GLNJde4fiU+W98UWpdeDQyYcWuYFqicxhQv69Jl3o1thKyJsgd3TGCchmy DLZO5DK6oZR3XquK0brDpgCy3OQYz2twk8qrvhsFokByiVWoCx6w8Hr0PFZ2DRpn flgCyw== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-transfer-encoding:content-type :date:from:in-reply-to:message-id:mime-version:references :subject:to:x-me-proxy:x-me-proxy:x-me-sender:x-me-sender :x-sasl-enc; s=fm1; bh=YdGMCsQ6IlyYyfsVNtptBGglpFmiogtBQYasLCTJ+ z8=; b=LRhlPiOcr4UOZAWGDajWT/cGQpbaxX7Sy2APfGcDTrlLJvhnYEtgp5sia B+QK8+bLaXUVax8DA6ymMpCeE8YUh9B/g3hrdk4TcYr6zAnL8ewzszgvh+cfqxsO HQqZFA4EzG6dpZzQewY7UBzobp9OWS4ejzNLIMjsZwQW+CTpXpYolEk0tCkbv1vI Sj9B9poE88gyRL19ZGv9Wt+ZiqfUNaIRWRxtWNNyuu31WdbG0aZLBMqSRW8k4Zdm Xm7L9kYZ+/b/1HOvewVX6jhXKf1UWeJQoxpNple9iWlpdMYbNiRJQ+3+uxZfDtqf kT3S3rttLknPM5LOxCo+op6ciQbww== X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedujedrudejgedgudekucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhephffvufgjfhgffffkgggtgfesthhqredttderjeenucfhrhhomhephfhilhhi phhpucfiuhhnsghinhcuoehfghhunhgsihhnsehfrghsthhmrghilhdrfhhmqeenucggtf frrghtthgvrhhnpeekvdekudelvddtvdelveejgeetudfgvdelgfeftefgkeejheehvefh gfetffeigfenucfkphepkeefrddutddvrddvtdefrdduieelnecuvehluhhsthgvrhfuih iivgeptdenucfrrghrrghmpehmrghilhhfrhhomhepfhhguhhnsghinhesfhgrshhtmhgr ihhlrdhfmh X-ME-Proxy: Original-Received: from fgunbin.local (unknown [83.102.203.169]) by mail.messagingengine.com (Postfix) with ESMTPA id 5E2891080057; Mon, 7 Dec 2020 05:41:39 -0500 (EST) In-Reply-To: ("Mattias =?UTF-8?Q?Engdeg=C3=A5rd?="'s message of "Sun, 6 Dec 2020 16:05:20 +0100") X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-BeenThere: bug-gnu-emacs@gnu.org List-Id: "Bug reports for GNU Emacs, the Swiss army knife of text editors" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Original-Sender: "bug-gnu-emacs" Xref: news.gmane.io gmane.emacs.bugs:195216 Archived-At: On 06/12/2020 16:05 +0100, Mattias Engdeg=C3=A5rd wrote: > Thus it looks like we can expect exactly one space each after the first a= nd second colon and after the comma, instead of zero-or-more spaces (the '?= ?' comments above). As a Gradle user, can you confirm this? > > The way the pattern is written makes it prone to matching other messages = entirely or partly, with potential negative consequences for correctness, p= erformance or both. It was me who put there those quantifiers, and I don't object to making the regexps stricter. But, we just need to be aware that Java tools usually don't expect the output to be parsed. Like, an IDE uses Gradle's API to run it, and Gradle uses compiler API to compile - this way none of them have to parse anything. So they output something that can be parsed, yes, but the format could change at any time. That is why I'm more inclined to making regexps more _lax_, not the other way around (and fix the problems with them once they appear). Filipp