Skip to content

Integrating test failures into QTCreator IDE build errors #46

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 22, 2013

Conversation

lijoantony
Copy link
Contributor

Some small changes to integrate the test run failures into QTCreator IDE build errors, there by showing the failures as issues in the Issue panel of the IDE.
more details: http://www.mail-archive.com/unittest-cpp-devel@lists.sourceforge.net/msg00367.html

@pjohnmeyer
Copy link
Member

Is there a way you can make this change additive instead of mutative? It may break integration with some other tools.

I may need to consider improving the mechanism for customizing the error format.

@lijoantony
Copy link
Contributor Author

Actually, after my change, the error format has become compliant to that of GCC. GCC error messages are written to stderr and has both line number and column number separated by colon. http://gcc.gnu.org/onlinedocs/gcc-4.7.3/gnat_ugn_unw/Output-and-Error-Message-Control.html . I guess, clang also follows this format.

Since GCC can be considered as the standard compiler on Linux/Mac, and all tools and IDEs on these platforms are expected to parse the error messages from gcc, I believe my change should not break any existing tools on these platforms. And, if this change is going to break any tool, that tool is already broken, as it cannot parse even the GCC error messages.

However, I am open for discussion. Do you have any suggestion?

pjohnmeyer added a commit that referenced this pull request Jul 22, 2013
Integrating test failures into QTCreator IDE build errors
@pjohnmeyer pjohnmeyer merged commit 9ad680a into unittest-cpp:master Jul 22, 2013
@pjohnmeyer
Copy link
Member

Nope, sounds good to me.

pjohnmeyer added a commit that referenced this pull request Apr 4, 2015
Integrating test failures into QTCreator IDE build errors
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants