Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 915702 - gui-libs/xdg-desktop-portal-wlr: please install portals config file
Summary: gui-libs/xdg-desktop-portal-wlr: please install portals config file
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Matthew Thode ( prometheanfire )
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 915704
  Show dependency tree
 
Reported: 2023-10-14 02:33 UTC by Sam James
Modified: 2024-04-21 02:02 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Sam James archtester Gentoo Infrastructure gentoo-dev Security 2023-10-14 02:33:32 UTC
See bug 915356, but the gist is that with >=xdg-desktop-portals-1.18.0, Portals (and possibly DEs/WMs if they lack their own portal) must provide their own Portals config file.

Upstream have https://github.com/emersion/xdg-desktop-portal-wlr/commit/e88e00ca9bc1b2e0e91bc297ff95c882dd1829ff in master as an example, but it's not installed by the build system, so you'll need to install it yourself.

Please also check that the settings in there are appropriate and tweak if needed.
Comment 1 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2023-10-30 09:51:12 UTC
ping
Comment 2 Alexis 2023-12-12 05:26:44 UTC
Have just run into this issue myself.
Comment 3 Matthew Thode ( prometheanfire ) archtester Gentoo Infrastructure gentoo-dev Security 2023-12-12 05:52:44 UTC
I'm not sure what file name to call it, it'd expect to load if called in sway from `sway-portals.conf`, but this package is not sway specific.

I suppose I could install it as a doc in `/usr/share/doc/${PN}-${PV}/`
Comment 4 Pacho Ramos gentoo-dev 2023-12-19 10:39:07 UTC
Fedora installs it as wlroots-portals.conf
https://src.fedoraproject.org/rpms/xdg-desktop-portal-wlr/c/e1dbb6e51fefc1860ccad67754647481d6e12d58?branch=rawhide

I would simply follow the same route :/
Comment 5 Larry the Git Cow gentoo-dev 2023-12-19 16:28:17 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=0d54becbae04dcc8410b9c4b041cb3467d1bd6bd

commit 0d54becbae04dcc8410b9c4b041cb3467d1bd6bd
Author:     Matthew Thode <prometheanfire@gentoo.org>
AuthorDate: 2023-12-19 16:27:23 +0000
Commit:     Matthew Thode <prometheanfire@gentoo.org>
CommitDate: 2023-12-19 16:28:14 +0000

    gui-libs/xdg-desktop-portal-wlr: install wlroots-portals.conf
    
    Closes: https://bugs.gentoo.org/915702
    
    Signed-off-by: Matthew Thode <prometheanfire@gentoo.org>

 gui-libs/xdg-desktop-portal-wlr/xdg-desktop-portal-wlr-9999.ebuild | 6 ++++++
 1 file changed, 6 insertions(+)
Comment 6 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2023-12-19 16:51:21 UTC
default is likely wrong, should be meson_src_install. What about non live too?
Comment 7 Larry the Git Cow gentoo-dev 2023-12-20 10:04:05 UTC
The bug has been referenced in the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=9bd2a1aad27a3fc93543ae37a2bb2f1a7dc8867e

commit 9bd2a1aad27a3fc93543ae37a2bb2f1a7dc8867e
Author:     Sam James <sam@gentoo.org>
AuthorDate: 2023-12-20 10:02:48 +0000
Commit:     Sam James <sam@gentoo.org>
CommitDate: 2023-12-20 10:03:28 +0000

    gui-libs/xdg-desktop-portal-wlr: add portal config file to 0.7.0-r1
    
    Not just live.
    
    Bug: https://bugs.gentoo.org/915702
    Signed-off-by: Sam James <sam@gentoo.org>

 ...dg-desktop-portal-wlr-0.7.0-portal-config.patch | 32 ++++++++++
 .../xdg-desktop-portal-wlr-0.7.0-r1.ebuild         | 69 ++++++++++++++++++++++
 .../xdg-desktop-portal-wlr-9999.ebuild             |  1 +
 3 files changed, 102 insertions(+)

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=843c3f23884f3d8df9fec3deccc391836f2297ed

commit 843c3f23884f3d8df9fec3deccc391836f2297ed
Author:     Sam James <sam@gentoo.org>
AuthorDate: 2023-12-20 10:00:07 +0000
Commit:     Sam James <sam@gentoo.org>
CommitDate: 2023-12-20 10:03:28 +0000

    gui-libs/xdg-desktop-portal-wlr: fix src_install for live
    
    Bug: https://bugs.gentoo.org/915702
    Fixes: 0d54becbae04dcc8410b9c4b041cb3467d1bd6bd
    Signed-off-by: Sam James <sam@gentoo.org>

 gui-libs/xdg-desktop-portal-wlr/xdg-desktop-portal-wlr-9999.ebuild | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
Comment 8 Franz Berger 2024-04-07 13:01:47 UTC
I believe the install path at `/usr/share/xdg-desktop-portal/portals` is wrong, it should be in `/usr/share/xdg-desktop-portal` (one level up).

Other portals are also installed there, and setting `XDG_CURRENT_DESKTOP=wlroots` in labwc leads to a broken portal setup. Copying the file to `~/.config/xdg-desktop-portal/wlroots-portals.conf` (the per-user location) fixes this.

Also see portals.conf(5).
Comment 9 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2024-04-21 02:01:33 UTC
```
$ tree /usr/share/xdg-desktop-portal/
/usr/share/xdg-desktop-portal/
├── kde-portals.conf
├── portals
│   └── kde.portal
└── portals.conf

2 directories, 3 files
```

So yeah, config files belong in the top-level dir. Thanks!
Comment 10 Larry the Git Cow gentoo-dev 2024-04-21 02:02:44 UTC
The bug has been referenced in the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=607050f54f5cc337415cb516ca0a9e634213e713

commit 607050f54f5cc337415cb516ca0a9e634213e713
Author:     Sam James <sam@gentoo.org>
AuthorDate: 2024-04-21 02:02:14 +0000
Commit:     Sam James <sam@gentoo.org>
CommitDate: 2024-04-21 02:02:14 +0000

    gui-libs/xdg-desktop-portal-wlr: install portal config to right directory
    
    Bug: https://bugs.gentoo.org/915702
    Signed-off-by: Sam James <sam@gentoo.org>

 ...tal-wlr-0.7.0-r1.ebuild => xdg-desktop-portal-wlr-0.7.0-r2.ebuild} | 4 ++--
 ...portal-wlr-0.7.1.ebuild => xdg-desktop-portal-wlr-0.7.1-r1.ebuild} | 2 +-
 gui-libs/xdg-desktop-portal-wlr/xdg-desktop-portal-wlr-9999.ebuild    | 4 ++--
 3 files changed, 5 insertions(+), 5 deletions(-)