[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#964846: reportbug-gtk: mua xdg-email: does not work with the GTK UI



On Sat, 2020-07-11 at 09:54 +0200, Nis Martensen wrote:

> Could you please test this with xdg-email and mailto URIs on the command
> line to narrow down the problem and file a bug with your desktop
> environment's mailto handling tool?

xdg-email works fine and xdg-open mailto: also, both in a terminal and
from the non-terminal-based GNOME command running dialog (Alt+F2). 

They also work with the reportbug text/urwid UIs, the issue only occurs
with the reportbug GTK UI for some reason.

If I add some debugging to the SystemPage execute function from the
gtk_ui.py file to print the xdg-email command that is run under a VTE
terminal. I then copied the command into a GNOME Terminal and it ran
there just fine. I then commented out the self.terminal.spawn_sync call
and the black screen in the reportbug UI stayed around instead of
closing immediately. Then I tested subprocess.run() instead of Vte
spawn_sync and it worked. Then I tested the mutt reportbug option and
it worked fine. Then I tested setting the reportbug mua option to the
evolution MUA (which xdg-email opens normally) and it worked fine. Then
I hacked the code to append "; sleep inf" to the command being run and
xdg-email worked fine. Then I tried "; sleep 0.1s" and it did not work.
0.5s does not work but 0.9s does work. So I think there is some sort of
race condition within VTE or something.

BTW, I think the GTK UI should not be running the MUAs under a VTE
terminal unless the MUA is marked as needing a terminal. Same for any
other commands it runs under a VTE terminal. For xdg-email probably
most of the commands that runs don't need a terminal.

BTW, I think that reportbug shouldn't be running commands via the shell
and definitely shouldn't be using bash even if shell really is needed.

BTW, while debugging I saw this deprecation warning:

<stdin>:1: DeprecationWarning: Vte.Terminal.spawn_sync is deprecated
https://developer.gnome.org/vte/unstable/VteTerminal.html#vte-terminal-spawn-sync

-- 
bye,
pabs

https://wiki.debian.org/PaulWise

Attachment: signature.asc
Description: This is a digitally signed message part


Reply to: