blue/squawk
blue/squawk
2
3
Fork 8
Code Issues 23 Pull requests 3 Releases 10 Wiki Activity Actions

Refactor CMakeLists #46

Merged
blue merged 5 commits from vae/squawk:build-refactor into messageFeed 2021-05-11 23:31:59 +00:00
Conversation 2 Commits 5 Files changed 29 +305 -382
vae commented 2021-05-11 19:26:43 +00:00
Contributor
Copy link
No description provided.
vae commented 2021-05-11 19:38:17 +00:00
Author
Contributor
Copy link

Also, this closes #45.

Also, this closes #45.
blue reviewed 2021-05-11 22:36:05 +00:00
blue left a comment
Owner
Copy link

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

Там есть 2 места, они наркоманские, я не уверен что сделал правильно, но так они сломаются
core/passwordStorageEngines/CMakeLists.txt
@ -25,1 +1,3 @@
wrappers/kwallet.cpp
if (WITH_KWALLET)
target_sources(squawk PRIVATE
wrappers/kwallet.cpp
blue commented 2021-05-11 22:24:25 +00:00
Owner
Copy link

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

А вот так вот точно нельзя, это должно быть миниатюрной разделяемой библиотекой, я её в рантайме пробую загрузить как плагин
blue marked this conversation as resolved
plugins/CMakeLists.txt Outdated
@ -25,2 +1,2 @@
install(TARGETS openFileManagerWindowJob DESTINATION ${CMAKE_INSTALL_LIBDIR})
endif()
if (WITH_KIO)
target_sources(squawk PRIVATE openfilemanagerwindowjob.cpp)
blue commented 2021-05-11 22:25:51 +00:00
Owner
Copy link

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

Тут тоже нельзя, вот эту штуку надо сделать разделяемой либой
vae commented 2021-05-11 22:39:40 +00:00
Author
Contributor
Copy link

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

ок, поправлю сейчас тогда
blue marked this conversation as resolved
blue merged commit bd07c49755 into messageFeed 2021-05-11 23:31:59 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
Bug

Something is not working

  
Documentation

Documentation related issues

  
Easy

   
Feature

New feature

  
Localization

Localization related issues

  
Packaging

Issues related to packaging and building

  
Refactoring

Need to reimplement something in a better way

  
Test

Need to test something

No labels
Bug
Documentation
Easy
Feature
Localization
Packaging
Refactoring
Test
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: blue/squawk#46
No description provided.
Delete branch "vae/squawk:build-refactor"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?