hoene / libmysofa

Reader for AES SOFA files to get better HRTFs
Other
137 stars 73 forks source link

1.3.2: no header files installed?🤔 #210

Closed wuch-g2v closed 1 year ago

wuch-g2v commented 1 year ago

Looks like in just released 1.3.2 install target do not install any header files:

+ cd libmysofa-1.3.2
+ DESTDIR=/home/tkloczko/rpmbuild/BUILDROOT/libmysofa-1.3.2-2.fc35.x86_64
+ /usr/bin/cmake --install x86_64-redhat-linux-gnu
-- Install configuration: "RelWithDebInfo"
-- Installing: /home/builder/rpmbuild/BUILDROOT/libmysofa-1.3.2-2.fc35.x86_64/usr/share/libmysofa/default.sofa
-- Installing: /home/builder/rpmbuild/BUILDROOT/libmysofa-1.3.2-2.fc35.x86_64/usr/share/libmysofa/MIT_KEMAR_normal_pinna.sofa
-- Installing: /home/builder/rpmbuild/BUILDROOT/libmysofa-1.3.2-2.fc35.x86_64/usr/lib64/pkgconfig/libmysofa.pc
-- Installing: /home/builder/rpmbuild/BUILDROOT/libmysofa-1.3.2-2.fc35.x86_64/usr/lib64/cmake/mysofa/mysofaTargets.cmake
-- Installing: /home/builder/rpmbuild/BUILDROOT/libmysofa-1.3.2-2.fc35.x86_64/usr/lib64/cmake/mysofa/mysofaTargets-relwithdebinfo.cmake
-- Installing: /home/builder/rpmbuild/BUILDROOT/libmysofa-1.3.2-2.fc35.x86_64/usr/lib64/cmake/mysofa/mysofaConfig.cmake
-- Installing: /home/builder/rpmbuild/BUILDROOT/libmysofa-1.3.2-2.fc35.x86_64/usr/lib64/cmake/mysofa/mysofaConfigVersion.cmake
-- Installing: /home/builder/rpmbuild/BUILDROOT/libmysofa-1.3.2-2.fc35.x86_64/usr/lib64/libmysofa.so.1.3.2
-- Installing: /home/builder/rpmbuild/BUILDROOT/libmysofa-1.3.2-2.fc35.x86_64/usr/lib64/libmysofa.so.1
-- Installing: /home/builder/rpmbuild/BUILDROOT/libmysofa-1.3.2-2.fc35.x86_64/usr/lib64/libmysofa.so
-- Installing: /home/builder/rpmbuild/BUILDROOT/libmysofa-1.3.2-2.fc35.x86_64/usr/bin/mysofa2json
-- Set runtime path of "/home/builder/rpmbuild/BUILDROOT/libmysofa-1.3.2-2.fc35.x86_64/usr/bin/mysofa2json" to ""
wuch-g2v commented 1 year ago

cmake -L output:

-- Cache values
ADDRESS_SANITIZE:BOOL=OFF
BUILD_SHARED_LIBS:BOOL=ON
BUILD_STATIC_LIBS:BOOL=OFF
BUILD_TESTS:BOOL=ON
CMAKE_BUILD_TYPE:STRING=RelWithDebInfo
CMAKE_INSTALL_PREFIX:PATH=/usr
CODE_COVERAGE:BOOL=OFF
MATH:FILEPATH=/usr/lib64/libm.so
hoene commented 1 year ago

Do you know how to fix it?

wuch-g2v commented 1 year ago

I've not been looking on that case yet. I've only noticed that automated rpm package build of updated version libmysofa failed because of missing header files.

hoene commented 1 year ago

@andreasfranck May you have a look on this issue because it was introduce by your changes? Includes are only exported if compile with the static option.

wuch-g2v commented 1 year ago

Includes are only exported if compile with the static option.

Which does not make any sense ..