Re: [OE-core] [oe-core][PATCH] powertop: update to v2.9

2017-08-01 Thread Burton, Ross
> +S = "${WORKDIR}/${PN}-v${PV}"

ERROR: lib32-powertop-2.9-r0 do_populate_lic: QA Issue: lib32-powertop:
LIC_FILES_CHKSUM points to an invalid file:
/home/pokybuild/yocto-autobuilder/yocto-worker/nightly-world/build/build/tmp/work/x86-pokymllib32-linux/lib32-powertop/2.9-r0/lib32-powertop-v2.9/COPYING
[license-checksum]

BPN, not PN.

Ross
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [oe-core][PATCH] powertop: update to v2.9

2017-07-31 Thread Tim Orling
* Upstream filename added "v" before version:
  - Add UPSTREAM_CHECK_REGEX
  - Update SRC_URI to include the "v"
  - Point ${S} to proper directory

* Patch for required headers no longer needed in current upstream

Signed-off-by: Tim Orling 
---
 ...0001-include-rquired-headers-for-typedefs.patch | 47 --
 .../powertop/{powertop_2.8.bb => powertop_2.9.bb}  | 10 +++--
 2 files changed, 6 insertions(+), 51 deletions(-)
 delete mode 100644 
meta/recipes-kernel/powertop/powertop/0001-include-rquired-headers-for-typedefs.patch
 rename meta/recipes-kernel/powertop/{powertop_2.8.bb => powertop_2.9.bb} (77%)

diff --git 
a/meta/recipes-kernel/powertop/powertop/0001-include-rquired-headers-for-typedefs.patch
 
b/meta/recipes-kernel/powertop/powertop/0001-include-rquired-headers-for-typedefs.patch
deleted file mode 100644
index 0fd6310..000
--- 
a/meta/recipes-kernel/powertop/powertop/0001-include-rquired-headers-for-typedefs.patch
+++ /dev/null
@@ -1,47 +0,0 @@
-From 0856d8145d187a7e5a49625247abe43a13f95acc Mon Sep 17 00:00:00 2001
-From: Khem Raj 
-Date: Tue, 15 Sep 2015 19:36:28 +
-Subject: [PATCH] include rquired headers for typedefs
-
-timeval struct needs to include sys/time.h and sprintf() usage requires
-to include stdio.h headers from system
-
-Fixes
-src/perf/perf_bundle.cpp:141:2: error: use of undeclared identifier 'sprintf'; 
did you mean 'vswprintf'?
-src/devices/devfreq.h:35:18: error: field has incomplete type 'struct timeval'
-
-Signed-off-by: Khem Raj 

-Upstream-Status: Pending
-
- src/devices/devfreq.h| 1 +
- src/perf/perf_bundle.cpp | 1 +
- 2 files changed, 2 insertions(+)
-
-diff --git a/src/devices/devfreq.h b/src/devices/devfreq.h
-index 16a60fb..4bede7c 100644
 a/src/devices/devfreq.h
-+++ b/src/devices/devfreq.h
-@@ -25,6 +25,7 @@
- #ifndef _INCLUDE_GUARD_DEVFREQ_H
- #define _INCLUDE_GUARD_DEVFREQ_H
- 
-+#include 
- #include "device.h"
- #include "../parameters/parameters.h"
- 
-diff --git a/src/perf/perf_bundle.cpp b/src/perf/perf_bundle.cpp
-index cf1ae11..232f894 100644
 a/src/perf/perf_bundle.cpp
-+++ b/src/perf/perf_bundle.cpp
-@@ -27,6 +27,7 @@
- #include 
- #include 
- #include 
-+#include 
- #include 
- #include 
- #include 
--- 
-2.5.2
-
diff --git a/meta/recipes-kernel/powertop/powertop_2.8.bb 
b/meta/recipes-kernel/powertop/powertop_2.9.bb
similarity index 77%
rename from meta/recipes-kernel/powertop/powertop_2.8.bb
rename to meta/recipes-kernel/powertop/powertop_2.9.bb
index 4d7a3e7..93ed5ba 100644
--- a/meta/recipes-kernel/powertop/powertop_2.8.bb
+++ b/meta/recipes-kernel/powertop/powertop_2.9.bb
@@ -6,17 +6,19 @@ DEPENDS = "ncurses libnl pciutils"
 LICENSE = "GPLv2"
 LIC_FILES_CHKSUM = "file://COPYING;md5=12f884d2ae1ff87c09e5b7ccc2c4ca7e"
 
-SRC_URI = 
"http://01.org/sites/default/files/downloads/powertop/powertop-${PV}.tar.gz \
-   file://0001-include-rquired-headers-for-typedefs.patch \
+SRC_URI = 
"http://01.org/sites/default/files/downloads/powertop/powertop-v${PV}.tar.gz \
 "
 
-SRC_URI[md5sum] = "c55fedb69203e480801b18bd7b886241"
-SRC_URI[sha256sum] = 
"a87b563f73106babfa3e74dcf92f252938c061e309ace20a361358bbfa579c5a"
+SRC_URI[md5sum] = "583518c5c4434c6e9b9c58c3920950b6"
+SRC_URI[sha256sum] = 
"aa7fb7d8e9a00f05e7d8a7a2866d85929741e0d03a5bf40cab22d2021c959250"
 
 UPSTREAM_CHECK_URI = "https://01.org/powertop/downloads;
+UPSTREAM_CHECK_REGEX = "powertop-[v]?(?P\d+(\.\d+)+)\.tar"
 
 inherit autotools gettext pkgconfig
 
+S = "${WORKDIR}/${PN}-v${PV}"
+
 # we do not want libncursesw if we can
 do_configure_prepend() {
 # configure.ac checks for delwin() in "ncursesw ncurses" so let's drop 
first one
-- 
2.9.4

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core