Skip to content

build: fix builds with new setuptools#220

Merged
dupondje merged 1 commit intomasterfrom
fix/copr
Jul 8, 2025
Merged

build: fix builds with new setuptools#220
dupondje merged 1 commit intomasterfrom
fix/copr

Conversation

@dupondje
Copy link
Copy Markdown
Member

@dupondje dupondje commented Jul 8, 2025

Since setuptools v69.3.0, name of the tar file is modified according to PEP 625. Therefor update the code so it builds correctly again with newer setuptools versions.

@dupondje dupondje requested review from aesteve-rh and nirs as code owners July 8, 2025 10:41
@dupondje dupondje force-pushed the fix/copr branch 2 times, most recently from 124037f to 01b14b2 Compare July 8, 2025 10:57
Since setuptools v69.3.0, name of the tar file is modified according
to PEP 625. Therefor update the code so it builds correctly again
with newer setuptools versions.

Signed-off-by: Jean-Louis Dupond <jean-louis@dupond.be>
@dupondje
Copy link
Copy Markdown
Member Author

dupondje commented Jul 8, 2025

Copy link
Copy Markdown
Contributor

@JasperB-TeamBlue JasperB-TeamBlue left a comment

Choose a reason for hiding this comment

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

I feel like it is against best practice to mix ovirt-imageio and ovirt_imageio. Should it not be better if it was written everywhere one of either way?

@dupondje
Copy link
Copy Markdown
Member Author

dupondje commented Jul 8, 2025

I feel like it is against best practice to mix ovirt-imageio and ovirt_imageio. Should it not be better if it was written everywhere one of either way?

That would be indeed :)
But spec/rpm packaging guidelines are requiring dashes (-)
While python expects underscores (_) ...

@dupondje dupondje requested a review from JasperB-TeamBlue July 8, 2025 12:54
@dupondje dupondje merged commit 57cbbb3 into master Jul 8, 2025
26 checks passed
@dupondje dupondje deleted the fix/copr branch July 8, 2025 14:28
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