openbox - menu.xml typos

[dentonlt]

Judging by the number of openbox threads, the extension is uber popular! Add to the list:

~/.config/openbox/menu.xml has a few typos that keep menu items from working: vi, a few aterm calls.

The original file is a link pointing to extension's original at /usr/local/etc/skel/.config/openbox/menu.xml. It's possible to remove the link and place ~/.config/openbox/menu.xml. It will be restored with /home/tc on reboots. Fixed version or patch (whatever your fancy):

menu.xml-edits

Code: [Select]

<?xml version="1.0" encoding="UTF-8"?>

<openbox_menu xmlns="http://openbox.org/3.4/menu">

<menu id="apps-tools-menu" label="Tools">
  <item label="Add to Bootlocal">
    <action name="Execute">
      <command>add2file /opt/bootlocal.sh</command>
    </action>
  </item>
  <item label="Add to Filetool">
    <action name="Execute">
      <command>add2file /opt/.filetool.lst</command>
    </action>
  </item>
  <item label="Add to Xfiletool">
    <action name="Execute">
      <command>add2file /opt/.xfiletool.lst</command>
    </action>
  </item>
  <item label="Apps Audit">
    <action name="Execute">
      <command>appsaudit</command>
    </action>
  </item>
  <item label="Screenshot">
    <action name="Execute">
      <command>screenshot.sh</command>
    </action>
  </item>
  <item label="Picture Viewer">
    <action name="Execute">
      <command>flpicsee</command>
    </action>
  </item>
  <item label="Run Command">
    <action name="Execute">
      <command>flrun</command>
    </action>
  </item> 
  <item label="TC Terminal Server">
    <action name="Execute">
      <command>aterm +tr +sb -T "TC Terminal Server" -e sudo /usr/bin/tc-terminal-server</command>
    </action>
  </item>
  <item label="Time via Internet Time Server">
    <action name="Execute">
      <command>sudo /usr/bin/getTime.sh</command>
    </action>
  </item>
  <item label="Top">
    <action name="Execute">
      <command>aterm +tr +sb -T "Top" -e /usr/bin/top</command>
    </action>
  </item>
  <item label="Vi">
    <action name="Execute">
      <command>aterm +tr +sb -T "Vi" -e /bin/vi</command>
    </action>
  </item>
  <item label="Xkill">
    <action name="Execute">
      <command>xkill</command>
    </action>
  </item>
  <item label="XVesa Setup">
    <action name="Execute">
      <command>aterm +tr +sb -T "X Setup" -e /usr/bin/xsetup.sh</command>
    </action>
  </item>
</menu>

<menu id="apps-term-menu" label="Shells">
  <item label="Dark">
    <action name="Execute">
      <command>aterm +tr -T "sh" -e /bin/sh</command>
    </action>
  </item>
  <item label="Light">
    <action name="Execute">
      <command>aterm +tr -bg white -fg black -cr blue -T "sh" -e /bin/sh</command>
    </action>
  </item>
  <item label="Transparent">
    <action name="Execute">
      <command>aterm -tr -bg white -fg black -cr blue -T "sh" -e /bin/sh</command>
    </action>
  </item>
  <menu id="apps-tools-submenu" label="Root Access">
    <item label="Dark">
      <action name="Execute">
        <command>aterm +tr -cr red -T "Root Shell" -e sudo -s</command>
      </action>
    </item>
    <item label="Light">
      <action name="Execute">
        <command>aterm +tr -bg white -fg black -cr red -T "Root Shell" -e sudo -s</command>
      </action>
    </item>
    <item label="Transparent">
      <action name="Execute">
        <command>aterm -tr -cr red -T "Root Shell" -e sudo -s</command>
      </action>
    </item>
  </menu>
</menu>

<menu id="system-menu" label="System">
  <item label="Openbox Configuration Manager">
    <action name="Execute">
      <command>obconf</command>
      <startupnotify><enabled>yes</enabled></startupnotify>
    </action>
  </item>
  <separator />
  <item label="Reconfigure Openbox">
    <action name="Reconfigure" />
  </item>
</menu>

<menu id="root-menu" label="Openbox 3">
  <separator label="TinyCoreLinux" />
  <menu id="apps-menu"/>
  <item label="Appbrowser">
    <action name="Execute">
      <command>appbrowser</command>
    </action>
  </item>
  <item label="Control Panel">
    <action name="Execute">
      <command>cpanel</command>
    </action>
  </item>
  <menu id="apps-tools-menu"/>
  <menu id="apps-term-menu"/>
  <separator label="System" />
  <menu id="system-menu"/>
  <separator />
  <item label="Exit">
    <action name="Execute">
      <command>exittc</command>
    </action>
  </item>
</menu>

</openbox_menu>

menu.xml-patch

Code: [Select]

--- menu.xml Wed Dec 23 18:32:41 2009
+++ menu.xml-fixed Wed Dec 23 18:32:08 2009
@@ -55,7 +55,7 @@
   </item>
   <item label="Vi">
     <action name="Execute">
-      <command>aterm +tr +sb -T "Vi" -e /usr/bin/vi</command>
+      <command>aterm +tr +sb -T "Vi" -e /bin/vi</command>
     </action>
   </item>
   <item label="Xkill">
@@ -78,12 +78,12 @@
   </item>
   <item label="Light">
     <action name="Execute">
-      <command>aterm +tr -bg white -fg black -cd blue -T "sh" -e /bin/sh</command>
+      <command>aterm +tr -bg white -fg black -cr blue -T "sh" -e /bin/sh</command>
     </action>
   </item>
   <item label="Transparent">
     <action name="Execute">
-      <command>aterm +tr -bg white -fg black -cd blue -T "sh" -e /bin/sh</command>
+      <command>aterm -tr -bg white -fg black -cr blue -T "sh" -e /bin/sh</command>
     </action>
   </item>
   <menu id="apps-tools-submenu" label="Root Access">
@@ -94,7 +94,7 @@
     </item>
     <item label="Light">
       <action name="Execute">
-        <command>aterm +tr -bg white -fg black -cd red -T "Root Shell" -e sudo -s</command>
+        <command>aterm +tr -bg white -fg black -cr red -T "Root Shell" -e sudo -s</command>
       </action>
     </item>
     <item label="Transparent">