This is an archive of the discontinued LLVM Phabricator instance.

[python] Make the collections import future-proof
ClosedPublic

Authored by jstasiak on Jan 4 2019, 4:37 PM.

Details

Summary

On Python 3.7 the old code raises a warning:

DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated, and in 3.8 it will stop working
    class ArgumentsIterator(collections.Sequence):

On Python 3.8 it wouldn't work anymore.

Diff Detail

Event Timeline

jstasiak created this revision.Jan 4 2019, 4:37 PM
This revision is now accepted and ready to land.Jan 5 2019, 12:54 AM

Thanks. If you could commit this on my behalf I'd appreciate it (I don't have commit access).

This revision was automatically updated to reflect the committed changes.