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

Fixes #1

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Fixes #1

wants to merge 9 commits into from

Conversation

KOLANICH
Copy link

@KOLANICH KOLANICH commented Feb 7, 2018

No description provided.

@Antonin-Deniau
Copy link
Owner

Hi, and thanks for the PR ! This is looking good ! I will merge it but I saw that your kaitaiStructCompile failed to build on travis, will it have an impact on my project ?

@KOLANICH
Copy link
Author

KOLANICH commented Mar 27, 2018

I will merge it but I saw that your kaitaiStructCompile failed to build on travis, will it have an impact on my project ?

I don't know, it needs some investigation, travis logs show that they fail on the stage of upgrading setuptools and for me for now it's not clear why it fails there.

The tool is very alpha quality (it doesn't mean it shouldn't be used, we are all developers and from developers I expect ability to fix some minor flaws and ability to create PR and issues on bug tracker, the tool won't improve itself, it's developers who improve things), and even if it was release quality I would not guarantee anything. You know: every dependency poses some risk and every update too, even super-popular libraries like sympy are not immune to it. So I guess that you can start merging the changes not introducing the dependency leaving the one involving the dependency for later, they are pretty permutable. Though I insist on testing of every change before merging.

Please also note that the tool is explicitly incompatible with python 2 by design: python 3.7 is called the last release of python 3, the next one will be python 4, and we don't want the hell of supporting 3 incompatible versions, python 2 needs coup de grace (like x86_32 and Windows XP and old Androids do, but unlike it (I beleive we shouldn't say people that they must pay Intel, Microsoft, Google and Samsung in order to use our software ) python 3 works even on ancient OSes and hardware, I have seen a PIII PC with 256 MiB of RAM running python 3.7 pretty well, so everyone can upgrade), everyone realizes it, lot of projects have dropped it. So lack of support for python 2 in my libs is my small contribution to that coup de grace. So if you need it be buildable with python 2 you cannot use it (at least in my edition, but you are free to fork).

Repository owner deleted a comment from ThuretGbenougan Mar 4, 2024
@github-staff github-staff deleted a comment from robincj Apr 27, 2024
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