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

build: support find_package basisu via CMake + CMake improvements #383

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

aminya
Copy link

@aminya aminya commented Nov 1, 2024

This adds needed CMake commands to make basisu usable via find_package(basisu CONFIG), which is the preferred method when using basisu in another project.

This is how the installation directory would look like

image

I added an option that allows for disabling the building of the examples, which can be optionally enabled. I added the code for forwarding the BASISID CMake definitions to the targets if defined externally.

Finally, I fixed an issue with stripping when using Multi-config generators with LLVM.

Here's how you can test this:

cmake -S . -B ./build/ -G "Ninja Multi-Config" -DEXAMPLES=OFF
cmake --build ./build/ --config Release
cmake --install ./build/ --config Release --prefix ./install

@aminya aminya changed the title build: support find_package(basisu CONFIG) build: support find_package basisu via CMake Nov 10, 2024
@aminya aminya changed the title build: support find_package basisu via CMake build: support find_package basisu via CMake + CMake improvements Nov 10, 2024
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.

1 participant