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

Cmake build #406

Open
wants to merge 10 commits into
base: develop
Choose a base branch
from
Open

Cmake build #406

wants to merge 10 commits into from

Conversation

selmf
Copy link
Member

@selmf selmf commented Oct 31, 2023

This PR introduces the first draft of a CMake based build system for YACReader. It is not feature complete and currently only supports building on Linux with system dependencies, but the necessary plumbing and key structures like options, module layout and best practices are mostly present.

Features:

  • Enforce out-of-source builds
  • Proper dependency management
  • Support promoting warnings to errors without affecting system headers (like Qt)
  • Faster builds
  • Can build and install YACReader on Linux and other Unix systems

Going forward, the idea is to include this WIP in our CI to keep it working and use it as a test platform for more strict compile flags than we currently set in qmake.

Missing features, which should be considered advanced - like downloading and managing external source dependencies like 7zip - should be introduced by later PRs until we can drop support for qmake builds.

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