MantisBT - CMake
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0012627CMakeCTestpublic2011-12-15 05:052012-05-09 15:26
ReporterNils Gladitz 
Assigned ToBrad King 
PrioritynormalSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOSOS Version
Product VersionCMake 2.8.4 
Target VersionCMake 2.8.7Fixed in VersionCMake 2.8.7 
Summary0012627: CTest does not match catastrophic error produced by intel c++ 11.1
DescriptionI've got errors of the form:
C:\some\dir\source.cpp(17): catastrophic error: could not open source file "some_header.h"

Which CTest does not detect as an error.
The closest match in cmCTestBuildHandler.cxx seems to be:
"([^:]+)\\(([^\\)]+)\\) : (error|fatal error|catastrophic error)"

I tried "([^:]+)\\(([^\\)]+)\\): (error|fatal error|catastrophic error)" which seems to work (I removed the space between ')' and ':').

Can this space be made optional?
e.g. "([^:]+)\\(([^\\)]+)\\): ?(error|fatal error|catastrophic error)"
Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2011-12-15 05:05Nils GladitzNew Issue
2011-12-15 08:47Brad KingNote Added: 0027987
2011-12-15 09:08Nils GladitzNote Added: 0027988
2011-12-15 09:40Brad KingNote Added: 0027989
2011-12-15 09:40Brad KingStatusnew => resolved
2011-12-15 09:40Brad KingResolutionopen => fixed
2011-12-15 09:40Brad KingAssigned To => Brad King
2011-12-16 17:27David ColeFixed in Version => CMake 2.8.7
2011-12-16 17:27David ColeTarget Version => CMake 2.8.7
2012-05-09 15:26David ColeNote Added: 0029453
2012-05-09 15:26David ColeStatusresolved => closed

Notes
(0027987)
Brad King   
2011-12-15 08:47   
Perhaps the space on each side of the ':' can be made optional:

  "([^:]+)\\(([^\\)]+)\\) ?: ?(error|fatal error|catastrophic error)"

If you compile CMake with that does it pick up the error you need?
(0027988)
Nils Gladitz   
2011-12-15 09:08   
I copied the LastBuild_*.log created on the windows client to my system and did:
grep -P "([^:]+)\\(([^\\)]+)\\) ?: ?(error|fatal error|catastrophic error)"

Which seems to work.
Hope that is close enough(?)
(0027989)
Brad King   
2011-12-15 09:40   
After reading the sample message more carefully I realized we only need the space before the : to be optional. Originally I was confused by a typo in your "e.g." example which shows the space after the : optional. Anyway, this should fix it:

http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=56f29f48 [^]
(0029453)
David Cole   
2012-05-09 15:26   
Closing resolved issues that have not been updated in more than 4 months.