Package: xdg-desktop-portal / 1.6.0-1

Metadata

Package Version Patches format
xdg-desktop-portal 1.6.0-1 3.0 (quilt)

Patch series

view the series file
Patch File delta Description
Fix the build on old glib.patch | (download)

tests/Makefile.am.inc | 2 2 + 0 - 0 !
tests/backend/Makefile.am.inc | 2 2 + 0 - 0 !
tests/backend/email.c | 1 1 + 0 - 0 !
tests/filechooser.c | 2 2 + 0 - 0 !
tests/glib-backports.c | 24 24 + 0 - 0 !
tests/glib-backports.h | 8 8 + 0 - 0 !
6 files changed, 39 insertions(+)

 fix the build on old glib

Add backports for new api that we're using in tests.

xml Avoid fancy quotes.patch | (download)

data/org.freedesktop.portal.Email.xml | 12 6 + 6 - 0 !
1 file changed, 6 insertions(+), 6 deletions(-)

 xml: avoid fancy quotes

This breaks the build on older versions of gdbus-codegen, such
as the one in Ubuntu xenial.

open uri Fix criticals if no default handler for desired .patch | (download)

src/open-uri.c | 12 10 + 2 - 0 !
1 file changed, 10 insertions(+), 2 deletions(-)

 open-uri: fix criticals if no default handler for desired type

Signed-off-by: Simon McVittie <smcv@debian.org>

tests Fix race condition in various Lockdown tests.patch | (download)

tests/Makefile.am.inc | 2 2 + 0 - 0 !
tests/camera.c | 17 15 + 2 - 0 !
tests/filechooser.c | 16 14 + 2 - 0 !
tests/openuri.c | 16 14 + 2 - 0 !
tests/print.c | 30 26 + 4 - 0 !
tests/utils.c | 28 28 + 0 - 0 !
tests/utils.h | 9 9 + 0 - 0 !
7 files changed, 108 insertions(+), 10 deletions(-)

 tests: fix race condition in various lockdown tests

On the client side, functions like xdp_impl_lockdown_set_disable_printing
initiate an asynchronous property-set operation but do not wait for it
to finish. As a result, if we want other modules to have caught up with
the change, we have to wait for the set to succeed.

Signed-off-by: Simon McVittie <smcv@debian.org>

tests Install test backends.patch | (download)

Makefile.am | 1 1 + 0 - 0 !
tests/backend/Makefile.am.inc | 14 8 + 6 - 0 !
tests/test-portals.c | 4 3 + 1 - 0 !
3 files changed, 12 insertions(+), 7 deletions(-)

 tests: install test-backends

This is necessary for test-portals to work when run as an installed-test.

Signed-off-by: Simon McVittie <smcv@debian.org>

tests When installed look for executables in libexecdir.patch | (download)

tests/Makefile.am.inc | 2 2 + 0 - 0 !
tests/test-portals.c | 18 14 + 4 - 0 !
2 files changed, 16 insertions(+), 4 deletions(-)

 tests: when installed, look for executables in libexecdir

They won't normally be in the PATH when installed.

Signed-off-by: Simon McVittie <smcv@debian.org>

tests Install test.portal for installed tests.patch | (download)

tests/Makefile.am.inc | 6 2 + 4 - 0 !
1 file changed, 2 insertions(+), 4 deletions(-)

 tests: install test.portal for installed-tests

Otherwise, when running installed-tests, xdg-desktop-portal will
find a real installed backend such as xdg-desktop-portals-gtk (if any),
rather than using the intended test backend.

Fixes: https://github.com/flatpak/xdg-desktop-portal/issues/431
Signed-off-by: Simon McVittie <smcv@debian.org>

tests Skip Location tests if we have no D Bus system bus.patch | (download)

tests/location.c | 22 22 + 0 - 0 !
1 file changed, 22 insertions(+)

 tests: skip location tests if we have no d-bus system bus

We can't expect Geoclue to work in that situation. Ideally the location
portal would report this as an error, but at the moment the test just
hangs if we cannot connect to the system bus.

Signed-off-by: Simon McVittie <smcv@debian.org>

tests Divert service stdout to stderr.patch | (download)

tests/test-portals.c | 23 22 + 1 - 0 !
1 file changed, 22 insertions(+), 1 deletion(-)

 tests: divert service stdout to stderr

These tests produce structured TAP output on stdout, so we cannot run
any subprocesses that would produce unstructured stdout. Unfortunately,
by default g_debug() writes to stdout, so running with G_MESSAGES_DEBUG
to get better logging from the various subprocesses breaks the test.
Use stderr instead.

Signed-off-by: Simon McVittie <smcv@debian.org>

tests Always produce verbose output from subprocesses.patch | (download)

tests/test-portals.c | 6 3 + 3 - 0 !
1 file changed, 3 insertions(+), 3 deletions(-)

 tests: always produce verbose output from subprocesses

This helps to debug what is going on if a test fails.

Signed-off-by: Simon McVittie <smcv@debian.org>

Revert Stop building the icon validator.patch | (download)

configure.ac | 1 1 + 0 - 0 !
src/Makefile.am.inc | 5 5 + 0 - 0 !
src/validate-icon.c | 53 53 + 0 - 0 !
3 files changed, 59 insertions(+)

 revert "stop building the icon validator"

This reverts commit 2fb498f92f8e1c41049dedf887690083fa12f6d1.

validate icon Add a define for bwrap.patch | (download)

src/Makefile.am.inc | 2 1 + 1 - 0 !
1 file changed, 1 insertion(+), 1 deletion(-)

 validate-icon: add a #define for bwrap

Signed-off-by: Simon McVittie <smcv@debian.org>

Replace the icon validator with the one from Flatpak 1.6..patch | (download)

src/notification.c | 14 3 + 11 - 0 !
src/validate-icon.c | 212 204 + 8 - 0 !
2 files changed, 207 insertions(+), 19 deletions(-)

 replace the icon validator with the one from flatpak 1.6.0

For now we still want to avoid relying on Flatpak, since that dependency
would be quite circular.

Signed-off-by: Simon McVittie <smcv@debian.org>