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

CXX-Qt-CMake patch release 0.7.1 #5

Open
wants to merge 2 commits into
base: 0.7
Choose a base branch
from
Open

CXX-Qt-CMake patch release 0.7.1 #5

wants to merge 2 commits into from

Conversation

LeonMatthesKDAB
Copy link
Collaborator

Also note that we're using a branch name 0.7, so we should start tagging with 0.7.1, instead of v0.7.1 as well.

That way it's consistent with Rusts semver behavior.

LeonMatthesKDAB and others added 2 commits January 9, 2025 08:14
As per the CMake documentation, by default IMPORTED targets are only
visible within the current directory.
This isn't useful behavior in our case, so make it globally accessible.

See also: https://cmake.org/cmake/help/latest/command/add_library.html#imported-libraries
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