From: Liliana Marie Prikler <liliana.prikler@gmail.com>
To: John Kehayias <john.kehayias@protonmail.com>, 50789@debbugs.gnu.org
Subject: bug#50789: syncthing-gtk creates autostart file with wrong bin
Date: Sat, 25 Sep 2021 00:55:56 +0200 [thread overview]
Message-ID: <33a689c52f5896213b480e01fc3498a26877b392.camel@gmail.com> (raw)
In-Reply-To: <xULpkaOJk5Q64f48ce9xy1griAdpvLDSdgaTcyiZyHYK2xOZWIcYQG9VAQlmPfp2n4MDehVPHbUH4wjEebJVQ5C-UrF7RqOYVs7ucq5EECQ=@protonmail.com>
Hi,
Am Freitag, den 24.09.2021, 21:33 +0000 schrieb John Kehayias:
> 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.
We should patch syncthing to not write a desktop file at all. Note how
even if the binary name itself was correct, syncthing would link
directly into the store, potentially producing a stale desktop file
that breaks with garbage collection.
Cheers
next prev parent reply other threads:[~2021-09-24 22:57 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-09-24 21:33 bug#50789: syncthing-gtk creates autostart file with wrong bin John Kehayias via Bug reports for GNU Guix
2021-09-24 22:55 ` Liliana Marie Prikler [this message]
2021-09-24 23:15 ` Leo Famulari
2021-09-25 0:45 ` John Kehayias via Bug reports for GNU Guix
2021-09-25 7:02 ` Liliana Marie Prikler
2021-09-27 19:04 ` John Kehayias via Bug reports for GNU Guix
2021-09-27 21:23 ` Liliana Marie Prikler
2021-09-30 19:29 ` John Kehayias via Bug reports for GNU Guix
2021-09-30 20:00 ` Liliana Marie Prikler
2021-09-30 20:13 ` John Kehayias via Bug reports for GNU Guix
2021-10-01 7:44 ` Liliana Marie Prikler
2022-01-05 19:31 ` John Kehayias via Bug reports for GNU Guix
2022-01-05 21:13 ` Leo Famulari
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://guix.gnu.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=33a689c52f5896213b480e01fc3498a26877b392.camel@gmail.com \
--to=liliana.prikler@gmail.com \
--cc=50789@debbugs.gnu.org \
--cc=john.kehayias@protonmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/guix.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).