x11-misc/xdg-utils: fix xdg-open with pcmanfm
authorMike Gilbert <floppym@gentoo.org>
Sun, 17 Jun 2018 18:26:54 +0000 (14:26 -0400)
committerMike Gilbert <floppym@gentoo.org>
Sun, 17 Jun 2018 18:30:01 +0000 (14:30 -0400)
Closes: https://bugs.gentoo.org/658186
Package-Manager: Portage-2.3.40_p14, Repoman-2.3.9_p246

x11-misc/xdg-utils/files/xdg-utils-1.1.3-xdg-open-pcmanfm.patch [new file with mode: 0644]
x11-misc/xdg-utils/xdg-utils-1.1.3-r1.ebuild [moved from x11-misc/xdg-utils/xdg-utils-1.1.3.ebuild with 95% similarity]

diff --git a/x11-misc/xdg-utils/files/xdg-utils-1.1.3-xdg-open-pcmanfm.patch b/x11-misc/xdg-utils/files/xdg-utils-1.1.3-xdg-open-pcmanfm.patch
new file mode 100644 (file)
index 0000000..1f6cc29
--- /dev/null
@@ -0,0 +1,21 @@
+From 31525d3855f876ddf2e29091b2e8d376f923e09e Mon Sep 17 00:00:00 2001
+From: Rex Dieter <rdieter@gmail.com>
+Date: Thu, 24 May 2018 14:40:53 -0500
+Subject: xdg-open: better pcmanfm check (BR106636,BR106161)
+
+diff --git a/scripts/xdg-open.in b/scripts/xdg-open.in
+index 630e63e..bf9da4c 100644
+--- a/scripts/xdg-open.in
++++ b/scripts/xdg-open.in
+@@ -451,7 +451,7 @@ open_lxde()
+ {
+     # pcmanfm only knows how to handle file:// urls and filepaths, it seems.
+-    if pcmanfm --help >/dev/null 2>&1 -a is_file_url_or_path "$1"; then
++    if pcmanfm --help >/dev/null 2>&1 && is_file_url_or_path "$1"; then
+         local file="$(file_url_to_path "$1")"
+         # handle relative paths
+-- 
+cgit v1.1
+
similarity index 95%
rename from x11-misc/xdg-utils/xdg-utils-1.1.3.ebuild
rename to x11-misc/xdg-utils/xdg-utils-1.1.3-r1.ebuild
index af9be47a61ba72fa0468dbec00a727213f96ec21..178f20fca728ad82c6827159d327a3489d2901fa 100644 (file)
@@ -34,6 +34,10 @@ DOCS=( README RELEASE_NOTES TODO ) # ChangeLog is bogus, see git instead
 
 RESTRICT="test" # Disabled because of sandbox violation(s)
 
+PATCHES=(
+       "${FILESDIR}"/xdg-utils-1.1.3-xdg-open-pcmanfm.patch
+)
+
 #S=${WORKDIR}/${MY_P}
 
 src_prepare() {
@@ -42,14 +46,9 @@ src_prepare() {
        # to run `autoconf` in ./ and `make scripts-clean` in ./scripts/ to refresh
        # all the files
        if [[ -d "${WORKDIR}/patch" ]]; then
-               eapply patch
+               eapply "${WORKDIR}/patch"
        fi
        eautoreconf
-
-       if pushd scripts &>/dev/null ; then
-               make scripts-clean
-               popd &>/dev/null
-       fi
 }
 
 src_configure() {