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

libtiff-dev and is missing from package.xml for noetic #21

Open
kjeremy opened this issue Aug 30, 2024 · 3 comments
Open

libtiff-dev and is missing from package.xml for noetic #21

kjeremy opened this issue Aug 30, 2024 · 3 comments
Assignees

Comments

@kjeremy
Copy link

kjeremy commented Aug 30, 2024

No description provided.

@amock
Copy link
Member

amock commented Oct 1, 2024

Indeed libtiff-dev is missing (fixed with PR #22 ). However, lz4 seems to be already existing in all branches. @kjeremy , could you specify the exact branch/tag/commit you are using?

@kjeremy
Copy link
Author

kjeremy commented Oct 1, 2024

Thanks for fixing this with libtiff-dev!

Looking back on this I think I misspoke with lz4. It does appear to be specified correctly. I had to modify the build logic for nixos so that the package could find lz4: lopsided98/nix-ros-overlay@2de811f and that's probably when I thought the package wasn't specified.

@kjeremy kjeremy changed the title libtiff-dev and lz4 are missing from package.xml for noetic libtiff-dev and is missing from package.xml for noetic Oct 1, 2024
@amock
Copy link
Member

amock commented Nov 19, 2024

Hi @kjeremy ,

I saw you added a post script that changed some of the variables set by the custom lz4 find script. We recently had some problems with some of the custom cmake-find scripts as well. So I changed the lz4 find-package script to use cmake's PkgConfig which I guess is currently the correct way of integrating lz4 into a cmake projects. I don't know whether it effects your nix build. Perhaps you can even remove the postPatch instructions after our changes.

@amock amock self-assigned this Nov 19, 2024
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

No branches or pull requests

2 participants