CMake: Improve creating wayland protocol files (PR #25945)
Maarten
Create the files in a temporary directory and only copy them to the final directory if the files are different.
This prevents the source files from being rebuilt every time cmake is run and the protocol files are replaced.
Exclude the protocols folder from being installed.
The entire setup header directory is installed, so exclude protocols.
see e.g. this build, and copied below for reference:
-- Installing: /usr/local/lib/wx/include/gtk3-unicode-3.3
-- Installing: /usr/local/lib/wx/include/gtk3-unicode-3.3/wx
-- Installing: /usr/local/lib/wx/include/gtk3-unicode-3.3/wx/setup.h
-- Installing: /usr/local/lib/wx/include/gtk3-unicode-3.3/wx/protocols
-- Installing: /usr/local/lib/wx/include/gtk3-unicode-3.3/wx/protocols/pointer-warp-v1-client-protocol.h
-- Installing: /usr/local/lib/wx/include/gtk3-unicode-3.3/wx/protocols/pointer-warp-v1-client-protocol.c
You can view, comment on, or merge this pull request online at:
https://github.com/wxWidgets/wxWidgets/pull/25945
Commit Summary
- 0d2bb06 CMake: Improve creating wayland protocol files
File Changes
(2 files)
- M build/cmake/init.cmake (18)
- M build/cmake/install.cmake (1)
Patch Links:
- https://github.com/wxWidgets/wxWidgets/pull/25945.patch
- https://github.com/wxWidgets/wxWidgets/pull/25945.diff
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
VZ
vadz left a comment (wxWidgets/wxWidgets#25945)Thanks, I didn't think about it, but it was obviously wrong to do what I did, so thank you for cleaning it up.
Will merge soon.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
Maarten
MaartenBent left a comment (wxWidgets/wxWidgets#25945)I actually asked about installing them in #25769 (comment), but assumed as well they wouldn't be installed automatically.
Seems I copy-pasted wrong, let me fix it...
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
Maarten
@MaartenBent pushed 1 commit.
- 5927a9b CMake: Improve creating wayland protocol files
—
View it on GitHub or unsubscribe.
You are receiving this because you are subscribed to this thread.
VZ
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.