* [Buildroot] [PATCH 2/3] package/htop: use official tarball
2022-04-03 11:04 [Buildroot] [PATCH 1/3] package/htop: fix sensors option Fabrice Fontaine
@ 2022-04-03 11:04 ` Fabrice Fontaine
2022-04-04 17:51 ` Peter Korsgaard
2022-04-03 11:04 ` [Buildroot] [PATCH 3/3] package/htop: fix hwloc static build Fabrice Fontaine
` (2 subsequent siblings)
3 siblings, 1 reply; 7+ messages in thread
From: Fabrice Fontaine @ 2022-04-03 11:04 UTC (permalink / raw)
To: buildroot; +Cc: Fabrice Fontaine
Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
package/htop/htop.hash | 4 +++-
package/htop/htop.mk | 3 ++-
2 files changed, 5 insertions(+), 2 deletions(-)
diff --git a/package/htop/htop.hash b/package/htop/htop.hash
index dffe5ca1ad..d7c39be4d0 100644
--- a/package/htop/htop.hash
+++ b/package/htop/htop.hash
@@ -1,3 +1,5 @@
+# From https://github.com/htop-dev/htop/releases/download/3.1.2/htop-3.1.2.tar.xz.sha256
+sha256 884bce5b58cb113127860b9e368609019e92416a81550fdf0752052f3a64b388 htop-3.1.2.tar.xz
+
# Locally calculated
-sha256 fe9559637c8f21f5fd531a4c072048a404173806acbdad1359c6b82fd87aa001 htop-3.1.2.tar.gz
sha256 8177f97513213526df2cf6184d8ff986c675afb514d4e68a404010521b880643 COPYING
diff --git a/package/htop/htop.mk b/package/htop/htop.mk
index 210518da2f..1e7b171a17 100644
--- a/package/htop/htop.mk
+++ b/package/htop/htop.mk
@@ -5,7 +5,8 @@
################################################################################
HTOP_VERSION = 3.1.2
-HTOP_SITE = $(call github,htop-dev,htop,$(HTOP_VERSION))
+HTOP_SOURCE = htop-$(HTOP_VERSION).tar.xz
+HTOP_SITE = https://github.com/htop-dev/htop/releases/download/$(HTOP_VERSION)
HTOP_DEPENDENCIES = ncurses
HTOP_AUTORECONF = YES
# Prevent htop build system from searching the host paths
--
2.35.1
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 7+ messages in thread* [Buildroot] [PATCH 3/3] package/htop: fix hwloc static build
2022-04-03 11:04 [Buildroot] [PATCH 1/3] package/htop: fix sensors option Fabrice Fontaine
2022-04-03 11:04 ` [Buildroot] [PATCH 2/3] package/htop: use official tarball Fabrice Fontaine
@ 2022-04-03 11:04 ` Fabrice Fontaine
2022-04-04 17:49 ` Peter Korsgaard
2022-04-03 18:58 ` [Buildroot] [PATCH 1/3] package/htop: fix sensors option Yann E. MORIN
2022-04-04 17:48 ` Peter Korsgaard
3 siblings, 1 reply; 7+ messages in thread
From: Fabrice Fontaine @ 2022-04-03 11:04 UTC (permalink / raw)
To: buildroot; +Cc: Fabrice Fontaine
Fix the following static build failure with hwloc raised since commit
c83bf9720632bcd4d438094a9fc30feff69fc555:
checking for hwloc_get_proc_cpubind in -lhwloc... no
configure: error: can not find required library libhwloc
Fixes:
- http://autobuild.buildroot.org/results/5d815ec08c580005a863df6ac9ac29deff7d4128
Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
...igure.ac-fix-static-build-with-hwloc.patch | 54 +++++++++++++++++++
1 file changed, 54 insertions(+)
create mode 100644 package/htop/0001-configure.ac-fix-static-build-with-hwloc.patch
diff --git a/package/htop/0001-configure.ac-fix-static-build-with-hwloc.patch b/package/htop/0001-configure.ac-fix-static-build-with-hwloc.patch
new file mode 100644
index 0000000000..0ac79766c5
--- /dev/null
+++ b/package/htop/0001-configure.ac-fix-static-build-with-hwloc.patch
@@ -0,0 +1,54 @@
+From 4ccad4604586c921b4ad831b254496c2cdece9fc Mon Sep 17 00:00:00 2001
+From: Fabrice Fontaine <fontaine.fabrice@gmail.com>
+Date: Sat, 2 Apr 2022 17:27:01 +0200
+Subject: [PATCH] configure.ac: fix static build with hwloc
+
+Retrieve hwloc dependencies through pkg-config to avoid the following
+static build failure:
+
+checking for hwloc_get_proc_cpubind in -lhwloc... no
+configure: error: can not find required library libhwloc
+
+This build failure is raised because without pkg-config, hwloc
+dependencies such as libxml2 are not retrieved:
+
+configure:8999: checking for hwloc_get_proc_cpubind in -lhwloc
+configure:9022: /home/autobuild/autobuild/instance-0/output-1/host/bin/powerpc-buildroot-linux-uclibc-gcc -o conftest -D_GNU_SOURCE -I/home/autobuild/autobuild/instance-0/output-1/host/powerpc-buildroot-linux-uclibc/sysroot/usr/bin/../../usr/include -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -D_FILE_OFFSET_BITS=64 -Og -g0 -static -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -D_FILE_OFFSET_BITS=64 -static conftest.c -lhwloc -llzma -L/home/autobuild/autobuild/instance-0/output-1/host/powerpc-buildroot-linux-uclibc/sysroot/usr/bin/../../usr/lib -lncurses -lm >&5
+/home/autobuild/autobuild/instance-0/output-1/host/lib/gcc/powerpc-buildroot-linux-uclibc/10.3.0/../../../../powerpc-buildroot-linux-uclibc/bin/ld: /home/autobuild/autobuild/instance-0/output-1/host/powerpc-buildroot-linux-uclibc/sysroot/usr/bin/../../usr/lib/libhwloc.a(topology-xml-libxml.o): in function `hwloc_libxml_free_buffer':
+topology-xml-libxml.c:(.text+0x6a): undefined reference to `xmlFree'
+
+Fixes:
+ - http://autobuild.buildroot.org/results/5d815ec08c580005a863df6ac9ac29deff7d4128
+
+Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
+[Retrieved from:
+https://github.com/htop-dev/htop/commit/4ccad4604586c921b4ad831b254496c2cdece9fc]
+---
+ configure.ac | 14 ++++++++++++--
+ 1 file changed, 12 insertions(+), 2 deletions(-)
+
+diff --git a/configure.ac b/configure.ac
+index 4ecac1ecf..6bb7eb69f 100644
+--- a/configure.ac
++++ b/configure.ac
+@@ -456,8 +456,18 @@ case "$enable_hwloc" in
+ no)
+ ;;
+ yes)
+- AC_CHECK_LIB([hwloc], [hwloc_get_proc_cpubind], [], [AC_MSG_ERROR([can not find required library libhwloc])])
+- AC_CHECK_HEADERS([hwloc.h], [], [AC_MSG_ERROR([can not find require header file hwloc.h])])
++ m4_ifdef([PKG_PROG_PKG_CONFIG], [
++ PKG_PROG_PKG_CONFIG()
++ PKG_CHECK_MODULES(HWLOC, hwloc, [
++ CFLAGS="$CFLAGS $HWLOC_CFLAGS" LIBS="$LIBS $HWLOC_LIBS"
++ ], [
++ AC_CHECK_LIB([hwloc], [hwloc_get_proc_cpubind], [], [AC_MSG_ERROR([can not find required library libhwloc])])
++ AC_CHECK_HEADERS([hwloc.h], [], [AC_MSG_ERROR([can not find require header file hwloc.h])])
++ ])
++ ], [
++ AC_CHECK_LIB([hwloc], [hwloc_get_proc_cpubind], [], [AC_MSG_ERROR([can not find required library libhwloc])])
++ AC_CHECK_HEADERS([hwloc.h], [], [AC_MSG_ERROR([can not find require header file hwloc.h])])
++ ])
+ ;;
+ *)
+ AC_MSG_ERROR([bad value '$enable_hwloc' for --enable-hwloc])
--
2.35.1
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [Buildroot] [PATCH 1/3] package/htop: fix sensors option
2022-04-03 11:04 [Buildroot] [PATCH 1/3] package/htop: fix sensors option Fabrice Fontaine
2022-04-03 11:04 ` [Buildroot] [PATCH 2/3] package/htop: use official tarball Fabrice Fontaine
2022-04-03 11:04 ` [Buildroot] [PATCH 3/3] package/htop: fix hwloc static build Fabrice Fontaine
@ 2022-04-03 18:58 ` Yann E. MORIN
2022-04-04 17:48 ` Peter Korsgaard
3 siblings, 0 replies; 7+ messages in thread
From: Yann E. MORIN @ 2022-04-03 18:58 UTC (permalink / raw)
To: Fabrice Fontaine; +Cc: buildroot
Fabrice, All,
On 2022-04-03 13:04 +0200, Fabrice Fontaine spake thusly:
> Commit 811846df48745d58d96f6c0fbab0dec5f17d614b wrongly used
> --{with,without}-sensors instead of --{dis,en}able-sensors
>
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
Series of three patches applied to master, thanks.
Regards,
Yann E. MORIN.
> ---
> package/htop/htop.mk | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/package/htop/htop.mk b/package/htop/htop.mk
> index 7caa47ce13..210518da2f 100644
> --- a/package/htop/htop.mk
> +++ b/package/htop/htop.mk
> @@ -28,10 +28,10 @@ HTOP_CONF_OPTS += --disable-capabilities
> endif
>
> ifeq ($(BR2_PACKAGE_LM_SENSORS),y)
> -HTOP_CONF_OPTS += --with-sensors
> +HTOP_CONF_OPTS += --enable-sensors
> HTOP_DEPENDENCIES += lm-sensors
> else
> -HTOP_CONF_OPTS += --without-sensors
> +HTOP_CONF_OPTS += --disable-sensors
> endif
>
> ifeq ($(BR2_PACKAGE_NCURSES_WCHAR),y)
> --
> 2.35.1
>
> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [Buildroot] [PATCH 1/3] package/htop: fix sensors option
2022-04-03 11:04 [Buildroot] [PATCH 1/3] package/htop: fix sensors option Fabrice Fontaine
` (2 preceding siblings ...)
2022-04-03 18:58 ` [Buildroot] [PATCH 1/3] package/htop: fix sensors option Yann E. MORIN
@ 2022-04-04 17:48 ` Peter Korsgaard
3 siblings, 0 replies; 7+ messages in thread
From: Peter Korsgaard @ 2022-04-04 17:48 UTC (permalink / raw)
To: Fabrice Fontaine; +Cc: buildroot
>>>>> "Fabrice" == Fabrice Fontaine <fontaine.fabrice@gmail.com> writes:
> Commit 811846df48745d58d96f6c0fbab0dec5f17d614b wrongly used
> --{with,without}-sensors instead of --{dis,en}able-sensors
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
This is in fact because htop upstream has changed the option name in
3.1.x, so it was correct back in 811846.
Committed to 2022.02.x, thanks.
--
Bye, Peter Korsgaard
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 7+ messages in thread