#46 Refactor CMakeLists

Merged
blue merged 5 commits from vae/squawk:build-refactor into messageFeed 3 months ago
vae commented 3 months ago
There is no content yet.
vae added 4 commits 3 months ago
vae commented 3 months ago
Poster

Also, this closes #45.

Also, this closes #45.
blue reviewed 3 months ago
blue left a comment

Там есть 2 места, они наркоманские, я не уверен что сделал правильно, но так они сломаются

core/passwordStorageEngines/CMakeLists.txt
wrappers/kwallet.cpp
if (WITH_KWALLET)
target_sources(squawk PRIVATE
wrappers/kwallet.cpp
blue commented 3 months ago

А вот так вот точно нельзя, это должно быть миниатюрной разделяемой библиотекой, я её в рантайме пробую загрузить как плагин

А вот так вот точно нельзя, это должно быть миниатюрной разделяемой библиотекой, я её в рантайме пробую загрузить как плагин
blue marked this conversation as resolved
plugins/CMakeLists.txt
install(TARGETS openFileManagerWindowJob DESTINATION ${CMAKE_INSTALL_LIBDIR})
endif()
if (WITH_KIO)
target_sources(squawk PRIVATE openfilemanagerwindowjob.cpp)
blue commented 3 months ago

Тут тоже нельзя, вот эту штуку надо сделать разделяемой либой

Тут тоже нельзя, вот эту штуку надо сделать разделяемой либой
vae commented 3 months ago

ок, поправлю сейчас тогда

ок, поправлю сейчас тогда
blue marked this conversation as resolved
vae added 1 commit 3 months ago
blue merged commit bd07c49755 into messageFeed 3 months ago
The pull request has been merged as bd07c49755.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.