cmake conversion #1

Merged
SlavekB merged 1 commits from feat/cmakeConv into master 5 years ago
Ghost commented 5 years ago
There is no content yet.
SlavekB reviewed 5 years ago
SlavekB left a comment
Owner

It looks good, but one small adjustment is necessary.

It looks good, but one small adjustment is necessary.
CMakeLists.txt Outdated
include( TDESetupPaths )
tde_setup_paths( )
Owner

Because TDE is usually installed outside of usual system paths, we need to add a path to pkg-config files:

##### set PKG_CONFIG_PATH

set( ENV{PKG_CONFIG_PATH} "${PKG_CONFIG_PATH}:$ENV{PKG_CONFIG_PATH}:${LIB_INSTALL_DIR}/pkgconfig" )
Because TDE is usually installed outside of usual system paths, we need to add a path to pkg-config files: ``` ##### set PKG_CONFIG_PATH set( ENV{PKG_CONFIG_PATH} "${PKG_CONFIG_PATH}:$ENV{PKG_CONFIG_PATH}:${LIB_INSTALL_DIR}/pkgconfig" ) ```
Owner

In addition to the above, I have ready a patch for add KDE_EXPORT to make the hidden visibility work properly…

In addition to the above, I have ready a patch for add `KDE_EXPORT` to make the hidden visibility work properly…
SlavekB approved these changes 5 years ago
SlavekB left a comment
Owner

It looks good. I have no other comments.

It looks good. I have no other comments.
SlavekB closed this pull request 5 years ago
SlavekB deleted branch feat/cmakeConv 5 years ago
SlavekB added this to the R14.0.7 release milestone 5 years ago
The pull request has been merged as 41ffd4d2f7.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: TDE/compizconfig-backend-tdeconfig#1
Loading…
There is no content yet.