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

Draft: CMake-ify the build #809

Merged
merged 13 commits into from
Aug 23, 2022

Conversation

adriaandegroot
Copy link

This is a branch, not-finished-yet, that ports the build of core/ and attendant bits from qmake, to CMake. There is one source-code change. There is also one new build-time dependency: KDE's extra-cmake-modules, which provide all kinds of useful CMake modules for Qt-based applications (and in particular, a FindXCB module).

Filing the PR in draft state so it can be seen to be "in-progress".

- installs only the executable and some sound files; this is enough
  to convince lumina-desktop to start, not enough for it to be
  useful. The pkg-plist shows many more files, so those need to
  be chased down.
The icon-themes are special because they live outside the
source-tree for lumina-core; introduce a function for
installing themes to simplify upcoming material-design themes.
- add an extension to the install_theme function to help with
  placing SVGs in suitable subdirectories
@q5sys
Copy link
Member

q5sys commented Feb 21, 2022

I really appreciate your work on this.
I see you got to some of the theme stuff, there is a lingering bug in there somewhere that I need to dig into ( Issue #789 )
But that's around the installation of the theme files and then the existing code for what gets loaded when. I don't think that'll have to much impact on what you're doing in this PR. And if I do need to do a clean implementation of which files get installed where, I'll be able to update your code easy enough.
I think I might have to take some vacation time from work so I can actually get the time I need to work on things. haha

@q5sys q5sys changed the base branch from master to cmakify-all-the-things August 23, 2022 19:48
@q5sys
Copy link
Member

q5sys commented Aug 23, 2022

Merging this into a branch here in this repo so if I or other get time we can base off of your work.

@q5sys q5sys merged commit 8967f02 into lumina-desktop:cmakify-all-the-things Aug 23, 2022
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