Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 320186 - Support MSbuild error pattern in VC error parser
Summary: Support MSbuild error pattern in VC error parser
Status: RESOLVED FIXED
Alias: None
Product: CDT
Classification: Tools
Component: cdt-build (show other bugs)
Version: 7.0   Edit
Hardware: PC Windows All
: P3 normal (vote)
Target Milestone: 10.3.0   Edit
Assignee: Marc-André Laperle CLA
QA Contact: Marc-André Laperle CLA
URL:
Whiteboard:
Keywords:
: 552455 572436 (view as bug list)
Depends on:
Blocks:
 
Reported: 2010-07-18 02:36 EDT by Marc-André Laperle CLA
Modified: 2021-04-02 22:47 EDT (History)
2 users (show)

See Also:

Attachments
Patch (1.04 KB, patch)
2010-07-18 02:36 EDT, Marc-André Laperle CLA 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Marc-André Laperle CLA 2010-07-18 02:36:53 EDT 
Created attachment 174570 [details]
Patch

Tested with VS 2008 and .NET 3.5

Building with msbuild:

c:\foo\main.cpp(5): error C2039

with devenv, cl:

c:\foo\main.cpp(5) : error C2039
Comment 1 Andrew Gvozdev CLA 2010-07-18 10:09:03 EDT 
A small patch, huh? I have some comments:
- VCerrorParser should be converted to RegexErrorParser style. That is not hard to do, it just takes definition in plugin.xml. Could you do it? Take a look at GLDErrorParser for example.
- Those error parsers need JUnit test cases. I don't see one for VCErrorParser. Could you create one, the simplest example would be GLDErrorParserTests in org.eclipse.cdt.core.tests/misc/org.eclipse.cdt.core.internal.errorparsers.tests
- If you work with VS, perhaps you could take a look at bug 79704, bug 157625, bug 161625, bug 246170?
Comment 2 Marc-André Laperle CLA 2010-07-20 00:56:54 EDT 
Thanks for the comments. I will work on all three points and also test with multiple versions of VS. I do wish for full support of VS and this might be the easiest place to start.
Comment 3 Andrew Gvozdev CLA 2011-03-11 18:18:42 EST 
Passing the task to sit on your plate.
Comment 4 Marc-André Laperle CLA 2020-08-23 09:49:15 EDT 
*** Bug 552455 has been marked as a duplicate of this bug. ***
Comment 5 Marc-André Laperle CLA 2021-04-02 12:25:49 EDT 
*** Bug 572436 has been marked as a duplicate of this bug. ***
Comment 6 Marc-André Laperle CLA 2021-04-02 12:27:59 EDT 
I never got to doing the change to regex parser and tests as it was always very low priority yet people keep getting the issue (myself included) so I'll just commit the small fix which is very safe anyway.
Comment 7 Eclipse Genie CLA 2021-04-02 13:03:47 EDT 
New Gerrit change created: https://git.eclipse.org/r/c/cdt/org.eclipse.cdt/+/178792