-
Notifications
You must be signed in to change notification settings - Fork 47
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
Code portability issue, can't locate libraries on aarch64 #212
Comments
Yes, I agree, it definitely looks like that doesn't allow for the installation paths used by I suspect you know your way around python setup scripts better than I do1, is there any chance you know what the proper way here would be? Otherwise I'll try my hand at it, but that'll take some time ... If you can't wait, I'd recommend building Cyclone from source, all you need are Footnotes |
@eboasson Thank you for your answer! Install from sources cyclonedds-python#via-pypi, and cyclonedds-python#via-git works well for me. Btw. Got the same problem when trying to build in
Can make more verbose output if needed, but it looks similar to verbose output in my previous comment . This issue basically can be closed, but it is nice to have a way how to install I can create another issue in which we can continue the discussion about how we can make |
@eboasson Btw. I implemented fixes for buildhelp/cyclone_search.py#L48-L52 and buildhelp/oxidize_library.py#L49-L53, but because installation of cyclonedds from apt ( potential_libdirs = [Path("lib") / "aarch64-linux-gnu", "lib64", "lib"]
# Iterate over potential directories and use the first one that exists
for potential_dir in potential_libdirs:
libdir = dir / potential_dir
if libdir.exists():
break
else:
# No valid library directory found
return None
Think there should be a proper way to provide at least source distribution on PyPi (https://pypi.org/project/cyclonedds/#files) which can be built regardless of the system. |
It seems there might be some challenges with the portability of the code.
Please refer to buildhelp/cyclone_search.py#L48-L52 and buildhelp/oxidize_library.py#L49-L53 for reference.
Distro: Docker container with Debian Bookworm.
List of performed actions to reproduce:
debian:bookworm
The text was updated successfully, but these errors were encountered: