GNU bug report logs - #50789
syncthing-gtk creates autostart file with wrong bin

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Leo Famulari <leo <at> famulari.name>
Cc: tracker <at> debbugs.gnu.org
Subject: bug#50789: closed (syncthing-gtk creates autostart file with
 wrong bin)
Date: Wed, 05 Jan 2022 21:14:03 +0000

[Message part 1 (text/plain, inline)]

Your message dated Wed, 5 Jan 2022 16:13:26 -0500
with message-id <YdYJ9vtYJax8pi2/@jasmine.lan>
and subject line Re: bug#50789: syncthing-gtk creates autostart file with wrong bin
has caused the debbugs.gnu.org bug report #50789,
regarding syncthing-gtk creates autostart file with wrong bin
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs <at> gnu.org.)


-- 
50789: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=50789
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]

From: John Kehayias <john.kehayias <at> protonmail.com>
To: "bug-guix <at> gnu.org" <bug-guix <at> gnu.org>
Subject: syncthing-gtk creates autostart file with wrong bin
Date: Fri, 24 Sep 2021 21:33:56 +0000

Hello,

syncthing-gtk has an option to enable autostart, which it does by creating the .desktop file in ~/.config/autostart However, this has the wrong exec line, getting the -real script instead of syncthing-gtk. This won't work as it needs to be run as syncthing-gtk. Namely, it produces:

Exec=/gnu/store/vf5h9jqhq40x8r46afaa0jgw7awg1361-syncthing-gtk-0.9.4.4-1.c46fbd8/bin/.syncthing-gtk-real

Instead of

Exec=/gnu/store/vf5h9jqhq40x8r46afaa0jgw7awg1361-syncthing-gtk-0.9.4.4-1.c46fbd8/bin/syncthing-gtk

This is due to syncthing-gtk getting its executable name to write to the .desktop file in get_executable(): https://salsa.debian.org/debian/syncthing-gtk/-/blob/master/syncthing_gtk/tools.py#L409 where due to wrapping it will find something that won't work when run directly.

How should this be solved in Guix? Should we patch this part of the code to explicitly rewrite the path to have "syncthing-gtk" instead of ".syncthing-gtk-real"? Related would be the discussion at https://lists.gnu.org/r/guix-devel/2021-09/msg00088.html which I will message separately.

Thanks,
John

[Message part 3 (message/rfc822, inline)]

From: Leo Famulari <leo <at> famulari.name>
To: John Kehayias via Bug reports for GNU Guix <bug-guix <at> gnu.org>
Cc: 50789-done <at> debbugs.gnu.org
Subject: Re: bug#50789: syncthing-gtk creates autostart file with wrong bin
Date: Wed, 5 Jan 2022 16:13:26 -0500

On Fri, Sep 24, 2021 at 09:33:56PM +0000, John Kehayias via Bug reports for GNU Guix wrote:
> Hello,
> 
> syncthing-gtk has an option to enable autostart, which it does by creating the .desktop file in ~/.config/autostart However, this has the wrong exec line, getting the -real script instead of syncthing-gtk. This won't work as it needs to be run as syncthing-gtk. Namely, it produces:
> 
> Exec=/gnu/store/vf5h9jqhq40x8r46afaa0jgw7awg1361-syncthing-gtk-0.9.4.4-1.c46fbd8/bin/.syncthing-gtk-real
> 
> Instead of
> 
> Exec=/gnu/store/vf5h9jqhq40x8r46afaa0jgw7awg1361-syncthing-gtk-0.9.4.4-1.c46fbd8/bin/syncthing-gtk

Fixed with commit c37559e81979232feee07aa1eb39faacb093c5ca

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.