Age | Commit message (Collapse) | Author | Files | Lines |
|
|
|
the name
|
|
If the desktop environment was GNOME and none of the commands for
detecting the MIME type of a file was available, xdg-mime used to return
exit code 0 even though it failed. The man page says it should return
exit code 3. This commit fixes the issue.
I have considered returning exit code 4 instead since that is what would
be returned if info_kde() or info_generic() fail to find a tool for MIME
type detection. But I have decided to implement the behavior as
specified in the man page.
The exit code of an if construct is 0 if no condition
tested true [^1]. The author of the original code probably was not aware
of this.
[^1]: https://www.gnu.org/software/bash/manual/html_node/Conditional-Constructs.html#Conditional-Constructs
|
|
As per RFC 2396[1] a valid URI scheme may contain a digit:
scheme = alpha *( alpha | digit | + | - | . )
[1]: https://www.ietf.org/rfc/rfc2396.txt
|
|
|
|
|
|
|
|
|
|
|
|
Also bump Rex Dieter’s copyright date
Signed-off-by: Simon Lees <sflees@suse.de>
|
|
Although xdg-su is not currently shipped, fix some trivial documentation
TODOs.
Signed-off-by: Simon Lees <sflees@suse.de>
|
|
libdbus expects string arguments to be valid UTF-8. If they are not,
then it aborts, which causes our backgrounded command to terminate
abnormally.
Signed-off-by: Simon Lees <sflees@suse.de>
|
|
Signed-off-by: Simon Lees <sflees@suse.de>
|
|
xdg-terminal is not working when it needs gsettings to obtain the
default terminal. Thus, xdg-terminal cannot be used in MATE, Cinnamon
or GNOME. This issue was already reported in:
https://bugs.freedesktop.org/show_bug.cgi?id=93231
Thus, this patch provides a temporary workaround until upstream fixes
it.
Link: https://bugs.freedesktop.org/show_bug.cgi?id=93231
Signed-off-by: Simon Lees <sflees@suse.de>
|
|
It was observed that t-xdg-open.sh exits after only a fraction of the tests have
been run, e.g.
ASSERTION FAILED: expected command to be run: gio open http://www.freedesktop.org/
ASSERTION FAILED: expected command to be run: gio open http://www.freedesktop.org/
- opens a URL with gvfs-open if gio open is missing in GNOME 3, GNOME 2, and Cinnamon
gio: http://www.freedesktop.org/: Operation not supported
make: *** [Makefile:21: t-xdg-open.sh] Error 4
Given that some tests are failing on my machine (likely because of the
peculiarities of my archlinux install), the root cause is `set -e`. The nonzero
exit code returned by xdg-open gets returned by run(), which is also returned by
e.g. test_open_url(), which causes the script to exit immediately.
All test passes/failures at present are being defined by explicit assertions,
not by exit codes, and it seems at least plausible that some xdg-open calls are
meant to fail. So rather than report the nonzero error code, just ignore it, and
trust that the assertions determine the results.
Signed-off-by: Richard Tollerton <rich.tollerton@ni.com>
|
|
Presently, file://localhost/ URLs are totally unsupported: is_file_url_or_path
correctly considers them files, but they are undecoded and hence
check_input_file fails.
While the standardization surrounding file: URLs is admittedly vague [1], AFAIK,
*all* literature, and other implementations, unambiguously demonstrate that
file://localhost/ should be equivalent to file:///:
- The "File URI specification" explicitly linked to from the xdg-utils homepage [2]
- RFC 8089 section 1.1
- RFC 1738 section 3.10
- Observed implementations of Windows `start`, macOS `open`, Firefox, Chrome, IE
Fix this by adding some simple carve-outs for file://localhost specifically in
file_url_to_path.
[1] https://lists.freedesktop.org/archives/xdg/2004-November/003711.html
[2] https://edeproject.org/spec/file-uri-spec.txt
Signed-off-by: Richard Tollerton <rich.tollerton@ni.com>
|
|
Reverts a small part of prior commit
56991bc165577f011f9ad7ca721c5a5134710e33
|
|
|
|
|
|
|
|
|
|
Please enter the commit message for your changes. Lines starting
|
|
|
|
In DDE, a tools called `dde-open` is used to open preferred app.
DDE use `Deepin` as a desktop name currently,
but in some older version, it was `DEEPIN` or `deepin`
dde-open is provide by dde-api, see [1].
[1]: https://cr.deepin.io/c/dde/dde-api/+/33751
|
|
|
|
|
|
specified (BR106343)
patch courtesy of Simon Lees (SUSE) <sflees@suse.de>
BUG: 106343
|
|
|
|
Commit 6387086e4938d568c2bab185632f60e1619b3f68 introduced LXQt support.
In xdg-open it assumed that LXQt and LXDE are the same. They aren't. LXQt
does not have pcmanfm, it has pcmanfm-qt.
LXQt doesn't want to rely on pcmamfm-qt to handle it: LXQt is very modular
and can be deployed without pcmanfm-qt.
open_generic() works for LXQt but it depends on mimeopen and mimetype. In
the future we will want, for sure, to drop those dependencies. That's the
reason for the existence of open_lxqt().
|
|
|
|
Credit to Ville Skyttä <ville.skytta@iki.fi>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
The list of fallback browsers includes "chromium-browser"; however,
current installations of Chromium install the browser as "chromium".
(Some Chromium packages used to install "chromium-browser" due to binary
name conflicts, subsequently resolved.)
|
|
This was introduced in 0d6eebb27c562e8644ccf616ebdbddf82d0d2dd8 and
prevents the script from running altogether.
https://bugs.freedesktop.org/show_bug.cgi?id=99699
|
|
|
|
gvfs- utils were deprecated in favour of new gio tool. Prefer gio
tool in recent GNOME 3 and Cinnamon over gvfs- utils.
|
|
|
|
When we are in the flatpak sandbox, use the flatpak portal to open
mailto uris.
|
|
When we are in the flatpak sandbox, use the flatpak portal to open
uris.
|
|
|