Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] Change code to run with Python 3 #63

Closed
wants to merge 4 commits into from
Closed

[WIP] Change code to run with Python 3 #63

wants to merge 4 commits into from

Conversation

NicolasCARPi
Copy link
Contributor

Hello,

As discussed here, I made a python3 compatible version (see https://pythonclock.org/).

The code runs, I believe I changed all the necessary bits and pieces so it runs. BUT, and it's a big but, it seems it doesn't manage to recover partitions.

So please have a look at the code and try it out on real data and let me know if you find something. Because I'm not familiar with the codebase, I might have missed something.

~Nico

* replace print with print()
* replace unichr() with chr()
* replace xrange with range()
* fix the imports
also removed the content of __init__.py
* master:
  Fixing #61. Using Unicode string instead of ASCII to display warning when calling "tree" command (#62)
@Lazza
Copy link
Owner

Lazza commented Nov 22, 2019

see https://pythonclock.org/

Porting RecuperaBit to Python 3 might take any arbitrary amount of time, so I would say that it needs to be done properly regardless on when they plan to stop updating Python 2.

I will take a look, however I do not commit to deadlines on FLOSS projects. 😅

@NicolasCARPi
Copy link
Contributor Author

Sure, no problem! Here I just wanted to facilitate the work by making sure the software globally runs with python3, but I realize the inner parts might take more time to get updated!

At least now there is a basis for the porting :)

@NicolasCARPi NicolasCARPi changed the title Change code to run with Python 3 [WIP] Change code to run with Python 3 Nov 22, 2019
@Lazza
Copy link
Owner

Lazza commented Nov 22, 2019

Here I just wanted to facilitate the work

I understand and this is much appreaciated, thank you for what you have done so far!

@NicolasCARPi
Copy link
Contributor Author

Superseeded by #73

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants