[PATCH] D25869: [Driver] Add unit tests for Distro detection

2016-11-28 Thread Michał Górny via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL288062: [Driver] Add unit tests for Distro detection (authored by mgorny). Changed prior to commit: https://reviews.llvm.org/D25869?vs=78625&id=79444#toc Repository: rL LLVM https://reviews.llvm.org

[PATCH] D25869: [Driver] Add unit tests for Distro detection

2016-11-19 Thread Michał Górny via cfe-commits
mgorny updated this revision to Diff 78625. mgorny added a comment. Added a test for Exherbo. https://reviews.llvm.org/D25869 Files: unittests/Driver/CMakeLists.txt unittests/Driver/DistroTest.cpp Index: unittests/Driver/DistroTest.cpp ==

[PATCH] D25869: [Driver] Add unit tests for Distro detection

2016-11-18 Thread Michał Górny via cfe-commits
mgorny added a comment. In https://reviews.llvm.org/D25869#56, @bruno wrote: > LGTM! Please add this before https://reviews.llvm.org/D26850, which should > contain a testcase on top of this! I should probably also copy here that as I've mentioned on https://reviews.llvm.org/D26850, this a

[PATCH] D25869: [Driver] Add unit tests for Distro detection

2016-11-18 Thread Bruno Cardoso Lopes via cfe-commits
bruno added a comment. LGTM! Please add this before https://reviews.llvm.org/D26850, which should contain a testcase on top of this! Thanks https://reviews.llvm.org/D25869 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.o

[PATCH] D25869: [Driver] Add unit tests for Distro detection

2016-11-18 Thread Michał Górny via cfe-commits
mgorny updated this revision to Diff 78506. mgorny added a comment. Added tests for SUSE and Arch Linux (and found a bug for the former, https://reviews.llvm.org/D26850). https://reviews.llvm.org/D25869 Files: unittests/Driver/CMakeLists.txt unittests/Driver/DistroTest.cpp Index: unittest

[PATCH] D25869: [Driver] Add unit tests for Distro detection

2016-11-18 Thread Michał Górny via cfe-commits
mgorny retitled this revision from "[Driver] Add unit tests for DetectDistro()" to "[Driver] Add unit tests for Distro detection". mgorny updated the summary for this revision. mgorny updated this revision to Diff 78494. mgorny added a comment. Ok, here are the tests updated for the new API. I'll