summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Fix two tiny memory leakswip/hughsie/trivial-memleaksRichard Hughes2018-07-102-2/+3
|
* doc: Fix grammarMatthew Leeds2018-07-101-2/+2
|
* transaction: drop installation caches when neededAlexander Larsson2018-07-101-0/+10
| | | | | | | | | | If a remote is added or modified in the cloned FlatpakDir we also drop caches in the one in the installation. Fixes #1882 Closes: #1883 Approved by: alexlarsson
* update-remote-config: Fix potential NULL dereferenceAlexander Larsson2018-07-101-1/+2
| | | | | | | This fixed coverity issue 1471683. Closes: #1880 Approved by: mwleeds
* dir: Use flatpak_dir_use_system_helper() moreMatthew Leeds2018-07-091-7/+3
| | | | | | | | Use the helper function to decide whether to use the system helper in one more place. Closes: #1874 Approved by: alexlarsson
* system-helper: Fix race condition with idle exitAlexander Larsson2018-07-091-219/+349
| | | | | | | | | | | | | | | | If the system helper process exits due to being idle the FlatpakSystemHelper dbus proxy will return errors on all calls, because it resolves the well known name to a unique name on creation, which means it will try to talk to the old idle-exited instance. The fix is to not use GDBusProxy, but instead manually call g_dbus_connection_call_sync(). Fixes #1845 Closes: #1873 Approved by: mwleeds
* Update Polish translationPiotr Drąg2018-07-081-388/+468
| | | | | Closes: #1872 Approved by: alexlarsson
* Run uncrustifyAlexander Larsson2018-07-0881-3456/+3745
| | | | | Closes: #1870 Approved by: alexlarsson
* uncrustify: Don't modify valgrind header.Alexander Larsson2018-07-081-1/+1
| | | | | Closes: #1870 Approved by: alexlarsson
* uncrustify: Don't modify bracketsAlexander Larsson2018-07-081-6/+7
| | | | | | | | | I don't think there is really an acceptable setting that does the right thing always, so disable any adding/removing of brackets. Closes: #1870 Approved by: alexlarsson
* uncrustify.sh: We no longer have gvdb in the repoAlexander Larsson2018-07-081-1/+1
| | | | | Closes: #1870 Approved by: alexlarsson
* Fix error handling while deploying AppStreamSimon McVittie2018-07-071-1/+1
| | | | | | | | | | | | Setting an error with second_error->message is going to work poorly when second_error has never been set non-NULL. Related to #1845, although not necessarily the full solution. Signed-off-by: Simon McVittie <smcv@collabora.com> Closes: #1867 Approved by: alexlarsson
* transaction: Move the handling of bundle runtime remote from cli codeAlexander Larsson2018-07-072-161/+107
| | | | | | | | This allows libflatpak users to also add runtime remotes as needed by bundles. Closes: #1868 Approved by: alexlarsson
* transaction: Move flatpakref installation to transaction codeAlexander Larsson2018-07-078-124/+395
| | | | | | | | | | | | | | | | This adds the flatpak_transaction_add_install_flatpakref method to easily install from flatpakref files. It additionally adds a new signal called add-new-remote which is called in two cases: To ask if the user wants to add a "normal" remote for the url specified by a flatpakref file. When a new remote is required for the runtimes the application depends on. Closes: #1868 Approved by: alexlarsson
* transaction: Add flatpak_transaction_set_default_archAlexander Larsson2018-07-073-0/+15
| | | | | | | | This will be used as the default when installing something that doesn't specify a ref, such as a flatpakref file. Closes: #1868 Approved by: alexlarsson
* WIP: Add a create-usb commandMatthew Leeds2018-07-079-0/+986
| | | | | | | Fixes https://github.com/flatpak/flatpak/issues/1672 Closes: #1828 Approved by: alexlarsson
* doc: Point to the source of the generated HTMLMatthew Leeds2018-07-071-0/+5
| | | | | | | | When you're looking at the Flatpak Command Reference on docs.flatpak.org, it's not clear that it was generated from the flatpak repo. So add a notice to the docbook that's used to generate the HTML. This was pointed out by https://github.com/flatpak/flatpak-docs/pull/134
* tests: Fix up ioctl suppressions for testsAlexander Larsson2018-07-071-1/+0
| | | | | | The FICLONE ioctl suppression was hit via a different codepath too, so tweak the callstack for the suppression.
* system-helper: Add EnsureRepo operationAlexander Larsson2018-07-073-72/+138
| | | | | | | | | | | | This is used to create the /var/lib/flatpak repo if needed so that other later operations work. We have some partial support for it not working in various operations (using the allow_empty argument) but this is in no way complete. For example, this can easily happen if you have a per-user installation but no system one and then you run flatpak install with no --user, then it will try to figure out which one to use and die.
* testlibrary: Capture stderr from more commandsSimon McVittie2018-07-071-36/+11
| | | | | | | | | | This is the equivalent of commit 86fd56dce for the commands added in commit 2465cf4 to test #1587. Signed-off-by: Simon McVittie <smcv@debian.org> Closes: #1864 Approved by: alexlarsson
* testlibrary: Remove unused argv_strSimon McVittie2018-07-071-1/+0
| | | | | | | | | This was left over after commit 86fd56d. Signed-off-by: Simon McVittie <smcv@debian.org> Closes: #1864 Approved by: alexlarsson
* flatpak_dir_deploy_appstream: Handle the no-local-commit exists caseAlexander Larsson2018-07-051-0/+3
| | | | | | | | For some reason I hit this with a test remote, and it crashes. Make it return an error instead. Closes: #1854 Approved by: alexlarsson
* Update Polish translationPiotr Drąg2018-07-021-45/+122
| | | | | Closes: #1849 Approved by: alexlarsson
* build: Include built files from app/Simon McVittie2018-07-021-0/+1
| | | | | | | | | This fixes an out-of-tree build. Signed-off-by: Simon McVittie <smcv@collabora.com> Closes: #1848 Approved by: alexlarsson
* build: Tell make how to build app/*-dbus-generated.hSimon McVittie2018-07-021-0/+2
| | | | | | | | | This was already set up for common/, but not yet for app/. Signed-off-by: Simon McVittie <smcv@collabora.com> Closes: #1848 Approved by: alexlarsson
* Update Ukrainian translationYuri Chornoivan2018-07-021-347/+262
| | | | | Closes: #1847 Approved by: alexlarsson
* dir: Improve metadata fetch error messageMatthew Leeds2018-07-021-2/+12
| | | | | | | | | | | | | | | | | In the case that the collection ID configured on a remote is NULL the repo metadata comes from the summary file. But if there's an error fetching the summary in _flatpak_dir_get_remote_state() and then flatpak_remote_state_ensure_metadata() is called, the metadata_fetch_error variable will be NULL and "unknown error" will be returned as the error message. This commit returns the summary_fetch_error message in that case so the user gets a slightly more helpful error message. Some Endless computers in Africa have recently been hitting this code path and printing "unknown error" messages. Closes: #1844 Approved by: alexlarsson
* Fixed a 'maybe uninitialized' warning and a code style change.Grzegorz Antoniak2018-07-021-3/+3
| | | | | Closes: #1851 Approved by: alexlarsson
* NULL pointer check after calling getgrgid()Grzegorz Antoniak2018-07-021-4/+11
| | | | | | | | | | | | When the user runs with gid=X, but X doesn't have an entry in /etc/groups, then a segmentation fault occured. This was happening because flatpak-run.c didn't guard against a NULL pointer after calling getgrgid() in flatpak_run_setup_base_argv. Closes: #1850 Closes: #1851 Approved by: alexlarsson
* Fix parallel buildsMatthias Clasen2018-06-291-1/+2
| | | | | | | | | | With the newly introduced generated sources in app/, the (parallel) GNOME ci builds were failing. Add the generated sources to BUILT_SOURCES to fix that. For good measure, also add them to CLEAN_FILES. Closes: #1843 Approved by: mwleeds
* doc/flatpak.xml: Add permission-* commandsMatthew Leeds2018-06-291-0/+32
|
* Update POTFILES.inPiotr Drąg2018-06-291-0/+4
| | | | | Closes: #1842 Approved by: mwleeds
* doc: Fix minor mistakesMatthew Leeds2018-06-291-4/+1
|
* transaction: Add get_installation()Alexander Larsson2018-06-293-0/+18
| | | | | Closes: #1841 Approved by: alexlarsson
* transaction: Mark error const in operation-error signalAlexander Larsson2018-06-292-2/+2
| | | | | Closes: #1841 Approved by: alexlarsson
* Fix a copy-paste errorMatthias Clasen2018-06-291-1/+1
| | | | | | | RevokePermissions revokes permissions, not grants them. Closes: #1837 Approved by: alexlarsson
* Add a permission-reset commandMatthias Clasen2018-06-297-0/+317
| | | | | | | | This command removes all permissions for a given app from the permission store. Closes: #1837 Approved by: alexlarsson
* Add a permission-show commandMatthias Clasen2018-06-297-0/+341
| | | | | | | | This shows all the entries pertaining to a particular app from the permission store. Closes: #1837 Approved by: alexlarsson
* Add a permission-remove commandMatthias Clasen2018-06-297-0/+309
| | | | | | | This removes an item from the permission store. Closes: #1837 Approved by: alexlarsson
* Add a permission-list commandMatthias Clasen2018-06-297-0/+404
| | | | | | | | This command dumps the content of the permission store database(s). Closes: #1837 Approved by: alexlarsson
* Generate code for permission store proxyMatthias Clasen2018-06-291-0/+13
| | | | | | | | This commit generates the code for the permission store interface. Subsequent commits will use it. Closes: #1837 Approved by: alexlarsson
* Add the permission store interfaceMatthias Clasen2018-06-292-0/+164
| | | | | | | | | This adds the xml interface definition for org.freedesktop.impl.portal.PermissionStore. Subsequent commits will use this interface. Closes: #1837 Approved by: alexlarsson
* CI: Don't build ostreeAlexander Larsson2018-06-291-11/+2
| | | | | | | ostree 2018.6 is in updates-testing now, use that. Closes: #1840 Approved by: alexlarsson
* tests: Run tests on /tmp (not /var/tmp)Alexander Larsson2018-06-291-2/+1
| | | | | | | Since we now are ok with no-xattrs this means we can run on tmpfs. Closes: #1839 Approved by: alexlarsson
* tests: Don't skip if no user xattrsAlexander Larsson2018-06-299-16/+0
| | | | | | | We don't require these anymore Closes: #1839 Approved by: alexlarsson
* Include appstream and icons in OCI bundles as annotationsOwen W. Taylor2018-06-292-54/+163
| | | | | | | | | | | | | | | | | | | Include annotations: org.freedesktop.appstream.appdata org.freedesktop.appstream.icon-{64,128} into OCI bundles. This not only makes the bundle self-describing, but also if the bundle is imported into a registry, it becomes possible to browse the registry and recreate an appstream by only retrieving the annotations for relevant images, without having to download the actual images. The icon annotations are formatted as data: URI's with base64 data. The idea is that a server-side process to collect annotations could extract the icons to separate storage and rewrite the URI's to remote URL's. Closes: #1834 Approved by: alexlarsson
* Update Polish translationPiotr Drąg2018-06-291-222/+149
| | | | | Closes: #1836 Approved by: alexlarsson
* common/dir: Fix a couple of leaks of GKeyFile config objectsPhilip Withnall2018-06-271-2/+2
| | | | | | | | | ostree_repo_copy_config() returns (transfer full). Signed-off-by: Philip Withnall <withnall@endlessm.com> Closes: #1831 Approved by: mwleeds
* Update pofiles0.99.2Alexander Larsson2018-06-2713-65/+65
|
* Update version to 0.99.2Alexander Larsson2018-06-271-1/+1
|