Skip to content

Conversation

@michael-o
Copy link
Contributor

@michael-o michael-o commented Nov 14, 2025

This update ensures compatibility with modern tools like pip, build, and
pyproject.toml, and helps avoid deprecation issues with legacy setup.py-based
builds. It also improves reproducibility and security by enabling isolated
builds, which is increasingly important for CI/CD pipelines and downstream
packaging systems.

@michael-o michael-o force-pushed the pep517 branch 2 times, most recently from 4b21785 to ae9eaee Compare November 15, 2025 16:45
@michael-o michael-o marked this pull request as ready for review November 15, 2025 16:48
@michael-o
Copy link
Contributor Author

For dynamic versioning there is https://pypi.org/project/git-versioner/

@michael-o
Copy link
Contributor Author

@ahebrank Would you mind having a look?

@ahebrank
Copy link
Owner

Please provide reasoning for the change and context in the MR description. This is a VERY old project so I'm not sure I see the benefit here.

@michael-o
Copy link
Contributor Author

Please provide reasoning for the change and context in the MR description. This is a VERY old project so I'm not sure I see the benefit here.

Will do, but VERY old does not mean broken or unusable. It works perfectly well.

This update ensures compatibility with modern tools like pip, build, and
pyproject.toml, and helps avoid deprecation issues with legacy setup.py-based
builds. It also improves reproducibility and security by enabling isolated
builds, which is increasingly important for CI/CD pipelines and downstream
packaging systems.
@michael-o
Copy link
Contributor Author

Please provide reasoning for the change and context in the MR description. This is a VERY old project so I'm not sure I see the benefit here.

Done. The source can flawlessly build on 3.12+ without legacy tools.

@michael-o
Copy link
Contributor Author

@ahebrank Did you get a chance to reevaluate my PR?

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