This is an archive of the discontinued LLVM Phabricator instance.

Fix failing python bindings test
AbandonedPublic

Authored by jbcoe on Feb 11 2016, 5:59 PM.

Details

Summary

Order of compilation database commands has changed causing test failure.

Reordering fixes the test cindex.test_cdb.test_all_compilecommand.

Other Python bindings tests currently fail. I will address these in a later patch.

Diff Detail

Event Timeline

jbcoe updated this revision to Diff 47758.Feb 11 2016, 5:59 PM
jbcoe retitled this revision from to Fix failing python bindings test.
jbcoe updated this object.
jbcoe added a reviewer: akyrtzi.
jbcoe set the repository for this revision to rL LLVM.
jbcoe added a subscriber: cfe-commits.
skalinichev edited edge metadata.Mar 19 2016, 4:41 AM

It reminded me http://reviews.llvm.org/D17278

So it seems like it should be already fixed, no?

jbcoe added a comment.Apr 4 2016, 7:20 AM

The other python test fix has been applied in r263170 (Differential Revision: http://reviews.llvm.org/D17226).

Has anyone had a chance to look at _this_ patch yet?