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

No progress updates from flatpak_installation_install_bundle() #404

Open
hughsie opened this issue Nov 14, 2016 · 3 comments
Open

No progress updates from flatpak_installation_install_bundle() #404

hughsie opened this issue Nov 14, 2016 · 3 comments
Labels
bug libflatpak Issues involving the library ostree

Comments

@hughsie
Copy link
Collaborator

hughsie commented Nov 14, 2016

The libflatpak API call flatpak_installation_install_bundle() takes a FlatpakProgressCallback as an input, but never actually calls it. This means when you install a large .flatpak application (for instance, LibreOffice) you don't get any indication of progress. The progress bar goes from 0% and them jumps to 100% after about 40 seconds. This isn't awesome.

Looking at the code, the progress and progress_data variables look completely unused, so I guess they need to be passed to flatpak_dir_install_bundle() and handled in the system helper.

Thanks!

@alexlarsson
Copy link
Member

The reason they are not passed on is that in the end we're calling into ostree to do a local "apply static delta" operation, and there is no progress reporting in ostree for this.

@hughsie
Copy link
Collaborator Author

hughsie commented Nov 15, 2016

But you'd agree that reporting no progress for 40 seconds is a bug, right? Surely ostree knows how many parts there are to deploy in the delta?

@alexlarsson
Copy link
Member

Yes, ostree knows how large the files are etc. I guess the problem is that ostree doesn't consider "local file operations" to be "slow", because only things that do network i/o report progress.

@cgwalters any chance we could get progress reporting also for local operations like checkouts and applying static deltas?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug libflatpak Issues involving the library ostree
Projects
None yet
Development

No branches or pull requests

3 participants