all repos — openbox @ 24c0d618471276fd514cd666bc7067d30be8d0ca

openbox fork - make it a bit more like ryudo 

add the 'engine="box"' to the theme format, and require it to be there so we can change that in the future
Dana Jansens danakj@orodu.net
commit

24c0d618471276fd514cd666bc7067d30be8d0ca

parent

974391d345adf122816831cc24cbdfec5616aed1

11 files changed, 29 insertions(+), 12 deletions(-)

jump to
M data/themerc.xsd → data/themerc.xsd 
@@ -163,5 +163,6 @@ <xs:element name="menu" type="obt:allmenus" minOccurs="0"/>
       <xs:element name="window" type="obt:allwindows" minOccurs="0"/>
     </xs:all>
     <xs:attribute name="version" type="xs:integer" use="required" fixed="1"/>
+    <xs:attribute name="engine" type="xs:string" use="required" fixed="box"/>
   </xs:complexType>
 </xs:schema>
M parser/parse.c → parser/parse.c 
@@ -108,20 +108,35 @@ {
     GSList *it;
     gchar *path;
     gboolean r = FALSE;
+    gchar *eng;
 
     /* backward compatibility.. */
     path = g_build_filename(g_get_home_dir(), ".themes", name,
                             "openbox-3", "themerc.xml", NULL);
-    if ((r = parse_load(path, "openbox_theme", doc, root)))
-        *retpath = g_path_get_dirname(path);
+    if (parse_load(path, "openbox_theme", doc, root) &&
+        parse_attr_string("engine", *root, &eng))
+    {
+        if (!strcmp(eng, "box")) {
+            *retpath = g_path_get_dirname(path);
+            r = TRUE;
+        }
+        g_free(eng);
+    }
     g_free(path);
 
     if (!r) {
         for (it = xdg_data_dir_paths; !r && it; it = g_slist_next(it)) {
             path = g_build_filename(it->data, "themes", name, "openbox-3",
                                     "themerc.xml", NULL);
-            if ((r = parse_load(path, "openbox_theme", doc, root)))
-                *retpath = g_path_get_dirname(path);
+            if (parse_load(path, "openbox_theme", doc, root) &&
+                parse_attr_string("engine", *root, &eng))
+            {
+                if (!strcmp(eng, "box")) {
+                    *retpath = g_path_get_dirname(path);
+                    r = TRUE;
+                }
+                g_free(eng);
+            }
             g_free(path);
         }
     }
M themes/Artwiz-boxed/openbox-3/themerc.xml → themes/Artwiz-boxed/openbox-3/themerc.xml 
@@ -1,5 +1,5 @@ 
 <?xml version="1.0"?>
-<openbox_theme version="1" xmlns="http://openbox.org/themerc">
+<openbox_theme version="1" engine="box" xmlns="http://openbox.org/themerc">
   <dimensions>
     <handle>4</handle>
     <padding x="1" y="1"/>
M themes/Bear2/openbox-3/themerc.xml → themes/Bear2/openbox-3/themerc.xml 
@@ -1,5 +1,5 @@ 
 <?xml version="1.0"?>
-<openbox_theme version="1" xmlns="http://openbox.org/themerc">
+<openbox_theme version="1" engine="box" xmlns="http://openbox.org/themerc">
   <dimensions>
     <handle>4</handle>
     <padding x="3" y="3"/>
M themes/Clearlooks-Olive/openbox-3/themerc.xml → themes/Clearlooks-Olive/openbox-3/themerc.xml 
@@ -1,5 +1,5 @@ 
 <?xml version="1.0"?>
-<openbox_theme version="1" xmlns="http://openbox.org/themerc">
+<openbox_theme version="1" engine="box" xmlns="http://openbox.org/themerc">
   <dimensions>
     <handle>3</handle>
     <padding x="2" y="2"/>
M themes/Clearlooks/openbox-3/themerc.xml → themes/Clearlooks/openbox-3/themerc.xml 
@@ -1,5 +1,5 @@ 
 <?xml version="1.0"?>
-<openbox_theme version="1" xmlns="http://openbox.org/themerc">
+<openbox_theme version="1" engine="box" xmlns="http://openbox.org/themerc">
   <dimensions>
     <handle>3</handle>
     <padding x="2" y="2"/>
M themes/Mikachu/openbox-3/themerc.xml → themes/Mikachu/openbox-3/themerc.xml 
@@ -1,5 +1,5 @@ 
 <?xml version="1.0"?>
-<openbox_theme version="1" xmlns="http://openbox.org/themerc">
+<openbox_theme version="1" engine="box" xmlns="http://openbox.org/themerc">
   <dimensions>
     <handle>0</handle>
     <padding x="1" y="1"/>
M themes/Natura/openbox-3/themerc.xml → themes/Natura/openbox-3/themerc.xml 
@@ -1,5 +1,5 @@ 
 <?xml version="1.0"?>
-<openbox_theme version="1" xmlns="http://openbox.org/themerc">
+<openbox_theme version="1" engine="box" xmlns="http://openbox.org/themerc">
   <dimensions>
     <handle>2</handle>
     <padding x="2" y="2"/>
M themes/Orang/openbox-3/themerc.xml → themes/Orang/openbox-3/themerc.xml 
@@ -1,5 +1,5 @@ 
 <?xml version="1.0"?>
-<openbox_theme version="1" xmlns="http://openbox.org/themerc">
+<openbox_theme version="1" engine="box" xmlns="http://openbox.org/themerc">
   <dimensions>
     <handle>3</handle>
     <padding x="1" y="1"/>
M themes/Syscrash/openbox-3/themerc.xml → themes/Syscrash/openbox-3/themerc.xml 
@@ -1,5 +1,5 @@ 
 <?xml version="1.0"?>
-<openbox_theme version="1" xmlns="http://openbox.org/themerc">
+<openbox_theme version="1" engine="box" xmlns="http://openbox.org/themerc">
   <dimensions>
     <handle>0</handle>
     <padding x="1" y="1"/>
M tools/themetoxml/themetoxml.c → tools/themetoxml/themetoxml.c 
@@ -314,6 +314,7 @@ 
     doc = xmlNewDoc((const xmlChar*) "1.0");
     xmlDocSetRootElement
         (doc,(root = xmlNewNode(NULL, (const xmlChar*)"openbox_theme")));
+    xmlSetProp(root, (const xmlChar*)"engine", (const xmlChar*)"box");
     xmlSetProp(root, (const xmlChar*)"version", (const xmlChar*)"1");
     xmlSetProp(root, (const xmlChar*)"xmlns",
                (const xmlChar*)"http://openbox.org/themerc");