Skip to content

Port CTest's log scraping logic to Spack#5561

Merged
tgamblin merged 1 commit intodevelopfrom
features/ctest-log-parsing
Oct 1, 2017
Merged

Port CTest's log scraping logic to Spack#5561
tgamblin merged 1 commit intodevelopfrom
features/ctest-log-parsing

Conversation

@tgamblin
Copy link
Copy Markdown
Member

@tgamblin tgamblin commented Oct 1, 2017

@aprokop: this doesn't cause a regression on your error message from #5236, does it?
@pramodskumbhar: FYI

@tgamblin tgamblin added build-environment tests General test capability(ies) labels Oct 1, 2017

This file takes the regexes verbatim from there and adds some parsing
algorithms that duplicate the way CTest scrapes log files. To keep this
up to date with CTest, just sure the ``*_matches`` and ``*_exceptions``
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

...just make sure...

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

fixed

@tgamblin tgamblin force-pushed the features/ctest-log-parsing branch from f1874fe to c4cc344 Compare October 1, 2017 04:27
- This steals the magic regular expressions that CTest uses to parse log
  files and addds them to Spack.  See here:

  https://github.com/Kitware/CMake/blob/master/Source/CTest/cmCTestBuildHandler.cxx

  These are BSD licensed, so the port is in `externa/ctest_log_parser.py`

- We currently use these to do better filtering of errors from build
  output.  Plan is to use them to generate good CDash output.
@tgamblin tgamblin merged commit 29ca18e into develop Oct 1, 2017
@junghans junghans deleted the features/ctest-log-parsing branch October 5, 2017 03:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build-environment tests General test capability(ies)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants