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

Qt5 compatibility #13

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

Qt5 compatibility #13

wants to merge 8 commits into from

Conversation

dzhoshkun
Copy link

Hi there,

I've added some CMake configuration options to make CuteLogger compatible with Qt5 as well, while keeping Qt4 as the default. As such, to use Qt5, specify -D USE_Qt5=ON when building.

I've also added a CMake find file and install options that allow for -D CuteLogger_DIR=/path/to/CuteLoggerConfig.cmake to be used when building other projects that depend on CuteLogger. Now make install installs to ${CMAKE_INSTALL_PREFIX}/include/cutelogger/ and ${CMAKE_INSTALL_PREFIX}/lib/cutelogger/

Thanks for this nice library. I'm using it for thread-safe logging to the console and file.

Cheers,
Dzhoshkun

Dzhoshkun Ismail Shakir added 5 commits November 13, 2015 14:32
- sample application not being built any more
- Qt5 compatibility specify "-D USE_Qt5=ON" when building, in addition to "-D Qt5Core_DIR=..."
- added CMake find file, which allows for CuteLogger to be found using "FIND_PACKAGE(CuteLogger CONFIG REQUIRED)" (NB: "make install" will install to e.g. /usr/local/include/cutelogger/ and /usr/local/lib/cutelogger/ now)
@cyberbobs cyberbobs self-assigned this Nov 16, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants