This is an archive of the discontinued LLVM Phabricator instance.

Re-enable a test for non-Windows
AbandonedPublic

Authored by probinson on May 9 2019, 12:23 PM.

Details

Summary

Required teaching lit.cfg.py to distinguish Windows the same way other projects do.

Diff Detail

Event Timeline

probinson created this revision.May 9 2019, 12:23 PM
Herald added a project: Restricted Project. · View Herald TranscriptMay 9 2019, 12:23 PM
stella.stamenova requested changes to this revision.May 9 2019, 12:29 PM
stella.stamenova added inline comments.
lldb/lit/Breakpoint/case-sensitive.test
1

This should be system-windows. See config.py in llvm. By using system-windows, the test would be consistent with other platforms. Incidentally, I am just about to fix this as well as the test in LLVM that you changed to use windows - it should also use system-windows.

lldb/lit/lit.cfg.py
101

This really should live in config.py in llvm so that it can be used across all projects. And it should be system-windows for consistency with other platforms.

This revision now requires changes to proceed.May 9 2019, 12:29 PM
probinson abandoned this revision.May 9 2019, 12:48 PM

@stella.stamenova fixed this using system-windows instead.