8000 Build and publish wheels by oriori1703 · Pull Request #186 · frida/frida-tools · GitHub
[go: up one dir, main page]

Skip to content

Build and publish wheels - #186

Merged
oleavr merged 3 commits into
frida:mainfrom
oriori1703:ci/wheels
Nov 5, 2026
Merged

oleavr merged 3 commits into
frida:mainfrom
oriori1703:ci/wheels

Conversation

@oriori1703
Copy link
Copy Markdown
Contributor

Build and publish frida-tools in the CI

This is a continuation of frida/frida#3482

fixes: #176

@oriori1703
Copy link
Copy Markdown
Contributor Author

@oleavr This supposedly works, but I haven't checked that the upload actually works, because you have to setup trusted publishing.

Also you might want to change when those jobs run

@oriori1703
oriori1703 marked this pull request as ready for review June 6, 2025 14:46
@oriori1703
Copy link
Copy Markdown
Contributor Author

Also I noticed that frida-python is specified as dependency in meson, which means that we have to build frida-python (and all of its deps) just to build this one.

IMO because frida-python is a runtime dependency it doesn't really fit in meson.
Maybe we should remove it, and just rely on pip for managing that dependency?

@oriori1703 oriori1703 mentioned this pull request Jun 6, 2025
3 tasks done
@oriori1703 oriori1703 changed the title Ci/wheels Build and publish wheels Jun 10, 2025
@oriori1703
oriori1703 force-pushed the ci/wheels branch 9 times, most recently from 27bda6f to a2f7ef0 Compare August 19, 2025 22:43
@oriori1703
Copy link
Copy Markdown
Contributor Author

Hey @oleavr, just a gentle ping on this PR when you have time. Would appreciate your review! 😄

Also I saw you switched back to setuptools in e7a4b28.
I didn't really understand what was the issue your were having, but if you want I would be happy to take a look with a bit more info.

oleavr and others added 3 commits November 5, 2026 20:36
frida-python is a runtime dependency managed by pip, so the test-suite
should pick it up from the environment rather than a subproject.
Publishing will be done from CI using trusted publishing, so the twine
issue that made us revert to setuptools no longer applies.

Also install tracer_ui.zip where the tracer expects it, and make the
wrapper scripts optional so they don't end up in wheels, which get
theirs from the entry points.
Build the sdist and wheel on every push and pull request, publish pushes
to TestPyPI, and publish tags to PyPI using trusted publishing.

Fixes frida#176
@oleavr
oleavr merged commit 66c21a2 into frida:main Nov 5, 2026
@oleavr
oleavr commented Nov 5, 2026
Copy link
Copy Markdown
Member

Thanks! 🙌 So sorry for the delay here!

@oriori1703
oriori1703 deleted the ci/wheels branch November 5, 2026 22:30
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.

publish wheels

2 participants

0