unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#39981: [PATCH] compilation-error-regexp-alist-alist: Support gulp.js
@ 2020-03-08  9:13 Štěpán Němec
  2020-03-08 16:07 ` bug#39981: #39981: " Mattias Engdegård
  2020-03-08 22:10 ` Štěpán Němec
  0 siblings, 2 replies; 7+ messages in thread
From: Štěpán Němec @ 2020-03-08  9:13 UTC (permalink / raw)
  To: 39981

* lisp/progmodes/compile.el (compilation-error-regexp-alist-alist):
Add support for gulp.js (<https://gulpjs.com/>) error format.
---
 lisp/progmodes/compile.el            | 4 ++++
 test/lisp/progmodes/compile-tests.el | 5 ++++-
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/lisp/progmodes/compile.el b/lisp/progmodes/compile.el
index 455f181f50..c2b7fb7e7b 100644
--- a/lisp/progmodes/compile.el
+++ b/lisp/progmodes/compile.el
@@ -249,6 +249,10 @@ compilation-error-regexp-alist-alist
        "(\\([0-9]+\\), *\\([0-9]+\\))")     ;line, column
      2 3 4 (1))
 
+    ;; https://gulpjs.com/
+    (gulp
+     "^\\(/.+\\)\n *\\([0-9]+\\):\\([0-9]+\\) +error " 1 2 3)
+
     (iar
      "^\"\\(.*\\)\",\\([0-9]+\\)\\s-+\\(?:Error\\|Warnin\\(g\\)\\)\\[[0-9]+\\]:"
      1 2 nil (3))
diff --git a/test/lisp/progmodes/compile-tests.el b/test/lisp/progmodes/compile-tests.el
index 75962566f1..57cbb2dcb0 100644
--- a/test/lisp/progmodes/compile-tests.el
+++ b/test/lisp/progmodes/compile-tests.el
@@ -138,6 +138,9 @@ compile-tests--test-regexps-data
      1 nil 10 "arrayclash.f")
     ("Nonportable usage near line 31 col 9 file assign.f: mixed default and explicit"
      24 9 31 "assign.f")
+    ;; https://gulpjs.com/
+    ("/home/user/directory/path/to/some/file.js\n  39:25  error  Extra semicolon  semi"
+     1 25 39 "/home/user/directory/path/to/some/file.js")
     ;; iar
     ("\"foo.c\",3  Error[32]: Error message"
      1 nil 3 "foo.c")
@@ -431,7 +434,7 @@ compile-test-error-regexps
           (compilation-num-warnings-found 0)
           (compilation-num-infos-found 0))
       (mapc #'compile--test-error-line compile-tests--test-regexps-data)
-      (should (eq compilation-num-errors-found 93))
+      (should (eq compilation-num-errors-found 94))
       (should (eq compilation-num-warnings-found 36))
       (should (eq compilation-num-infos-found 26)))))
 
-- 
2.25.1






^ permalink raw reply related	[flat|nested] 7+ messages in thread

* bug#39981: #39981: [PATCH] compilation-error-regexp-alist-alist: Support gulp.js
  2020-03-08  9:13 bug#39981: [PATCH] compilation-error-regexp-alist-alist: Support gulp.js Štěpán Němec
@ 2020-03-08 16:07 ` Mattias Engdegård
  2020-03-08 16:35   ` Štěpán Němec
  2020-03-08 22:10 ` Štěpán Němec
  1 sibling, 1 reply; 7+ messages in thread
From: Mattias Engdegård @ 2020-03-08 16:07 UTC (permalink / raw)
  To: Štěpán Němec; +Cc: 39981

Thank you! However, it's another rather odd message format, and in the already crowded compilation-error-regexp-alist-alist, it risks slowing down matching quite a bit and even produce false matches.

Do you think it would be possible to ask the Gulp authors to switch to the standard GNU message format? Their non-Emacs users would benefit as well, since that format is widely used and understood by other tools.






^ permalink raw reply	[flat|nested] 7+ messages in thread

* bug#39981: #39981: [PATCH] compilation-error-regexp-alist-alist: Support gulp.js
  2020-03-08 16:07 ` bug#39981: #39981: " Mattias Engdegård
@ 2020-03-08 16:35   ` Štěpán Němec
  2020-03-08 20:17     ` Mattias Engdegård
  0 siblings, 1 reply; 7+ messages in thread
From: Štěpán Němec @ 2020-03-08 16:35 UTC (permalink / raw)
  To: Mattias Engdegård; +Cc: 39981

On Sun, 08 Mar 2020 17:07:02 +0100
Mattias Engdegård wrote:

> Do you think it would be possible to ask the Gulp authors to switch to
> the standard GNU message format? Their non-Emacs users would benefit
> as well, since that format is widely used and understood by other
> tools.

I don't know anything about the project's development, but now I that
looked, the main repository seems to be this (MIT license):

  https://github.com/gulpjs/gulp

Furthemore, from https://github.com/gulpjs/gulp/blob/master/CONTRIBUTING.md:

"GitHub issues are the primary way for communicating about specific
proposed changes to this project."

I tried searching both open and closed issues and haven't found anything
relevant to this. Would you be willing to open an issue there and
describe the problem as you perceive it, or shall I?

-- 
Štěpán





^ permalink raw reply	[flat|nested] 7+ messages in thread

* bug#39981: #39981: [PATCH] compilation-error-regexp-alist-alist: Support gulp.js
  2020-03-08 16:35   ` Štěpán Němec
@ 2020-03-08 20:17     ` Mattias Engdegård
  2020-03-08 21:49       ` Štěpán Němec
  0 siblings, 1 reply; 7+ messages in thread
From: Mattias Engdegård @ 2020-03-08 20:17 UTC (permalink / raw)
  To: Štěpán Němec; +Cc: 39981

8 mars 2020 kl. 17.35 skrev Štěpán Němec <stepnem@gmail.com>:

> I tried searching both open and closed issues and haven't found anything
> relevant to this. Would you be willing to open an issue there and
> describe the problem as you perceive it, or shall I?

Oh, would you do it? I'm not a user of Gulp at all. I couldn't even find the code responsible for emitting errors in there!






^ permalink raw reply	[flat|nested] 7+ messages in thread

* bug#39981: #39981: [PATCH] compilation-error-regexp-alist-alist: Support gulp.js
  2020-03-08 20:17     ` Mattias Engdegård
@ 2020-03-08 21:49       ` Štěpán Němec
  2020-03-08 21:53         ` Mattias Engdegård
  0 siblings, 1 reply; 7+ messages in thread
From: Štěpán Němec @ 2020-03-08 21:49 UTC (permalink / raw)
  To: Mattias Engdegård; +Cc: 39981

close 39981
thanks

On Sun, 08 Mar 2020 21:17:51 +0100
Mattias Engdegård wrote:

> 8 mars 2020 kl. 17.35 skrev Štěpán Němec <stepnem@gmail.com>:
>
>> I tried searching both open and closed issues and haven't found anything
>> relevant to this. Would you be willing to open an issue there and
>> describe the problem as you perceive it, or shall I?
>
> Oh, would you do it? I'm not a user of Gulp at all. I couldn't even find the code responsible for emitting errors in there!

OK, after some fumbling through the maze of the various gulp module
repositories I realized that this particular output format is just the
default of one of the plugins, namely gulp-eslint, eslint being a
standalone JavaScript linter which is easily configurable, e.g. choosing
the "unix" formatter gives the standard "/pa/th:line:col:" format:

  https://eslint.org/docs/user-guide/formatters/#unix

So this all was just a false alarm and me being a fool (arguably
gulp-eslint could choose saner default, but in any case there is nothing
to fix in Emacs). Sorry for wasting your time!

I am hereby closing the bug report, thank you.

-- 
Štěpán





^ permalink raw reply	[flat|nested] 7+ messages in thread

* bug#39981: #39981: [PATCH] compilation-error-regexp-alist-alist: Support gulp.js
  2020-03-08 21:49       ` Štěpán Němec
@ 2020-03-08 21:53         ` Mattias Engdegård
  0 siblings, 0 replies; 7+ messages in thread
From: Mattias Engdegård @ 2020-03-08 21:53 UTC (permalink / raw)
  To: Štěpán Němec; +Cc: 39981

8 mars 2020 kl. 22.49 skrev Štěpán Němec <stepnem@gmail.com>:

> So this all was just a false alarm and me being a fool (arguably
> gulp-eslint could choose saner default, but in any case there is nothing
> to fix in Emacs). Sorry for wasting your time!

For the record, the investigation you just made was anything but the work of a fool. Well done!






^ permalink raw reply	[flat|nested] 7+ messages in thread

* bug#39981: #39981: [PATCH] compilation-error-regexp-alist-alist: Support gulp.js
  2020-03-08  9:13 bug#39981: [PATCH] compilation-error-regexp-alist-alist: Support gulp.js Štěpán Němec
  2020-03-08 16:07 ` bug#39981: #39981: " Mattias Engdegård
@ 2020-03-08 22:10 ` Štěpán Němec
  1 sibling, 0 replies; 7+ messages in thread
From: Štěpán Němec @ 2020-03-08 22:10 UTC (permalink / raw)
  To: 39981-done


False alarm; no Emacs action necessary.

-- 
Štěpán





^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2020-03-08 22:10 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-03-08  9:13 bug#39981: [PATCH] compilation-error-regexp-alist-alist: Support gulp.js Štěpán Němec
2020-03-08 16:07 ` bug#39981: #39981: " Mattias Engdegård
2020-03-08 16:35   ` Štěpán Němec
2020-03-08 20:17     ` Mattias Engdegård
2020-03-08 21:49       ` Štěpán Němec
2020-03-08 21:53         ` Mattias Engdegård
2020-03-08 22:10 ` Štěpán Němec

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).