<html><head></head><body><div>On lör, 2016-12-03 at 01:07 +0100, Jorge García wrote:</div><blockquote type="cite"><div>Hi,</div><div><br></div><div>I think this is a great addition that will make much easier the installation of flatpak apps. Please, take a look at this issue [1] for more ideas to make it even simpler.</div><div><br id="-x-evo-last-br"></div></blockquote><div><br></div><div>Yeah, that should be easy to add.</div><div><br></div><blockquote type="cite"><div dir="ltr"><div>Seen that you are improving the flatpakref files I have written some ideas/proposals that maybe could be useful too. You will find them below.<br><br>Thanks & best regards,<br>Jorge<br><br><br><br>* Consider adding a FlatpakRefSchema or FlatpakRefVersion field<br><br>  As the format of flatpakref files can be changed/improved in the future, this field can help parse them properly<br></div></div></blockquote><div><br></div><div>I think we can mostly extend them in compatible ways, but I guess we could add a version field so that in case we have to create an incompatible version we can make old versions fail on them in a sane way.</div><div><br></div><blockquote type="cite"><div dir="ltr"><div>* Consider using the same extension for .flatpakref and .flatpakrepo files<br><br>   gedit.flatpakref is a reference to the gedit application in a repository<br><br>   gnome-repository.flatpakref could be a reference to the gnome repository => Doesn't need to be named gnome.flatpakrepo (the flatpak program can read the contents and parse them accordingly)<br></div></div></blockquote><div><br></div><div>Its nice to have different names for them because then they get different mimetypes and the desktops can set different apps to handle them as needed.</div><div><br></div><blockquote type="cite"><div dir="ltr"><div>* Related to the new field RuntimeRepo, maybe you could add a RuntimeRepoIntegrity with the sha256 of the external file (or something a little bit more elaborated like browsers subresource-integrity [2]<br></div></div></blockquote><div><br></div><div>But then you would fail if some small part of the runtime repo file (like the title or icon) changes. We could store the GPG key though, that shouldn't change over time.</div><div><br></div><blockquote type="cite"><div dir="ltr"><div>* Consider adding a new field with the size of the app and required runtimes:<br><br>  Currently a user can't now beforehand how many megabytes will be downloaded when installing an app and its required runtimes. Is during the downloading process when some information is shown.<br></div></div></blockquote><div><br></div><div>We do have sizes stored in the summary files both for the app and the runtime, so its easy to calculate (without downloading), and I believe gnome-software does this already.</div><div><br></div><blockquote type="cite"><div dir="ltr"><div> What if the disk is almost full and the app and runtimes don't fit? flatpak should be able to warn the user before even beginning the downloading process.<br></div></div></blockquote><div><br></div><div>I dunno, that seems quite high level for a command line tool like flatpak. It would be nice if gnome-software did though.</div><div><br></div><blockquote type="cite"><div dir="ltr"><div>* Consider adding a new field with the arches (i386, x86_64, ...) available<br></div></div></blockquote><div><br></div><div>That is easily to figure out via the repo itself though, having it in the file just means you have to update it when the repo changes. I don't see the advantage in that.</div><div><br></div><blockquote type="cite"><div dir="ltr"><div>Currently flatpakref don't have any field with the info of the architectures this app is available in the repo. Is when the remote is added and queried when this info can be obtained. Maybe having this info beforehand could help flatpak/gnome software inform the user if the app can be installed.<br><br><br>* Consider adding a entry with the xdg_desktop_portal required to run the app<br><br>  It could be useful to know in the flatpakref file if the app requires and specific version of xdg_desktop_portal_XXX installed to run properly. See [3]<br></div></div></blockquote><div><br></div><div>Yeah, we haven't really figured out the story wrt versioning of things like portals (or other host-side services).</div><div><br></div><div><br></div><blockquote type="cite"><div dir="ltr"><div>* Mirrors<br><br>  Currently flatpakref files only have a url field. It could be useful to have a list of alternative mirrors<br></div></div></blockquote><div><br></div><div>OSTree actually supports mirror urls, so this should work on the lower level, however I believe it needs some minor work in flatpak to properly handle this.</div><div><br></div><blockquote type="cite"><div dir="ltr">* Repo requires authentication<br><div><br></div><div>  It could be useful to know if the repo requires some kind of user authentication (basic, oauth, ...) for downloading the apps/runtimes. THis could be used for private enterprise repos or for app purchases in a Flatpak App Store<br></div></div></blockquote><div><br></div><div>Well, first we need a story to support authentication at all. But yeah, that sounds useful.</div><div><br></div><blockquote type="cite"><div dir="ltr"><div>* Consider adding a entry about the required permissions that the app will need<br><br>  Users might want to know it the app requires to use the camera/whatever before installing it<br></div></div></blockquote><div><br></div><div>Same here, the repo has all this info in an efficient to get way, the idea is to avoid having to repeat the entire repo info in the flatpakref file so that you can create it once and then focus on maintaining your repo with up-to-date info.</div><div><span style="color: rgb(136, 136, 136);"><br></span></div><div><br></div><div><span><pre><pre>-- <br></pre>=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
 Alexander Larsson                                            Red Hat, Inc 
       alexl@redhat.com            alexander.larsson@gmail.com 
He's an impetuous coffee-fuelled paranormal investigator who hangs with 
the wrong crowd. She's a mentally unstable tomboy schoolgirl from the 
wrong side of the tracks. They fight crime! 
</pre></span></div></body></html>