public inbox for gentoo-commits@lists.gentoo.org
 help / color / mirror / Atom feed
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
-


             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