Linux-x86-power-tools not building on musl

[aux-j]

Hi there,

Linux-x86-power-tools was not building on musl due to a missing include in turbostat.c - Here's a patch I've just whipped up to get it working:

diff --git a/tools/power/x86/turbostat/turbostat.c b/tools/power/x86/turbostat/turbostat.c
index 5b1b807..055a514 100644
--- a/tools/power/x86/turbostat/turbostat.c
+++ b/tools/power/x86/turbostat/turbostat.c
@@ -30,6 +30,7 @@
#include <sys/stat.h>
#include <sys/resource.h>
#include <fcntl.h>
+#include <limits.h>
#include <signal.h>
#include <sys/time.h>
#include <stdlib.h>

[leezu]

You should propose this patch upstream in the linux kernel https://github.com/torvalds/linux/blob/master/tools/power/x86/turbostat/turbostat.c

[dywisor]

Sorry for catching this one late.

As @leezu suggested, please propose this patch upstream as it could also affect other packages relying on the same code base such as sys-apps/linux-misc-apps.

[aux-j]

Hi @dywisor,

Sorry for the late reply, I'll submit a PR. Thanks!

[dywisor]

@gitlab-j Can you provide me with a link to the corresponding bug report for tracking purposes?

I'll include a patch in the next sys-power/linux-x86-power-tools ebuild, but eventually I'd like to see this fixed upstream.