[PATCH] cups: fix desktop file QA issues

[PATCH] cups: fix desktop file QA issues

[Chase Maupin]

* This patch fixes the QA issues with version 1.2.12 of cups.
* The patch being applied was taken from
  http://bugs.gentoo.org/show_bug.cgi?id=236706
* Bump the PR

Signed-off-by: Chase Maupin <Chase.Maupin@ti.com>
---
 .../0001-cups-desktop-fix-cups-desktop-file.patch  |   48 ++++++++++++++++++++
 recipes/cups/cups_1.2.12.bb                        |    4 +-
 2 files changed, 51 insertions(+), 1 deletions(-)
 create mode 100644 recipes/cups/cups-1.2.12/0001-cups-desktop-fix-cups-desktop-file.patch

diff --git a/recipes/cups/cups-1.2.12/0001-cups-desktop-fix-cups-desktop-file.patch b/recipes/cups/cups-1.2.12/0001-cups-desktop-fix-cups-desktop-file.patch
new file mode 100644
index 0000000..bb76d17
--- /dev/null
+++ b/recipes/cups/cups-1.2.12/0001-cups-desktop-fix-cups-desktop-file.patch
@@ -0,0 +1,48 @@
+From 85710dd8f749a447301e1522e6022b62e29d6f5d Mon Sep 17 00:00:00 2001
+From: Chase Maupin <Chase.Maupin@ti.com>
+Date: Wed, 9 Feb 2011 14:20:51 -0600
+Subject: [PATCH] cups desktop: fix cups desktop file
+
+* Fix the cups.desktop file to address issues reported by the
+  desktop-file-validate program.  This resolves QA error
+  issues generated by the insane class:
+    ERROR: QA Issue with cups: cups.desktop: warning: value
+        "Application;System;X-Red-Hat-Base;" for key "Categories" in group "
+        Desktop Entry" contains a deprecated value "Application"
+    ERROR: QA Issue with cups: cups.desktop: warning: key "Encoding" in group
+        "Desktop Entry" is deprecated
+    ERROR: QA Issue with cups: cups.desktop: warning: value "" for key
+        "Path" in group "Desktop Entry" does not look like an absolute path
+    ERROR: QA Issue with cups: cups.desktop: warning: key "TerminalOptions"
+        in group "Desktop Entry" is deprecated
+* This patch was obtained from http://bugs.gentoo.org/show_bug.cgi?id=236706
+  and thank to Pacho Ramos <pacho@gentoo.org> for the fix
+
+Signed-off-by: Chase Maupin <Chase.Maupin@ti.com>
+---
+ desktop/cups.desktop |    5 +----
+ 1 files changed, 1 insertions(+), 4 deletions(-)
+
+diff --git a/desktop/cups.desktop b/desktop/cups.desktop
+index dabfba3..28b98d9 100644
+--- a/desktop/cups.desktop
++++ b/desktop/cups.desktop
+@@ -1,14 +1,11 @@
+ [Desktop Entry]
+-Categories=Application;System;X-Red-Hat-Base;
+-Encoding=UTF-8
++Categories=System;X-Red-Hat-Base;
+ Exec=htmlview http://localhost:631/
+ GenericName=
+ Icon=cups
+ MimeType=
+-Path=
+ StartupNotify=false
+ Terminal=false
+-TerminalOptions=
+ Type=Application
+ X-DCOP-ServiceType=
+ X-KDE-SubstituteUID=false
+-- 
+1.7.0.4
+
diff --git a/recipes/cups/cups_1.2.12.bb b/recipes/cups/cups_1.2.12.bb
index 6df928a..167d353 100644
--- a/recipes/cups/cups_1.2.12.bb
+++ b/recipes/cups/cups_1.2.12.bb
@@ -1,5 +1,7 @@
 require cups.inc
-PR = "${INC_PR}.2"
+PR = "${INC_PR}.3"
+
+SRC_URI += "file://0001-cups-desktop-fix-cups-desktop-file.patch"
 
 SRC_URI[md5sum] = "d410658468384b5ba5d04a808f6157fe"
 SRC_URI[sha256sum] = "b4ff8e934da7db32d5654360ea9068faa0ed5a00fde02161ae53c2052510d00f"
-- 
1.7.0.4

Re: [PATCH] cups: fix desktop file QA issues

[Denys Dmytriyenko]

On Wed, Feb 09, 2011 at 02:36:59PM -0600, Chase Maupin wrote:
> * This patch fixes the QA issues with version 1.2.12 of cups.
> * The patch being applied was taken from
>   http://bugs.gentoo.org/show_bug.cgi?id=236706
> * Bump the PR
> 
> Signed-off-by: Chase Maupin <Chase.Maupin@ti.com>

Acked-by: Denys Dmytriyenko <denys@ti.com>

> ---
>  .../0001-cups-desktop-fix-cups-desktop-file.patch  |   48 ++++++++++++++++++++
>  recipes/cups/cups_1.2.12.bb                        |    4 +-
>  2 files changed, 51 insertions(+), 1 deletions(-)
>  create mode 100644 recipes/cups/cups-1.2.12/0001-cups-desktop-fix-cups-desktop-file.patch
> 
> diff --git a/recipes/cups/cups-1.2.12/0001-cups-desktop-fix-cups-desktop-file.patch b/recipes/cups/cups-1.2.12/0001-cups-desktop-fix-cups-desktop-file.patch
> new file mode 100644
> index 0000000..bb76d17
> --- /dev/null
> +++ b/recipes/cups/cups-1.2.12/0001-cups-desktop-fix-cups-desktop-file.patch
> @@ -0,0 +1,48 @@
> +From 85710dd8f749a447301e1522e6022b62e29d6f5d Mon Sep 17 00:00:00 2001
> +From: Chase Maupin <Chase.Maupin@ti.com>
> +Date: Wed, 9 Feb 2011 14:20:51 -0600
> +Subject: [PATCH] cups desktop: fix cups desktop file
> +
> +* Fix the cups.desktop file to address issues reported by the
> +  desktop-file-validate program.  This resolves QA error
> +  issues generated by the insane class:
> +    ERROR: QA Issue with cups: cups.desktop: warning: value
> +        "Application;System;X-Red-Hat-Base;" for key "Categories" in group "
> +        Desktop Entry" contains a deprecated value "Application"
> +    ERROR: QA Issue with cups: cups.desktop: warning: key "Encoding" in group
> +        "Desktop Entry" is deprecated
> +    ERROR: QA Issue with cups: cups.desktop: warning: value "" for key
> +        "Path" in group "Desktop Entry" does not look like an absolute path
> +    ERROR: QA Issue with cups: cups.desktop: warning: key "TerminalOptions"
> +        in group "Desktop Entry" is deprecated
> +* This patch was obtained from http://bugs.gentoo.org/show_bug.cgi?id=236706
> +  and thank to Pacho Ramos <pacho@gentoo.org> for the fix
> +
> +Signed-off-by: Chase Maupin <Chase.Maupin@ti.com>
> +---
> + desktop/cups.desktop |    5 +----
> + 1 files changed, 1 insertions(+), 4 deletions(-)
> +
> +diff --git a/desktop/cups.desktop b/desktop/cups.desktop
> +index dabfba3..28b98d9 100644
> +--- a/desktop/cups.desktop
> ++++ b/desktop/cups.desktop
> +@@ -1,14 +1,11 @@
> + [Desktop Entry]
> +-Categories=Application;System;X-Red-Hat-Base;
> +-Encoding=UTF-8
> ++Categories=System;X-Red-Hat-Base;
> + Exec=htmlview http://localhost:631/
> + GenericName=
> + Icon=cups
> + MimeType=
> +-Path=
> + StartupNotify=false
> + Terminal=false
> +-TerminalOptions=
> + Type=Application
> + X-DCOP-ServiceType=
> + X-KDE-SubstituteUID=false
> +-- 
> +1.7.0.4
> +
> diff --git a/recipes/cups/cups_1.2.12.bb b/recipes/cups/cups_1.2.12.bb
> index 6df928a..167d353 100644
> --- a/recipes/cups/cups_1.2.12.bb
> +++ b/recipes/cups/cups_1.2.12.bb
> @@ -1,5 +1,7 @@
>  require cups.inc
> -PR = "${INC_PR}.2"
> +PR = "${INC_PR}.3"
> +
> +SRC_URI += "file://0001-cups-desktop-fix-cups-desktop-file.patch"
>  
>  SRC_URI[md5sum] = "d410658468384b5ba5d04a808f6157fe"
>  SRC_URI[sha256sum] = "b4ff8e934da7db32d5654360ea9068faa0ed5a00fde02161ae53c2052510d00f"
> -- 
> 1.7.0.4
> 
> 
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel