Skip to content
This repository has been archived by the owner on Nov 25, 2020. It is now read-only.

Add cmake build system #9

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

Conversation

cryptomilk
Copy link

Hi,

I've wanted to create RPM packages for rrcc, however the qmake based files do not provide a way and also it is normally ugly to get it fully working. So I've started to write cmake files.

It correctly builds and installs. However the resources are not found like icons or translation because it looks for them in the same location as the binary.

The best would be to define those locations in a "config.h" file and then load them from those locations. Maybe have two, a in build tree location and a installation location.

I've only created them for Linux, changes for other systems are still missing but it is a start ...

cmake can create Visual Studio, mingw or msys build files too.

@cryptomilk
Copy link
Author

For Windows you can create a NSIS installer using the cpack. You get a package target in mingw or VS

Example:
https://git.cryptomilk.org/projects/cmocka.git/tree/CPackConfig.cmake

@cryptomilk
Copy link
Author

This works on Linux and distributions could package rrcc now. It would be nice if rrcc would have a license ...

@LazyT
Copy link
Owner

LazyT commented Dec 21, 2018

Hi,

thanks for your work, but I've no time atm to look at this.

I never used cmake (don't like it, to complex) and for installers I prefer the Qt Installer Framework. Works on WIN/LIN/MAC without problems and is easy to use.

A license is added now.

@cryptomilk
Copy link
Author

Hey!

cmake isn't really that hard that hard to learn and offers nice features. Take a look at https://cliutils.gitlab.io/modern-cmake/

We could also add support for Windows. If I find the time I can fire up a windows install and work on supporting windows.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants