GNU bug report logs - #25778
25.1; [PATCH] Drastically simplify xdg-open check

Previous Next

Package: emacs;

Reported by: Vasilij Schneidermann <v.schneidermann <at> gmail.com>

Date: Fri, 17 Feb 2017 18:18:02 UTC

Severity: normal

Tags: patch

Merged with 18986

Found in version 25.1

Fixed in version 26.1

Done: Paul Eggert <eggert <at> cs.ucla.edu>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Eli Zaretskii <eliz <at> gnu.org>
To: Vasilij Schneidermann <v.schneidermann <at> gmail.com>
Cc: 25778 <at> debbugs.gnu.org
Subject: bug#25778: 25.1; [PATCH] Drastically simplify xdg-open check
Date: Sat, 18 Feb 2017 09:58:18 +0200
> Date: Fri, 17 Feb 2017 19:55:14 +0100
> From: Vasilij Schneidermann <v.schneidermann <at> gmail.com>
> Cc: 25778 <at> debbugs.gnu.org
> 
> > First, checking $DISPLAY doesn't yet mean you are in a GUI frame.  We
> > have display-graphic-p for that.
> 
> The assumption that you need to be in a GUI frame is incorrect.  If I
> run `emacs -nw` in a graphical terminal emulator, why would that prevent
> me from opening a graphical browser?  The only way I see to detect
> whether a graphical browser can be opened is by checking for X with
> $DISPLAY.

In that case, maybe we shouldn't test $DISPLAY at all?  Why restrict
this to X if xdg-open is already tested for availability?

> Do you have any actual objections based on a situation where one would
> *not* want to use xdg-open, even though it's installed and the user in a
> X11 session?

See Glenn's response about that.  I think we should be more careful
with these issues, and at least do some more research of them before
we can be sure they are no longer relevant.  If you could follow up on
that, it would be great.

Thanks.




This bug report was last modified 7 years and 96 days ago.

Previous Next


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