Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Timeshift 24.06.4 appdata file fails appstream-util validation #374

Open
darkshram opened this issue Dec 3, 2024 · 0 comments
Open

Timeshift 24.06.4 appdata file fails appstream-util validation #374

darkshram opened this issue Dec 3, 2024 · 0 comments

Comments

@darkshram
Copy link

darkshram commented Dec 3, 2024

Describe the bug
Timeshift 24.06.4 appdata file fails appstream-util validation

To Reproduce
Steps to reproduce the behavior:

  1. try to validate appstream data file with latest appstream-util (the tool used by most Linux/*BSD distributions to validate metainfo/appdata files)
  2. will fail
  3. Error message:
appstream-util validate-relax timeshift-24.06.4/debian/timeshift.appdata.xml 
timeshift-24.06.4/debian/timeshift.appdata.xml: FAILED:
• tag-missing           : <description> required
Validation of files failed

Expected behavior
Pass validation

System:

  • ALDOS 1.4
  • Desktop Xfce
  • Application Version: 24.06.4

The reason because it fails is because 1a7c1d9.

<p> tags should not be treated as xml tags adding xml:lang="cs". That's supposed to be for xml tags such as <description> tags, not the HTML tags. Example:

<description xml:lang:en>

Or

<description xml:lang:cs>

So, the fix for this issue is to revert 1a7c1d9.

diff -Naur timeshift-24.06.4.orig/debian/timeshift.appdata.xml timeshift-24.06.4/debian/timeshift.appdata.xml
--- timeshift-24.06.4.orig/debian/timeshift.appdata.xml	2024-12-03 04:15:14.000000000 -0600
+++ timeshift-24.06.4/debian/timeshift.appdata.xml	2024-12-03 10:29:01.995330182 -0600
@@ -10,29 +10,33 @@
 	<summary xml:lang="hr">Alat obnove sustava za Linux</summary>
 
 	<description>
-		<p xml:lang="en">
+		<p>
 			Timeshift protects your system by taking incremental snapshots of the file system at regular intervals.
 			These snapshots can be restored at a later date to undo all changes to the system. 
 		</p>
-		<p xml:lang="en">
+		<p>
 			Timeshift is designed to protect system files and settings.
 			It is NOT a backup tool and is not meant to protect user data.
 			Entire contents of users' home directories are excluded by default.
 		</p>
-		<p xml:lang="cs">
+	</description>
+	<description xml:lang="cs">
+		<p>
 			Timeshift chrání systém pořizováním přírůstkových zachycených stavů souborového systému v pravidelných intervalech.
 			Tyto zachycené stavy je možné později obnovit a vzít tak zpět veškeré změny v systému. 
 		</p>
-		<p xml:lang="cs">
+		<p>
 			Timeshift je navržen tak, aby chránil systémové soubory a nastavení.
 			NEJEDNÁ se o zálohovací nástroj a není určen k ochraně dat uživatelů.
 			Veškerý obsah domovských složek uživatelů je ve výchozím stavu vynecháván.
 		</p>
-		<p xml:lang="hr">
+	</description>
+	<description xml:lang="hr">
+		<p>
 			Timeshift štiti vaš sustav stvaranjem pojedinačnih snimaka datotečnog sustava u redovitim vremenskim razmacima.
 			Te snimke sustava kasnije se mogu obnoviti kako bi se poništile sve promjene načinjene u sustavu. 
 		</p>
-		<p xml:lang="hr">
+		<p>
 			Timeshift je dizajniran kako bi zaštitio vaše datoteke i postavke sustava.
 			Timeshift nije alat za sigurnosno kopiranje i namjena mu nije zaštita korisničkih podataka.
 			Sav sadržaj osobne mape korisnika po zadanome je izuzet.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

1 participant