Make creates new file #4

Closed
seanhess opened this Issue Nov 7, 2012 · 4 comments

Comments

Projects
None yet
2 participants
@seanhess

seanhess commented Nov 7, 2012

I think errorformat isn't matching correctly.

:make %

/Users/seanhess/sandbox/typescript/main.ts (62,15): Expected ';'
"~/sandbox/typescript/main.ts " [New File]  

It opens a new file. You can navigate back, but you lose your undo history and everything. See http://stackoverflow.com/questions/8141113/vims-make-creates-a-new-file

@seanhess

This comment has been minimized.

Show comment
Hide comment
@seanhess

seanhess Nov 7, 2012

Looks like errors can come in 2 formats. One with the space, one without. This matches the one with the space.

errorformat= %#%f (%l\,%c): %m

But I can't figure out how to get one to match BOTH.

seanhess commented Nov 7, 2012

Looks like errors can come in 2 formats. One with the space, one without. This matches the one with the space.

errorformat= %#%f (%l\,%c): %m

But I can't figure out how to get one to match BOTH.

@seanhess

This comment has been minimized.

Show comment
Hide comment
@seanhess

seanhess Nov 7, 2012

Ok, this works for me, but I don't understand what the %# and the space at the beginning is for, so I don't know what else it might break:

CompilerSet errorformat=\ %#%f\ (%l\\\,%c):\ %m
CompilerSet errorformat+=%f(%l\\\,%c):\ %m

seanhess commented Nov 7, 2012

Ok, this works for me, but I don't understand what the %# and the space at the beginning is for, so I don't know what else it might break:

CompilerSet errorformat=\ %#%f\ (%l\\\,%c):\ %m
CompilerSet errorformat+=%f(%l\\\,%c):\ %m
@leafgarland

This comment has been minimized.

Show comment
Hide comment
@leafgarland

leafgarland Nov 7, 2012

Owner

%# somewhat oddly ends up being * which gives \ * which is a regex for any amount of spaces including 0. The same trick can be used to deal with the odd space after the file name.

Owner

leafgarland commented Nov 7, 2012

%# somewhat oddly ends up being * which gives \ * which is a regex for any amount of spaces including 0. The same trick can be used to deal with the odd space after the file name.

@seanhess

This comment has been minimized.

Show comment
Hide comment
@seanhess

seanhess Nov 7, 2012

Aha, it makes sense now. Good job! It's actually getting usable now! Now I
just need to finish figuring out how to get these stupid AMD modules to
work nicely with the ambient declarations.

On Wed, Nov 7, 2012 at 2:26 PM, Leaf Garland notifications@github.comwrote:

%# somewhat oddly ends up being * which gives \ * which is a regex for
any amount of spaces including 0. The same trick can be used to deal with
the odd space after the file name.


Reply to this email directly or view it on GitHubhttps://github.com/leafgarland/typescript-vim/issues/4#issuecomment-10165787.

seanhess commented Nov 7, 2012

Aha, it makes sense now. Good job! It's actually getting usable now! Now I
just need to finish figuring out how to get these stupid AMD modules to
work nicely with the ambient declarations.

On Wed, Nov 7, 2012 at 2:26 PM, Leaf Garland notifications@github.comwrote:

%# somewhat oddly ends up being * which gives \ * which is a regex for
any amount of spaces including 0. The same trick can be used to deal with
the odd space after the file name.


Reply to this email directly or view it on GitHubhttps://github.com/leafgarland/typescript-vim/issues/4#issuecomment-10165787.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment