fix validate-appdata test

[iFoundSilentHouse]

closes #100

[tfuxu]

I think you should instead put the list inside a paragraph, so that formatting doesn't look weird when rendered

[iFoundSilentHouse]

Like this?

diff --git a/data/com.ranfdev.Geopard.metainfo.xml.in.in b/data/com.ranfdev.Geopard.metainfo.xml.in.in
index d2ffa1f..e27ffb2 100644
--- a/data/com.ranfdev.Geopard.metainfo.xml.in.in
+++ b/data/com.ranfdev.Geopard.metainfo.xml.in.in
@@ -88,10 +88,12 @@
     </release>
     <release version="1.1.0" date="2022-05-19">
       <description>
-        <p>Improved design, with completely new pages for downloads, input requests, external links, errors</p>
-        <p>Added more spacing between links to make them easier to click on small screens</p>
-        <p>Added zoom functionality, via shortcuts or directly from the popover menu</p>
-        <p>Streaming button (alpha) for some audio/video file types</p>
+        <p>
+          <li>Improved design, with completely new pages for downloads, input requests, external links, errors</li>
+          <li>Added more spacing between links to make them easier to click on small screens</li>
+          <li>Added zoom functionality, via shortcuts or directly from the popover menu</li>
+          <li>Streaming button (alpha) for some audio/video file types</li>
+        </p>
       </description>
     </release>
     <release version="1.0.1" date="2022-05-07">
'''

[iFoundSilentHouse]

Test failed that way with <p> is too short

[iFoundSilentHouse]

diff --git a/data/com.ranfdev.Geopard.metainfo.xml.in.in b/data/com.ranfdev.Geopard.metainfo.xml.in.in
index f35c019..8f58603 100644
--- a/data/com.ranfdev.Geopard.metainfo.xml.in.in
+++ b/data/com.ranfdev.Geopard.metainfo.xml.in.in
@@ -88,12 +88,10 @@
     </release>
     <release version="1.1.0" date="2022-05-19">
       <description>
-        <ul>
-          <li>Improved design, with completely new pages for downloads, input requests, external links, errors</li>
-          <li>Added more spacing between links to make them easier to click on small screens</li>
-          <li>Added zoom functionality, via shortcuts or directly from the popover menu</li>
-          <li>Streaming button (alpha) for some audio/video file types</li>
-        </ul>
+        <p><li>Improved design, with completely new pages for downloads, input requests, external links, errors</li></p>
+        <p><li>Added more spacing between links to make them easier to click on small screens</li></p>
+        <p><li>Added zoom functionality, via shortcuts or directly from the popover menu</li></p>
+        <p><li>Streaming button (alpha) for some audio/video file types</li></p>
       </description>
     </release>
     <release version="1.0.1" date="2022-05-07">

Failed with same error

[tfuxu]

No, you should put it like that:

<p>
    <ul>
        <li>Item 1</li>
        <li>Item 2</li>
        <li>Item 3</li>
    </ul>
</p>

[iFoundSilentHouse]

-        <ul>
-          <li>Improved design, with completely new pages for downloads, input requests, external links, errors</li>
-          <li>Added more spacing between links to make them easier to click on small screens</li>
-          <li>Added zoom functionality, via shortcuts or directly from the popover menu</li>
-          <li>Streaming button (alpha) for some audio/video file types</li>
-        </ul>
+        <p>
+          <ul>
+            <li>Improved design, with completely new pages for downloads, input requests, external links, errors</li>
+            <li>Added more spacing between links to make them easier to click on small screens</li>
+            <li>Added zoom functionality, via shortcuts or directly from the popover menu</li>
+            <li>Streaming button (alpha) for some audio/video file types</li>
+          </ul>
+        </p>

Failed with same error

[tfuxu]

I don't know how the packaging works in void, but can you use validate-relax in appstream-util?

Try patching data/meson.build line 50 by replacing validate with validate-relax.

[iFoundSilentHouse]

It worked! Thank you. Should I use it as main patch for void package?

[iFoundSilentHouse]

Also, I can add this as a commit in this pr. Is it needed?

[ranfdev]

thanks everyone