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

Bundle static versions of desktop-file-validate and zsyncmake #9

Open
probonopd opened this issue Jun 26, 2023 · 8 comments · May be fixed by #47
Open

Bundle static versions of desktop-file-validate and zsyncmake #9

probonopd opened this issue Jun 26, 2023 · 8 comments · May be fixed by #47
Milestone

Comments

@probonopd
Copy link
Member

probonopd commented Jun 26, 2023

To maintain drop-in compatibility, we also need to ship like we did in AppImageKit's appimagetool:

  • usr/bin/desktop-file-validate
  • usr/bin/zsyncmake

(fwiw, https://github.com/probonopd/static-tools/releases/tag/continuous has those, built statically on Alpine Linux.)

@TheAssassin
Copy link
Member

TheAssassin commented Jun 26, 2023

We should consider making zsync2 a core project again. Then we could utilize libzsync2 directly instead of having to build and ship separate tools.

Note: this just affects zsync2 itself, not AppImageUpdate.

@probonopd
Copy link
Member Author

utilize libzsync2

Maybe after 1.0

@TheAssassin
Copy link
Member

Since we want to make this repository be as self-contained as possible (except for AppImage core projects), I'll add build scripts for these inspired by the code in your repository just as we've done for mksquashfs. This is going to increase the build times slightly for now, but that is acceptable to me.

Is there a library for checking desktop entries, too?

@probonopd
Copy link
Member Author

probonopd commented Jun 26, 2023

Agree on making this repo as self-contained as possible (except for AppImage core projects). Mentioned static-tools only as a reference in case it is helpful.

I don't know a library for checking desktop entries but it shouldn't be all too hard to convert
https://gitlab.freedesktop.org/xdg/desktop-file-utils/-/blob/master/src/validate.c
into one. But then, we'd have to maintain it, so not sure whether it'd be worth the effort.

@TheAssassin TheAssassin added this to the 1.0.0 milestone Jun 26, 2023
@rodlie
Copy link

rodlie commented May 1, 2024

Would appreciate if you removed the need for desktop-file-validate, I upgraded appimagetool on my build server today and my build broke due to this (yes, my previous appimagetool was old, 2023-03-08, btw why have you wiped the git history?).

For various reasons I won't discuss I'm unable install desktop-file-validate, so I had to make a dummy file to work around this.

@probonopd
Copy link
Member Author

probonopd commented May 9, 2024

Well, I think we need to bundle a static build of desktop-file-validate (and other similar needed tools). Would that satisfy your use case @rodlie?

@probonopd probonopd linked a pull request May 9, 2024 that will close this issue
@ivan-hc
Copy link

ivan-hc commented Aug 12, 2024

Hi, I saw there's a PR pending for four months here.

I noticed that to use the new appimagetool we need to install "desktop-file-utils" so I come here in this repo and I've found this opened issue.

Any news?

@probonopd
Copy link
Member Author

Waiting on @TheAssassin.

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

Successfully merging a pull request may close this issue.

4 participants