Page MenuHome

Fix: replaced Python module "imp" with "implib"
ClosedPublic

Authored by Sybren A. Stüvel (sybren) on Jan 21 2015, 8:08 PM.

Details

Summary

Python 3.4.0 deprecated the "imp" module, and replaced it with
"importlib". This changes imp.reload() into implib.reload().

Diff Detail

Repository
rBA Blender Add-ons
Branch
temp-sybren-replace-imp-module

Event Timeline

Sybren A. Stüvel (sybren) retitled this revision from to Fix: replaced Python module "imp" with "implib".
Sybren A. Stüvel (sybren) updated this object.

Git updated origin/master but not master, so some changes on master branch ended up here.

The io_import_dxf addon uses imp.load_dynamic(), which no longer exists in the new importlib module. This is *not* fixed in this patch, as it requires more than trivial changes. It also depends on a file cytags.so that isn't shipped with Blender, so I wouldn't be able to test it without more info.

Campbell Barton (campbellbarton) requested changes to this revision.Jan 22 2015, 7:11 AM
Campbell Barton (campbellbarton) edited edge metadata.

The simple replacement for imp.reload seem fine, but theres some issue with render_renderfarmfi addon that should be resolved before committing.

(or just revert those changes and resolve later, along with the other non-trivial imp issues).

render_renderfarmfi/rpc.py
34

implib doesn't exist, and seems importlib.load_module doesnt exist either.

This revision now requires changes to proceed.Jan 22 2015, 7:11 AM
Sybren A. Stüvel (sybren) edited edge metadata.

Removed non-trivial change; now really the only thing changing is imp.reload() to importlib.reload()

This revision is now accepted and ready to land.Jan 23 2015, 12:28 AM
This revision was automatically updated to reflect the committed changes.