From: "Matt Turner" <mattst88@gentoo.org>
To: gentoo-commits@lists.gentoo.org
Subject: [gentoo-commits] repo/gentoo:master commit in: sys-power/powertop/files/
Date: Mon, 29 Jul 2019 15:49:05 +0000 (UTC) [thread overview]
Message-ID: <1564415334.fc6854ed667f0b5298ebb508eca9a0adf6052f39.mattst88@gentoo> (raw)
commit: fc6854ed667f0b5298ebb508eca9a0adf6052f39
Author: Matt Turner <mattst88 <AT> gentoo <DOT> org>
AuthorDate: Mon Jul 29 15:45:15 2019 +0000
Commit: Matt Turner <mattst88 <AT> gentoo <DOT> org>
CommitDate: Mon Jul 29 15:48:54 2019 +0000
URL: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=fc6854ed
sys-power/powertop: Restore mistakenly removed patch
When cleaning up, I deleted the wrong patch and left the unused one in
place.
Closes: https://bugs.gentoo.org/690992
Signed-off-by: Matt Turner <mattst88 <AT> gentoo.org>
.../files/powertop-2.8-ncurses_tinfo.patch | 41 ++++++++++++++++++++
sys-power/powertop/files/powertop-2.9-libc++.patch | 44 ----------------------
2 files changed, 41 insertions(+), 44 deletions(-)
diff --git a/sys-power/powertop/files/powertop-2.8-ncurses_tinfo.patch b/sys-power/powertop/files/powertop-2.8-ncurses_tinfo.patch
new file mode 100644
index 00000000000..97769fb6728
--- /dev/null
+++ b/sys-power/powertop/files/powertop-2.8-ncurses_tinfo.patch
@@ -0,0 +1,41 @@
+From e1295099f8b42670718ba875cb6749a90042293f Mon Sep 17 00:00:00 2001
+From: Zentaro Kavanagh <zentaro@chromium.org>
+Date: Thu, 14 Jun 2018 13:13:37 -0700
+Subject: [PATCH] Fix configure to support ncurses w/ tinfo
+
+- The existing code checked for both ncursesw and ncurses and if
+ both were not found, NCURSES_LIBS was not set correctly.
+- Removed redundant concatenation to $LIBS since the makefile.am
+ already maps NCURSES_LIBS into LIBS.
+- Patch sent upstream to powertop mailing list [1]
+
+[1] - https://lists.01.org/pipermail/powertop/2018-June/002021.html
+---
+ configure.ac | 11 +++++++----
+ 1 file changed, 7 insertions(+), 4 deletions(-)
+
+diff --git a/configure.ac b/configure.ac
+index d6a15e1..c6ee50a 100644
+--- a/configure.ac
++++ b/configure.ac
+@@ -110,10 +110,13 @@ AC_CHECK_FUNCS([ \
+
+ AC_SEARCH_LIBS([clock_gettime], [rt])
+
+-PKG_CHECK_MODULES([NCURSES], [ncursesw ncurses], [LIBS="$LIBS $ncurses_LIBS"], [
+- AC_SEARCH_LIBS([delwin], [ncursesw ncurses], [], [
+- AC_MSG_ERROR([ncurses is required but was not found])
+- ], [])
++PKG_CHECK_MODULES([ncursesw], [ncursesw],
++ [NCURSES_CFLAGS="$ncursesw_CFLAGS"; NCURSES_LIBS="$ncursesw_LIBS"], [
++ PKG_CHECK_MODULES([NCURSES], [ncurses], [], [
++ AC_SEARCH_LIBS([delwin], [ncursesw ncurses], [], [
++ AC_MSG_ERROR([ncurses is required but was not found])
++ ])
++ ])
+ ])
+
+ has_libpci=0
+--
+2.18.0.rc1.242.g61856ae69a-goog
+
diff --git a/sys-power/powertop/files/powertop-2.9-libc++.patch b/sys-power/powertop/files/powertop-2.9-libc++.patch
deleted file mode 100644
index d378280f3c6..00000000000
--- a/sys-power/powertop/files/powertop-2.9-libc++.patch
+++ /dev/null
@@ -1,44 +0,0 @@
-https://github.com/fenrus75/powertop/commit/b9c431aad6cc5383c9571007469eee8e64ec85a6
-
-From b9c431aad6cc5383c9571007469eee8e64ec85a6 Mon Sep 17 00:00:00 2001
-From: Manoj Gupta <manojgupta@chromium.org>
-Date: Sat, 4 Nov 2017 13:44:04 -0700
-Subject: [PATCH] Fix powertop build with libc++.
-
-<ctime> header is not automatically included with libc++.
-Add it explicitly to make powertop build with libc++.
-
-This fixes the following errors:
-devices/gpu_rapl_device.cpp:35:14: error: use of undeclared identifier
-'time'; did you mean 'tie'?
-last_time = time(NULL);
- ^~~~
-devices/gpu_rapl_device.cpp:45:14: error:use of undeclared identifier
-'time'; did you mean 'tie'?
-last_time = time(NULL);
- ^~~~
-
-parameters/learn.cpp:161:10: error: use of undeclared identifier
-'time'; did you mean 'tie'?
-start = time(NULL);
- ^~~~
----
- src/lib.h | 2 +-
- 1 file changed, 1 insertion(+), 1 deletion(-)
-
-diff --git a/src/lib.h b/src/lib.h
-index b64bb0f5433f..6d85eb6e4b40 100644
---- a/src/lib.h
-+++ b/src/lib.h
-@@ -54,7 +54,7 @@ extern const char *kernel_function(uint64_t address);
-
-
-
--
-+#include <ctime>
- #include <string>
- using namespace std;
-
---
-2.15.1
-
next reply other threads:[~2019-07-29 15:49 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-07-29 15:49 Matt Turner [this message]
-- strict thread matches above, loose matches on Subject: below --
2017-12-11 10:36 [gentoo-commits] repo/gentoo:master commit in: sys-power/powertop/files/ Michael Weber
2017-02-14 21:39 David Seifert
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1564415334.fc6854ed667f0b5298ebb508eca9a0adf6052f39.mattst88@gentoo \
--to=mattst88@gentoo.org \
--cc=gentoo-commits@lists.gentoo.org \
--cc=gentoo-dev@lists.gentoo.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox