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

Add libindiclient shared library #1776

Merged
merged 1 commit into from
Nov 30, 2022
Merged

Add libindiclient shared library #1776

merged 1 commit into from
Nov 30, 2022

Conversation

pawel-soja
Copy link
Contributor

Solving the problem of building a deb package.

@knro
Copy link
Contributor

knro commented Nov 30, 2022

Oh I actually noticed that libindiclient.so was missing from these files but didn't think it was the issue since it has been working fine for the last few years.

@pawel-soja
Copy link
Contributor Author

indiclient.so never built before, when I added options to build static and shared libraries, by default the shared library is additionally built. From what I can see, other libraries in INDI Core want to link it instead of static.

@pawel-soja
Copy link
Contributor Author

Ahhh fail. I will add package build testing to CI later.

@knro knro merged commit b616c96 into master Nov 30, 2022
@knro knro deleted the hotfix/missing-package-libs branch November 30, 2022 09:08
@knro
Copy link
Contributor

knro commented Dec 1, 2022

indiclient library now fails to build on KDE Craft build system due to missing zlib.h

I added ZLIB_INCLUDE_DIR to a couple of places, but it still fails to find the file. CMake reports that zlib was found in the system, so that was not an issue.

D:\CraftRoot\build\_\dec7706e\indiclient\libs\indicore\indililxml.h(488): warning C4267: 'argument': conversion from 'size_t' to 'int', possible loss of data
�[32mD:\CraftRoot\dev-utils\bin\ninja.exe: b"D:\\CraftRoot\\build\\_\\dec7706e\\indiclient\\libs\\indicore\\indililxml.h(488): warning C4267: 'argument': conversion from 'size_t' to 'int', possible loss of data"�[0m
D:\CraftRoot\build\_\dec7706e\indiclient\libs\indidevice\basedevice.cpp(42): fatal error C1083: Cannot open include file: 'zlib.h': No such file or directory
�[32mD:\CraftRoot\dev-utils\bin\ninja.exe: 

Any ideas?

@pawel-soja
Copy link
Contributor Author

I think I know what it is. Give me 1 hour and I'll be at the computer and upload the fix.

@pawel-soja
Copy link
Contributor Author

You were on the right track! indidevice compiles as an object library and required the zlib.h header to compile as shown by the error. To compile objects, linking of libraries is not required, but access to headers is required.

Following your lead, I put a similar commit on master.

Let me know if anything else comes up.

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