Re: [Buildroot] [PATCH 1/1] package/daemonize: new package
From: Joachim Wiberg <hidden>
Date: 2021-12-31 09:45:30
Hi, sorry for the late review: On 1/24/21 11:22 AM, Christian Stewart wrote:
Daemonize is a command line utility to run a program as a Unix daemon. Signed-off-by: Christian Stewart <redacted> ---
Missing entry in DEVELOPERS.
quoted hunk ↗ jump to hunk
package/Config.in | 1 + ...de-setpgrp-to-enable-cross-compiling.patch | 55 +++++++++++++++++++ package/daemonize/Config.in | 5 ++ package/daemonize/daemonize.hash | 1 + package/daemonize/daemonize.mk | 12 ++++ 5 files changed, 74 insertions(+) create mode 100644 package/daemonize/0001-configure-override-setpgrp-to-enable-cross-compiling.patch create mode 100644 package/daemonize/Config.in create mode 100644 package/daemonize/daemonize.hash create mode 100644 package/daemonize/daemonize.mkdiff --git a/package/Config.in b/package/Config.in index f42cc01032..b335aa4dd8 100644 --- a/package/Config.in +++ b/package/Config.in@@ -2413,6 +2413,7 @@ menu "System tools" source "package/coreutils/Config.in" source "package/cpuload/Config.in" source "package/daemon/Config.in" + source "package/daemonize/Config.in"
How does this program differ from the already existing daemon(1) we have in Buildroot? Are there programs that depend on it perhaps, or does it offer more functionality than daemon(1)?
quoted hunk ↗ jump to hunk
source "package/dc3dd/Config.in" source "package/dcron/Config.in" source "package/ddrescue/Config.in"diff --git a/package/daemonize/0001-configure-override-setpgrp-to-enable-cross-compiling.patch b/package/daemonize/0001-configure-override-setpgrp-to-enable-cross-compiling.patch new file mode 100644 index 0000000000..fd544f0441 --- /dev/null +++ b/package/daemonize/0001-configure-override-setpgrp-to-enable-cross-compiling.patch@@ -0,0 +1,55 @@ +From db172f4d7028c648f66f3c1db6202e6a5d62636d Mon Sep 17 00:00:00 2001 +From: Christian Stewart <christian@paral.in> +Date: Sun, 24 Jan 2021 02:16:36 -0800 +Subject: [PATCH] configure: override setpgrp to enable cross-compiling + +Signed-off-by: Christian Stewart <christian@paral.in> +--- + configure | 30 +----------------------------- + 1 file changed, 1 insertion(+), 29 deletions(-) + +diff --git a/configure b/configure +index ab7c0d4..209e165 100755 +--- a/configure ++++ b/configure +@@ -4147,36 +4147,8 @@ fi + + { $as_echo "$as_me:${as_lineno-$LINENO}: checking whether setpgrp takes no argument" >&5 + $as_echo_n "checking whether setpgrp takes no argument... " >&6; } +-if ${ac_cv_func_setpgrp_void+:} false; then : +- $as_echo_n "(cached) " >&6 +-else +- if test "$cross_compiling" = yes; then : +- as_fn_error $? "cannot check setpgrp when cross compiling" "$LINENO" 5 +-else +- cat confdefs.h - <<_ACEOF >conftest.$ac_ext +-/* end confdefs.h. */ +-$ac_includes_default +-int +-main () +-{ +-/* If this system has a BSD-style setpgrp which takes arguments, +- setpgrp(1, 1) will fail with ESRCH and return -1, in that case +- exit successfully. */ +- return setpgrp (1,1) != -1; +- ; +- return 0; +-} +-_ACEOF +-if ac_fn_c_try_run "$LINENO"; then : +- ac_cv_func_setpgrp_void=no +-else +- ac_cv_func_setpgrp_void=yes +-fi +-rm -f core *.core core.conftest.* gmon.out bb.out conftest$ac_exeext \ +- conftest.$ac_objext conftest.beam conftest.$ac_ext +-fi ++ac_cv_func_setpgrp_void=yes + +-fi + { $as_echo "$as_me:${as_lineno-$LINENO}: result: $ac_cv_func_setpgrp_void" >&5 + $as_echo "$ac_cv_func_setpgrp_void" >&6; } + if test $ac_cv_func_setpgrp_void = yes; then +-- +2.30.0
We don't usually allow for patches in the tree that are not submitted upstream, i.e. only backported patches. However, issues like these are usually better solved by instead overriding the autoconf cache value (ac_cv_foo) from the .mk file. There are lots of examples of this in the tree, e.g. package/mtools/mtools.mk
quoted hunk ↗ jump to hunk
diff --git a/package/daemonize/Config.in b/package/daemonize/Config.in new file mode 100644 index 0000000000..f7f3288d9a --- /dev/null +++ b/package/daemonize/Config.in@@ -0,0 +1,5 @@ +menuconfig BR2_PACKAGE_DAEMONIZE + bool "daemonize" + help + Command line utility to run a daemon. +
Here would be a good place to mention any added value in this package, as compared to BR2_PACKAGE_DAEMON.
quoted hunk ↗ jump to hunk
diff --git a/package/daemonize/daemonize.hash b/package/daemonize/daemonize.hash new file mode 100644 index 0000000000..8bf58e96f4 --- /dev/null +++ b/package/daemonize/daemonize.hash@@ -0,0 +1 @@ +sha256 20c4fc9925371d1ddf1b57947f8fb93e2036eb9ccc3b43a1e3678ea8471c4c60 daemonize-1.7.8.tar.gzdiff --git a/package/daemonize/daemonize.mk b/package/daemonize/daemonize.mk new file mode 100644 index 0000000000..9a86ae95d5 --- /dev/null +++ b/package/daemonize/daemonize.mk@@ -0,0 +1,12 @@ +################################################################################ +# +# daemonize +# +################################################################################ + +DAEMONIZE_VERSION = 1.7.8 +DAEMONIZE_SITE = $(call github,bmc,daemonize,release-$(DAEMONIZE_VERSION)) +DAEMONIZE_LICENSE = BSD-3-Clause +DAEMONIZE_LICENSE_FILES = LICENSE.md + +$(eval $(autotools-package))
Other than my comments above, the packaging looks fine to me. Reviewed-by: Joachim Wiberg <redacted> Best regards /Joachim _______________________________________________ buildroot mailing list buildroot@buildroot.org https://lists.buildroot.org/mailman/listinfo/buildroot