[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#989708: libxfce4ui-common: xfce4-keyboard-shortcuts.xml is not valid XML



Package: libxfce4ui-common
Version: 4.16.0-1
Severity: important
Control: forwarded -1 https://gitlab.xfce.org/xfce/libxfce4ui/-/issues/44
Control: tags -1 patch

Dear maintainers,

xfce4-keyboard-shortcuts.xml, installed to
/etc/xdg/xfce4/xfconf/xfce-perchannel-xml/xfce4-keyboard-shortcuts.xml,
is not valid XML.

Indeed, from the line 88 to the line 95, <Super> is used instead of
`&lt;Super&gt;.

Sadly, this crashes libs used to parse XML, like Python's
xml.etree.ElementTree.

This bug has been reported upstream, but I've attached a patch to this
bug report. Maybe it's not too late for it to make it to Bullseye?

Cheers,

-- 
  ⢀⣴⠾⠻⢶⣦⠀
  ⣾⠁⢠⠒⠀⣿⡁  Louis-Philippe Véronneau
  ⢿⡄⠘⠷⠚⠋   pollo@debian.org / veronneau.org
  ⠈⠳⣄
diff --git a/debian/patches/0001_Invalid_XML.patch b/debian/patches/0001_Invalid_XML.patch
new file mode 100644
index 0000000..137a747
--- /dev/null
+++ b/debian/patches/0001_Invalid_XML.patch
@@ -0,0 +1,30 @@
+This patch fixes invalid XML introduced in upstream commit
+b7a75bf99a6fb4ffffebeb2476d71209bafef29d.
+See https://gitlab.xfce.org/xfce/libxfce4ui/-/issues/44 for more details.
+Index: libxfce4ui/libxfce4kbd-private/xfce4-keyboard-shortcuts.xml
+===================================================================
+--- libxfce4ui.orig/libxfce4kbd-private/xfce4-keyboard-shortcuts.xml
++++ libxfce4ui/libxfce4kbd-private/xfce4-keyboard-shortcuts.xml
+@@ -85,14 +85,14 @@
+       <property name="&lt;Primary&gt;F10" type="string" value="workspace_10_key"/>
+       <property name="&lt;Primary&gt;F11" type="string" value="workspace_11_key"/>
+       <property name="&lt;Primary&gt;F12" type="string" value="workspace_12_key"/>
+-      <property name="<Super>KP_Left" type="string" value="tile_left_key"/>
+-      <property name="<Super>KP_Right" type="string" value="tile_right_key"/>
+-      <property name="<Super>KP_Up" type="string" value="tile_down_key"/>
+-      <property name="<Super>KP_Down" type="string" value="tile_up_key"/>
+-      <property name="<Super>KP_Page_Up" type="string" value="tile_up_right_key"/>
+-      <property name="<Super>KP_Home" type="string" value="tile_up_left_key"/>
+-      <property name="<Super>KP_End" type="string" value="tile_down_left_key"/>
+-      <property name="<Super>KP_Next" type="string" value="tile_down_right_key"/>
++      <property name="&lt;Super&gt;KP_Left" type="string" value="tile_left_key"/>
++      <property name="&lt;Super&gt;KP_Right" type="string" value="tile_right_key"/>
++      <property name="&lt;Super&gt;KP_Up" type="string" value="tile_down_key"/>
++      <property name="&lt;Super&gt;KP_Down" type="string" value="tile_up_key"/>
++      <property name="&lt;Super&gt;KP_Page_Up" type="string" value="tile_up_right_key"/>
++      <property name="&lt;Super&gt;KP_Home" type="string" value="tile_up_left_key"/>
++      <property name="&lt;Super&gt;KP_End" type="string" value="tile_down_left_key"/>
++      <property name="&lt;Super&gt;KP_Next" type="string" value="tile_down_right_key"/>
+     </property>
+   </property>
+ </channel>
diff --git a/debian/patches/series b/debian/patches/series
new file mode 100644
index 0000000..b145cce
--- /dev/null
+++ b/debian/patches/series
@@ -0,0 +1 @@
+0001_Invalid_XML.patch

Attachment: OpenPGP_signature
Description: OpenPGP digital signature


Reply to: