Skip to content

Commit

Permalink
extend file regexp to match files with embedded spaces.
Browse files Browse the repository at this point in the history
  • Loading branch information
@pnkfelix
pnkfelix committed Aug 26, 2013
1 parent fe7092d commit 9d8897b
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/etc/emacs/rust-mode.el
View file
Original file line number Diff line number Diff line change
Expand Up @@ -229,7 +229,7 @@ The initializer is `DEFAULT-TAB-WIDTH'.")
;; regexp (which is broken on a few edge cases), add our own 'rust
;; compilation error regexp and use it instead.
(defvar rustc-compilation-regexps
(let ((file "\\([^ \n]+\\)")
(let ((file "^\\([^\n]+\\)")
(start-line "\\([0-9]+\\)")
(start-col "\\([0-9]+\\)")
(end-line "\\([0-9]+\\)")
Expand Down

5 comments on commit 9d8897b

@bors
Copy link
Contributor

@bors bors commented on 9d8897b Aug 27, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

saw approval from graydon
at pnkfelix@9d8897b

@bors
Copy link
Contributor

@bors bors commented on 9d8897b Aug 27, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merging pnkfelix/rust/fsk-issue6887-fix-emacs-compilation-regexp = 9d8897b into auto

@bors
Copy link
Contributor

@bors bors commented on 9d8897b Aug 27, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pnkfelix/rust/fsk-issue6887-fix-emacs-compilation-regexp = 9d8897b merged ok, testing candidate = c0aadfd

@bors
Copy link
Contributor

@bors bors commented on 9d8897b Aug 27, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

all tests pass:
success: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/1117
success: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/1120
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-c/builds/226
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/226
success: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/1135
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-c/builds/226
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/226
success: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/1135
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-c/builds/226
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/226
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android/builds/308
success: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/1120
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-c/builds/226
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/226
success: http://buildbot.rust-lang.org/builders/auto-bsd-64-opt/builds/905

@bors
Copy link
Contributor

@bors bors commented on 9d8897b Aug 27, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fast-forwarding master to auto = c0aadfd

Please sign in to comment.