[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#47870] [PATCH 1/2] gnu: polkit-gnome: Add autostart .desktop file.
From: |
Brendan Tildesley |
Subject: |
[bug#47870] [PATCH 1/2] gnu: polkit-gnome: Add autostart .desktop file. |
Date: |
Mon, 19 Apr 2021 13:02:02 +0200 (CEST) |
> On 04/19/2021 9:36 AM Leo Prikler <leo.prikler@student.tugraz.at> wrote:
>
>
> Hi Brendan,
>
> Am Montag, den 19.04.2021, 08:56 +0200 schrieb Brendan Tildesley:
> > > On 04/18/2021 5:52 PM Leo Prikler <leo.prikler@student.tugraz.at>
> > > wrote:
> > >
> > >
> > > Hi,
> > >
> > > Am Sonntag, den 18.04.2021, 17:15 +0200 schrieb Brendan Tildesley:
> > > > > On 04/18/2021 4:47 PM Leo Prikler <
> > > > > leo.prikler@student.tugraz.at>
> > > > > wrote:
> > [...]
> > > > On second thought perhaps another option is just to use the MATE
> > > > polkit
> > > > agent instead? I tested it for changing network manager and it
> > > > worked
> > > > just the same, only the GUI looks slightly different. I was
> > > > searching
> > > > What do you think?
> > > If it works for XFCE, why not? Given that polkit-gnome has no
> > > dependents other than itself, should we perhaps also look into
> > > removing
> > > it?
> > >
> > One issue is that mate-polkit's .desktop file has the line
> > OnlyShowIn=MATE;
> > This prevents it from being launched in any other desktop. When
> > multiple
> > desktops are installed all these files sit in /run/current-
> > desktop/profile/etc/
> > and rely these lines to make them not appear in other desktops.
> >
> > I made a couple patches [attached] that adds XFCE to the the
> > .desktop's OnlyShowIn.
> > It's a little ugly though.
> This may potentially be bikeshedding, but what about defining a
> (potentially hidden) package, that inherits from mate-polkit (call it
> mate-polkit-for-xfce) and installs a .desktop file, that replaces MATE
> with XFCE? Ah, well, you'd also have to move the desktop file to a
> different location for there not to be a name clash.
>
I think it's good to think about. This way makes it more self contained.
> Other than that, the contents of your patches LGTM, but the ChangeLog
> is on a pretty verbose end. For instance, the message for 0001 should
> likely be rewritten as:
>
> ----
> gnu: polkit-mate: Enable autostarting in XFCE.
>
> Add XFCE to the OnlyShowIn field of the autostart .desktop file so it
> will be started by XFCE as well. This is for the same of making
> polkit-mate the de facto polkit agent for XFCE in Guix, since XFCE does
> not ship its own.
>
> A potential downside might be, that this desktop file ends up in the
> current system profile and can therefore be seen in XFCE
> configurations, that did not ask for it.
>
> * gnu/packages/mate.scm (polkit-mate)[#:phases]: Add 'patch-desktop-
> for-xfce'.
> ----
>
> You could also write significantly shorter commit messages, since
> "Enable autostarting in XFCE" already tells you everything the patch
> does.
>
> Regards, Leo
Yeah I suck at commit messages. I've attached new patches doing it this way
and confirmed it worked in a VM.
0001-gnu-Add-xfce-mate-polkit-autostart.patch
Description: Text Data
0002-gnu-xfce-Add-xfce-mate-polkit-autostart-to-inputs.patch
Description: Text Data
- [bug#47870] [PATCH 0/2] Autostart polkit-gnome in XFCE, Brendan Tildesley, 2021/04/18
- [bug#47870] [PATCH 1/2] gnu: polkit-gnome: Add autostart .desktop file., Brendan Tildesley, 2021/04/18
- [bug#47870] [PATCH 2/2] gnu: xfce: Start polkit-gnome agent on login., Brendan Tildesley, 2021/04/18
- [bug#47870] [PATCH 1/2] gnu: polkit-gnome: Add autostart .desktop file., Leo Prikler, 2021/04/18
- [bug#47870] [PATCH 1/2] gnu: polkit-gnome: Add autostart .desktop file., Brendan Tildesley, 2021/04/18
- [bug#47870] [PATCH 1/2] gnu: polkit-gnome: Add autostart .desktop file., Leo Prikler, 2021/04/18
- [bug#47870] [PATCH 1/2] gnu: polkit-gnome: Add autostart .desktop file., Brendan Tildesley, 2021/04/19
- [bug#47870] [PATCH 1/2] gnu: polkit-gnome: Add autostart .desktop file., Leo Prikler, 2021/04/19
- [bug#47870] [PATCH 1/2] gnu: polkit-gnome: Add autostart .desktop file.,
Brendan Tildesley <=
- [bug#47870] [PATCH 1/2] gnu: polkit-gnome: Add autostart .desktop file., Leo Prikler, 2021/04/19
- [bug#47870] [PATCH 1/2] gnu: polkit-gnome: Add autostart .desktop file., Brendan Tildesley, 2021/04/19
- [bug#47870] [PATCH 1/2] gnu: polkit-gnome: Add autostart .desktop file., Leo Prikler, 2021/04/19
- [bug#47870] [PATCH 1/2] gnu: polkit-gnome: Add autostart .desktop file., Brendan Tildesley, 2021/04/19
- [bug#47870] [PATCH 1/2] gnu: polkit-gnome: Add autostart .desktop file., Leo Prikler, 2021/04/19
- [bug#47870] [PATCH 1/2] gnu: polkit-gnome: Add autostart .desktop file., Brendan Tildesley, 2021/04/21
- bug#47870: [PATCH 1/2] gnu: polkit-gnome: Add autostart .desktop file., Leo Prikler, 2021/04/21