Hi, list.

I added lxde/pcmanfm handling in my xdg-open and am wondering if i did
it correctly (see inline patch). It seems to work, fwiw, but i may be
missing something.


--- /usr/bin/xdg-open   2010-06-18 07:20:30.000000000 +0200
+++ xdg-open    2010-10-15 01:09:47.181690717 +0200
@@ -305,6 +305,7 @@
     elif [ x"$GNOME_DESKTOP_SESSION_ID" != x"" ]; then DE=gnome;
     elif `dbus-send --print-reply --dest=org.freedesktop.DBus 
/org/freedesktop/DBus org.freedesktop.DBus.GetNameOwner 
string:org.gnome.SessionManager > /dev/null 2>&1` ; then DE=gnome;
     elif xprop -root _DT_SAVE_MODE 2> /dev/null | grep ' = \"xfce4\"$' 
>/dev/null 2>&1; then DE=xfce;
+    elif [ x"$DESKTOP_SESSION" == x"LXDE"]; then DE=lxde;
     fi
 }
 
@@ -378,6 +379,17 @@
     fi
 }
 
+open_lxde()
+{
+    pcmanfm "$1"
+
+    if [ $? -eq 0 ]; then
+        exit_success
+    else
+        exit_failure_operation_failed
+    fi
+}
+
 open_generic_xdg_mime()
 {
     filetype=`xdg-mime query filetype "$1" | sed "s/;.*//"`
@@ -511,6 +523,10 @@
     open_xfce "$url"
     ;;
 
+    lxde)
+    open_lxde "$url"
+    ;;
+
     generic)
     open_generic "$url"
     ;;

-- 
Vincent Cappe

------------------------------------------------------------------------------
Download new Adobe(R) Flash(R) Builder(TM) 4
The new Adobe(R) Flex(R) 4 and Flash(R) Builder(TM) 4 (formerly 
Flex(R) Builder(TM)) enable the development of rich applications that run
across multiple browsers and platforms. Download your free trials today!
http://p.sf.net/sfu/adobe-dev2dev
_______________________________________________
Lxde-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/lxde-list

Reply via email to