This is an archive of the discontinued LLVM Phabricator instance.

Disable TestProcessList on windows
ClosedPublic

Authored by wallace on Oct 17 2019, 9:29 AM.

Details

Summary

platform process list -v on windows doesn't show all the process arguments, making this test useless for that platform.
Windows trunk is broken as well

Event Timeline

wallace created this revision.Oct 17 2019, 9:29 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 17 2019, 9:29 AM
wallace edited the summary of this revision. (Show Details)Oct 17 2019, 9:31 AM
wallace added reviewers: clayborg, aadsm, labath.
stella.stamenova requested changes to this revision.Oct 17 2019, 9:44 AM

Please also file a bug and reference it in the skip statement.

This revision now requires changes to proceed.Oct 17 2019, 9:44 AM

oh girl, i need to get a bugzilla account. I hope i can get it soon. If it takes too long I can send another patch with the updated comment

I filed a bug for you, please reference it and submit: https://bugs.llvm.org/show_bug.cgi?id=43702 (we can update the bug description later).

teemperor accepted this revision.Oct 17 2019, 10:08 AM
wallace updated this revision to Diff 225466.Oct 17 2019, 10:50 AM

added bug link in the comment

This revision was not accepted when it landed; it landed in state Needs Review.Oct 17 2019, 10:52 AM
This revision was automatically updated to reflect the committed changes.