This is an archive of the discontinued LLVM Phabricator instance.

[third-party] Update pexpect to 4.6.
ClosedPublic

Authored by davide on Mar 8 2019, 2:33 PM.

Diff Detail

Repository
rLLDB LLDB

Event Timeline

davide created this revision.Mar 8 2019, 2:33 PM
Herald added a project: Restricted Project. · View Herald Transcript
davide retitled this revision from [testsuite] Recommit the TestTerminal directory. to [third-party] Update pexpect to 4.6..Mar 8 2019, 2:34 PM
davide edited the summary of this revision. (Show Details)
davide removed reviewers: jfb, serge-sans-paille.
davide edited subscribers, added: Restricted Project, lldb-commits; removed: javed.absar, fedor.sergeev, dexonsmith and 4 others.
JDevlieghere requested changes to this revision.EditedMar 8 2019, 2:37 PM

I don't think we should check in the 'test' or 'example' subdirectories.

This revision now requires changes to proceed.Mar 8 2019, 2:37 PM
ormris removed a subscriber: ormris.Mar 8 2019, 2:46 PM
labath accepted this revision.Mar 11 2019, 2:02 AM

Removing the tests directory and similar sounds like a good idea, though I would be fine with leaving it in too. Apart from that, this looks good to me.

Removing the tests directory and similar sounds like a good idea, though I would be fine with leaving it in too. Apart from that, this looks good to me.

There's a bunch of directories, like examples and docs. I'd prefer to have the minimum checked in. All this stuff is available online, and I see very little benefit for making the history bigger and a grep returning more results.

This revision was not accepted when it landed; it landed in state Needs Revision.Mar 12 2019, 1:42 PM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptMar 12 2019, 1:42 PM