Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[GLib] Script rewrite-compile-commands missing from release tarballs #26407

Merged
merged 1 commit into from Mar 25, 2024

Conversation

@aperezdc aperezdc requested review from zdobersek and a team as code owners March 25, 2024 14:16
@aperezdc aperezdc self-assigned this Mar 25, 2024
@aperezdc aperezdc added the Tools / Tests Tools in the Tools directory, build issues, test infrastructure, and bugs in test cases label Mar 25, 2024
@aperezdc aperezdc added the skip-ews Applied to prevent a change from being run on EWS label Mar 25, 2024
@aperezdc aperezdc requested review from carlosgcampos and removed request for zdobersek March 25, 2024 14:17
@psaavedra
Copy link
Contributor

Thanks for adding the Tools/Scripts/rewrite-compile-commands to the sources tarball. This fix builds with -DCMAKE_EXPORT_COMPILE_COMMANDS=ON like Yocto cmake.class does in the latest release.

@aperezdc aperezdc added the unsafe-merge-queue Applied to send a pull request to merge-queue, but skip building and testing label Mar 25, 2024
@aperezdc
Copy link
Contributor Author

Thanks for adding the Tools/Scripts/rewrite-compile-commands to the sources tarball. This fix builds with -DCMAKE_EXPORT_COMPILE_COMMANDS=ON like Yocto cmake.class does in the latest release.

You're welcome. I will be backporting the patch to the 2.44 branch as well.

https://bugs.webkit.org/show_bug.cgi?id=271644

Reviewed by Michael Catanzaro.

* Tools/gtk/manifest.txt.in: List Tools/Scripts/rewrite-compile-commands
  for inclusion in release tarballs.
* Tools/wpe/manifest.txt.in: Ditto.

Canonical link: https://commits.webkit.org/276622@main
@webkit-commit-queue
Copy link
Collaborator

Committed 276622@main (aaef68e): https://commits.webkit.org/276622@main

Reviewed commits have been landed. Closing PR #26407 and removing active labels.

@webkit-commit-queue webkit-commit-queue merged commit aaef68e into WebKit:main Mar 25, 2024
@webkit-commit-queue webkit-commit-queue removed the unsafe-merge-queue Applied to send a pull request to merge-queue, but skip building and testing label Mar 25, 2024
@aperezdc aperezdc deleted the dist-rewrite-commands branch March 25, 2024 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip-ews Applied to prevent a change from being run on EWS Tools / Tests Tools in the Tools directory, build issues, test infrastructure, and bugs in test cases
Projects
None yet
5 participants