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

Missing file in the tarball #590

Closed
2 tasks done
arthsmn opened this issue Mar 2, 2024 · 5 comments
Closed
2 tasks done

Missing file in the tarball #590

arthsmn opened this issue Mar 2, 2024 · 5 comments

Comments

@arthsmn
Copy link

arthsmn commented Mar 2, 2024

Checklist

  • I agree to follow the Code of Conduct that this project adheres to.
  • I have searched the issue tracker for a bug that matches the one I want to file, without success.

flatpak-builder version

1.4.2

Flatpak version

1.14.5

How to reproduce

  1. Try building flatpak-builder from the tarball provided (not the Github's automatic one).

Expected Behavior

It would build just fine.

Actual Behavior

It fails saying that the file glnx-backport-testutils.c is missing.

Additional Information

I'm trying to update flatpak-builder in nixpkgs. I downloaded the tarball manually and checked that the file is missing. In Gnome's Gitlab, the file is there, so I think it's a problem with the creation of the tarball.

The actual PR of the update: NixOS/nixpkgs#280152

@TingPing
Copy link
Member

TingPing commented Mar 2, 2024

Here is a fixed tarball for now: https://github.com/flatpak/flatpak-builder/releases/download/1.4.2/flatpak-builder-1.4.2-fixed-libglnx.tar.xz

@arthsmn
Copy link
Author

arthsmn commented Mar 3, 2024

It worked! Thanks!

@arthsmn arthsmn closed this as completed Mar 3, 2024
@smcv
Copy link
Collaborator

smcv commented Mar 7, 2024

Am I correct to think that the problem was the tarball being based on an outdated Makefile-libglnx.am, due to not regenerating subprojects/libglnx/Makefile-libglnx.am.inc by re-running autogen.sh after updating the submodule?

I think the long term fix for this is to remove the Autotools build system (#591). A step towards that could be to release 1.5.0 and up via meson dist (same as Flatpak 1.15.3+), so that if anyone still needs to build with Autotools, they still can, but they will have to run autogen.sh themselves. Flatpak 1.15.7 is going to remove Autotools completely.

@TingPing
Copy link
Member

TingPing commented Mar 7, 2024

due to not regenerating subprojects/libglnx/Makefile-libglnx.am.inc by re-running autogen.sh after updating the submodule?

Bingo.

@TingPing
Copy link
Member

TingPing commented Mar 7, 2024

A step towards that could be to release 1.5.0 and up via meson dist (same as Flatpak 1.15.3+), so that if anyone still needs to build with Autotools, they still can,

I just want to get rid of it in 1.5.0 personally. I'm really not sure what platforms both

  • Don't Meson available (a fairly conservative version can be used)
  • Package the latest version of flatpak-builder

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

3 participants