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

properly set lib dir for 64 bit systems #168

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

Conversation

sergey-safarov
Copy link

No description provided.

@andywolk
Copy link
Contributor

This is from our CI:

-- Found PkgConfig: /usr/bin/pkg-config (found version "0.29") 
-- Checking for module 'uuid'
--   Found uuid, version 2.33.1
Found UUID setup target at imported location: /usr/lib/x86_64-linux-gnu/libuuid.so
-- Found OpenSSL: /usr/lib/x86_64-linux-gnu/libcrypto.so (found version "1.1.1n")  
-- Found OpenSSL: /usr/lib/x86_64-linux-gnu/libcrypto.so (found version "1.1.1n") found components:  Crypto SSL 
-- Looking for pthread.h
-- Looking for pthread.h - found
-- Looking for pthread_create
-- Looking for pthread_create - not found
-- Check if compiler accepts -pthread
-- Check if compiler accepts -pthread - yes
-- Found Threads: TRUE  
-- Performing Test COMPILER_HAS_HIDDEN_VISIBILITY
-- Performing Test COMPILER_HAS_HIDDEN_VISIBILITY - Success
Compiler supports visibility
CMake Error at CMakeLists.txt:559 (install):
  install TARGETS given no LIBRARY DESTINATION for shared library target
  "ks2".


-- Configuring incomplete, errors occurred!

@sergey-safarov
Copy link
Author

sergey-safarov commented Jul 30, 2023

Could you execute this test commands

wget https://raw.githubusercontent.com/sergey-safarov/libks/tmp/pkg/rpm/libks.spec
rpmbuild -bb --undefine _disable_source_fetch libks.spec

Expected result, compiled rpm packages.
Tested on CentOS 8 stream.

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

Successfully merging this pull request may close these issues.

2 participants