Skip to content

Comments

Fix gobject-introspection build#1718

Closed
ladaapp wants to merge 1 commit intowingtk:mainfrom
ladaapp:fix_building_gobject_introspection
Closed

Fix gobject-introspection build#1718
ladaapp wants to merge 1 commit intowingtk:mainfrom
ladaapp:fix_building_gobject_introspection

Conversation

@ladaapp
Copy link
Contributor

@ladaapp ladaapp commented Feb 17, 2026

Hi, recent version of setuptools breaks gobject-introspection build (https://gitlab.gnome.org/GNOME/gobject-introspection/-/issues/575)

Locally I fixed this via

uv tool install gvsbuild --with "setuptools<81"

So I think for now it's probably best to restrict setuptools upper version

You can see the error also in this PR from dependabot #1717 which updated to latest setuptools

@danyeaw
Copy link
Member

danyeaw commented Feb 17, 2026

Hi @ladaapp, thanks so much for the contribution! I have a patch here that corrects this, just waiting for a review: e7ae272

@ladaapp
Copy link
Contributor Author

ladaapp commented Feb 17, 2026

Hmm, I'm wondering if this would then work with setuptools < 81? Maybe better to inspect if CCompiler.init has a dry_run arg (<81) then pass the arg and if not (>=81) then don't.

@danyeaw
Copy link
Member

danyeaw commented Feb 17, 2026

Hi @ladaapp, it should work because we are passing in keyword arguments with the patched version and all the arguments in <81 and >=81 are optional.

You can see the change in the base module here: pypa/setuptools@75a8954#diff-432dd6776972c4ddedab02e1eb654946ac43f98f9cc7fc7663040a6acdcd45b5

@ladaapp
Copy link
Contributor Author

ladaapp commented Feb 17, 2026

Okay cool, even better. Thanks for the fix then. Closing.

@ladaapp ladaapp closed this Feb 17, 2026
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