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
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
==
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
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
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
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