Skip to content

Conversation

@arjpur01
Copy link

@arjpur01 arjpur01 commented Jun 3, 2025

Add a pyproject.toml to devlib.
Since pyproject.toml doesnt directly support dynamic versioning, added only [build-system] section so that tools like pip will refer back to setup.py to install the core of the project
So currently, this has to be used in conjunction with setup.py.

In the future, we can look into fully migrating to a pyproject.toml only approach, but that involves more things to handle versioning.

Fixes #702

@douglas-raillard-arm douglas-raillard-arm changed the title add pyproject.toml. address https://github.com/ARM-software/devlib/issues/702 add pyproject.toml Jun 4, 2025
Copy link
Collaborator

@douglas-raillard-arm douglas-raillard-arm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please reword the commit message as well to match the style normally used in that repo

@arjpur01
Copy link
Author

Please reword the commit message as well to match the style normally used in that repo

could you suggest what the message should be? currently i have put it as just "add pyproject.toml". should it be elaborated more?

address ARM-software#702

Fix the pip warning explained here:
pypa/pip#11457

By following this procedure:
pypa/pip#11457 (comment)
@@ -0,0 +1,5 @@
[build-system]
requires = [
"setuptools>=80.9.0",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

setuptools 80.9.0 require a higher version than devlib:

Requires: Python >=3.9

https://pypi.org/project/setuptools/80.9.0/

However, I'm not so sure devlib actually works with 3.7 anymore, and Ubuntu 20.04 (focal) has reached EOL earlier this year. So the next LTS is Jammy 22.04, which ships Python 3.10.

As a result, I think it's acceptable to bump the required Python version in devlib to 3.9 at least. I think this was discussed on another PR but I can't remember which one.

@marcbonnici Any thoughts ?

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.

Missing pyproject.toml

2 participants