search

reticulatedpines / magiclantern_simplified

A Git based version of Magic Lantern, for those unwilling or unable to work using Mercurial. The vast majority of branches have been removed, with those thought to be important brought in individually and merged.
GNU General Public License v2.0
142 stars 50 forks source link

Remove Python 2 dependency #95

Closed ilia3101 closed 8 months ago

ilia3101 commented 1 year ago

Minimal changes this time.

Turns out it wasn't necessary to update html2text, it just needed a few syntax tweaks for Python 3.

Integer division was also taken care of.

branylagaffe commented 11 months ago

Should also the python2 dependencies to create the features' table also be handled, OR, the script isn't used any more ?

reticulatedpines commented 8 months ago

Should also the python2 dependencies to create the features' table also be handled, OR, the script isn't used any more ?

That isn't used in builds for cams, so the urgency is lower. Currently, we don't generate the feature matrix from this repo - if we bring that back we can fix it then.

reticulatedpines commented 8 months ago

Merged here: https://github.com/reticulatedpines/magiclantern_simplified/commit/8e3c5eeca4e8eff0211687c6618431b4a83ffb4c

I also deleted some unused files requiring python2.

Thanks for the help!