Page MenuHomePhabricator

Disable TestProcessList on windows
ClosedPublic

Authored by wallace on Thu, Oct 17, 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

Diff Detail

Event Timeline

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

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

This revision now requires changes to proceed.Thu, Oct 17, 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.Thu, Oct 17, 10:08 AM
wallace updated this revision to Diff 225466.Thu, Oct 17, 10:50 AM

added bug link in the comment

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