[Buildroot] [PATCH 1/1] package/monkey: bump to latest git commit

Arnout Vandecappelle arnout at mind.be
Wed May 5 20:00:16 UTC 2021



On 26/04/2021 23:53, Fabrice Fontaine wrote:
> This will fix a build failure with gcc 10
> 
> - Update indentation in hash file (two spaces)
> - Drop INSTALL_SYSCONFDIR, INSTALL_WEBROOTDIR and WITH_SYSTEM_MALLOC
>   (not available since
>   https://github.com/monkey/monkey/commit/df145932e33fca0d4a1dcd9d7675f996c8e6a73b)
> - Set WITHOUT_HEADERS to ON because headers are not needed and to avoid
>   the following build failure:
> 
> CMake Error at include/cmake_install.cmake:46 (file):
>   file INSTALL cannot find
>   "/home/fabrice/buildroot/output/build/monkey-f54856ce250c4e25735434dc75717a4b7fbfc45b/include/mk_core.h":
>   No such file or directory.
> Call Stack (most recent call first):
>   cmake_install.cmake:69 (include)

 Has this problem been reported upstream?

> 
> Upstream is aware than the lack of release is an issue but no comments
> since 2018: https://github.com/monkey/monkey/issues/276
> 
> Fixes:
>  - http://autobuild.buildroot.org/results/0b723937ca048228082d040100f6e6324ac8300b
> 
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice at gmail.com>
> ---
>  package/monkey/monkey.hash |  7 +++----
>  package/monkey/monkey.mk   | 18 ++++++++----------
>  2 files changed, 11 insertions(+), 14 deletions(-)
> 
> diff --git a/package/monkey/monkey.hash b/package/monkey/monkey.hash
> index 83fc0def60..27ff806442 100644
> --- a/package/monkey/monkey.hash
> +++ b/package/monkey/monkey.hash
> @@ -1,4 +1,3 @@
> -# md5 from http://monkey-project.com/releases/1.6/monkey-1.6.9.tar.gz.md5, sha256 locally computed:
> -md5    c401734985a936ee07dbff5eca7d176a 				monkey-1.6.9.tar.gz
> -sha256 f1122e89cda627123286542b0a18fcaa131cbe9d4f5dd897d9455157289148fb	monkey-1.6.9.tar.gz
> -sha256 0d542e0c8804e39aa7f37eb00da5a762149dc682d7829451287e11b938e94594	LICENSE
> +# Locally computed:
> +sha256  c988d9f4f577a789aa244e5a0f99ebfd46ebddc328656f319acf64edc9fd5199  monkey-f54856ce250c4e25735434dc75717a4b7fbfc45b.tar.gz
> +sha256  0d542e0c8804e39aa7f37eb00da5a762149dc682d7829451287e11b938e94594  LICENSE
> diff --git a/package/monkey/monkey.mk b/package/monkey/monkey.mk
> index 5264713120..2cb1a5f298 100644
> --- a/package/monkey/monkey.mk
> +++ b/package/monkey/monkey.mk
> @@ -4,31 +4,29 @@
>  #
>  ################################################################################
>  
> -MONKEY_VERSION_MAJOR = 1.6
> -MONKEY_VERSION = $(MONKEY_VERSION_MAJOR).9
> -MONKEY_SITE = http://monkey-project.com/releases/$(MONKEY_VERSION_MAJOR)
> +MONKEY_VERSION = f54856ce250c4e25735434dc75717a4b7fbfc45b
> +MONKEY_SITE = $(call github,monkey,monkey,$(MONKEY_VERSION))
>  MONKEY_LICENSE = Apache-2.0
>  MONKEY_LICENSE_FILES = LICENSE
>  
>  MONKEY_CONF_OPTS = \
> -	-DINSTALL_SYSCONFDIR=/etc/monkey \
> -	-DINSTALL_WEBROOTDIR=/var/www \
> -	-DWITH_SYSTEM_MALLOC=1
> +	-DMK_PATH_WWW=/var/www \
> +	-DWITHOUT_HEADERS=ON
>  
>  ifeq ($(BR2_TOOLCHAIN_USES_UCLIBC),y)
> -MONKEY_CONF_OPTS += -DWITH_UCLIB=1 -DWITH_BACKTRACE=0
> +MONKEY_CONF_OPTS += -DMK_UCLIB=ON -DMK_BACKTRACE=OFF

 MK_BACKTRACE seems to be unused. It's a user option that defaults to ON and is
set to OFF if execinfo.h is not found, but it has no effect by itself. Instead,
execinfo.h is used if available, independent of this option.

 MK_UCLIB also doesn't seem to have much effect - it defines MK_HAVE_UCLIB but
that macro is not used anywhere.

>  endif
>  
>  ifeq ($(BR2_TOOLCHAIN_USES_MUSL),y)
> -MONKEY_CONF_OPTS += -DWITH_MUSL=1 -DWITH_BACKTRACE=0
> +MONKEY_CONF_OPTS += -DMK_MUSL=ON -DMK_BACKTRACE=OFF

 Same same.

>  endif
>  
>  ifeq ($(BR2_ENABLE_DEBUG),y)
> -MONKEY_CONF_OPTS += -DWITH_DEBUG=1
> +MONKEY_CONF_OPTS += -DMK_DEBUG=ON

 The only thing this does is set CMAKE_BUILD_TYPE to Debug - which we do anyway.
So I'd drop this.

>  endif
>  
>  ifeq ($(BR2_PACKAGE_MONKEY_SSL),y)
> -MONKEY_CONF_OPTS += -DWITH_PLUGINS=tls -DWITH_MBEDTLS_SHARED=1
> +MONKEY_CONF_OPTS += -DMK_PLUGIN_TLS=ON -DMK_MBEDTLS_SHARED=ON
>  MONKEY_DEPENDENCIES += mbedtls
>  endif

There are a bunch of options in monkey:

MK_ACCEPT:BOOL=OFF
MK_ACCEPT4:BOOL=ON
MK_BACKTRACE:BOOL=ON
MK_DEBUG:BOOL=OFF
MK_FUZZ_MODE:BOOL=OFF
MK_HTTP2:BOOL=OFF
MK_LINUX_KQUEUE:BOOL=OFF
MK_LINUX_TRACE:BOOL=OFF
MK_MBEDTLS_SHARED:BOOL=OFF
MK_MUSL:BOOL=OFF
MK_PLUGIN_AUTH:BOOL=OFF
MK_PLUGIN_CGI:BOOL=OFF
MK_PLUGIN_CHEETAH:BOOL=OFF
MK_PLUGIN_DIRLISTING:BOOL=ON
MK_PLUGIN_FASTCGI:BOOL=OFF
MK_PLUGIN_LIANA:BOOL=ON
MK_PLUGIN_LOGGER:BOOL=OFF
MK_PLUGIN_MANDRIL:BOOL=ON
MK_PLUGIN_TLS:BOOL=OFF
MK_PTHREAD_TLS:BOOL=OFF
MK_STATIC_LIB_MODE:BOOL=OFF
MK_TRACE:BOOL=OFF
MK_UCLIB:BOOL=OFF
MK_VALGRIND:BOOL=OFF
MK_WITHOUT_BIN:BOOL=OFF
MK_WITHOUT_CONF:BOOL=OFF

For most of those, I guess it's OK to keep them to default.  MK_TRACE however
enables lttng support, so that's possibly useful. Would be nice to investigate.


 I've anyway applied to master. The cleanups I mention can be applied later.

 Regards,
 Arnout



More information about the buildroot mailing list