Skip to content

Comments

Fix AppImage error with libpcap#971

Merged
GyulyVGC merged 2 commits intoGyulyVGC:test-appimage-fixfrom
AlleM43:squashed_commits
Sep 18, 2025
Merged

Fix AppImage error with libpcap#971
GyulyVGC merged 2 commits intoGyulyVGC:test-appimage-fixfrom
AlleM43:squashed_commits

Conversation

@AlleM43
Copy link
Contributor

@AlleM43 AlleM43 commented Sep 17, 2025

#859 (comment) Seems to have been caused by Debian 13 changing the name of the libpcap package. I've updated the manifest to use the new name.

@GyulyVGC
Copy link
Owner

Will this work with older distributions though? 🤔

@GyulyVGC
Copy link
Owner

And why was it happening even if we were using bookworm?

https://packages.debian.org/search?searchon=sourcenames&keywords=libpcap

@AlleM43
Copy link
Contributor Author

AlleM43 commented Sep 17, 2025

It was happening because the manifest was pulling the package from the repository for the latest version of Debian. I could've modified to keep it pulling from the bookworm repository, but since the .deb is now being built for trixie I decided to update that line and keep it pulling from stable.

@GyulyVGC GyulyVGC added packaging Application packaging bug Something isn't working Linux Something related to Linux operating system labels Sep 17, 2025
@GyulyVGC GyulyVGC added this to the v1.4.2 milestone Sep 17, 2025
@AlleM43
Copy link
Contributor Author

AlleM43 commented Sep 17, 2025

Will this work with older distributions though? 🤔

I did not consider this, though. libpcap from Trixie requires glibc 2.38. Seeing as the main sniffnet binary requires glibc 2.29, I'll be changing it to a version compatible as far back as possible

@GyulyVGC GyulyVGC changed the title Update AppImage to stable Debian. Fix AppImage error with libpcap Sep 17, 2025
@AlleM43
Copy link
Contributor Author

AlleM43 commented Sep 17, 2025

Changed it to use Debian 10 libpcap. It would be possible to use Debian 11, but that would cause breakage in August next year when it's archived.

@GyulyVGC
Copy link
Owner

GyulyVGC commented Sep 17, 2025

Thanks, I prefer this solution.
Why not keeping bookworm at this point?

I'm also wondering whether I should also fix something in the .deb package building and the Docker package (they're both built on a ubuntu-latest runner) 🤔

@AlleM43
Copy link
Contributor Author

AlleM43 commented Sep 17, 2025

Bookworm libpcap requires glibc 2.33, which would have prevented running it on, among others, Debian 11 and Ubuntu 20.04.

@GyulyVGC
Copy link
Owner

What about using ubuntu-latest for building the AppImage as well? Is that possible? So we're consistent with how I'm building the other packages.

@AlleM43
Copy link
Contributor Author

AlleM43 commented Sep 17, 2025

Please elaborate. In what way would ubuntu-latest be used in the AppImage build, beyond being the runner?

@GyulyVGC
Copy link
Owner

Using it as the dist in the ingredients section. But I'm not really sure if it's possible and meaningful.

@AlleM43
Copy link
Contributor Author

AlleM43 commented Sep 17, 2025

That would cause the same problem as using Debian 13, with the libpcap requiring too new versions of glibc.

@GyulyVGC
Copy link
Owner

GyulyVGC commented Sep 18, 2025

Makes sense.
Before merging I'd like to replicate the issue they pointed out to verify that this PR fixes it, but unluckily I'm not able to reproduce the problem.
The original AppImage built on bookworm with libpcap0.8 just works fine for me on my Debian:

image

EDIT: ok, I'm unsure why it's working on Debian, but I was able to finally replicate the issue on Fedora.

@AlleM43
Copy link
Contributor Author

AlleM43 commented Sep 18, 2025

If you remove the libpcap package from your host system you should be able to replicate the issue, or you can run the AppImage with the flag --appimage-extract and verify that the folder that's extracted doesn't contain libpcap.

@GyulyVGC GyulyVGC changed the base branch from main to test-appimage-fix September 18, 2025 08:58
@GyulyVGC GyulyVGC merged commit 665e9de into GyulyVGC:test-appimage-fix Sep 18, 2025
3 checks passed
@GyulyVGC
Copy link
Owner

Works fine now!!
This changes are now on main, and the release was updated to include the newly built AppImages.

Thanks so much for your responsiveness and availability.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working Linux Something related to Linux operating system packaging Application packaging

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants