* [Buildroot] [PATCH 1/1] package/daemonize: new package @ 2021-01-24 10:22 Christian Stewart 2021-12-31 9:45 ` Joachim Wiberg 0 siblings, 1 reply; 3+ messages in thread From: Christian Stewart @ 2021-01-24 10:22 UTC (permalink / raw) To: buildroot Daemonize is a command line utility to run a program as a Unix daemon. Signed-off-by: Christian Stewart <christian@paral.in> --- 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.mk diff --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" 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 + 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. + 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.gz diff --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)) -- 2.30.0 ^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [Buildroot] [PATCH 1/1] package/daemonize: new package 2021-01-24 10:22 [Buildroot] [PATCH 1/1] package/daemonize: new package Christian Stewart @ 2021-12-31 9:45 ` Joachim Wiberg 2022-01-09 19:06 ` Romain Naour 0 siblings, 1 reply; 3+ messages in thread From: Joachim Wiberg @ 2021-12-31 9:45 UTC (permalink / raw) To: Christian Stewart, buildroot 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 <christian@paral.in> > --- Missing entry in DEVELOPERS. > 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.mk > > diff --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)? > 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 > 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. > 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.gz > diff --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 <troglobit@gmail.com> Best regards /Joachim _______________________________________________ buildroot mailing list buildroot@buildroot.org https://lists.buildroot.org/mailman/listinfo/buildroot ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [Buildroot] [PATCH 1/1] package/daemonize: new package 2021-12-31 9:45 ` Joachim Wiberg @ 2022-01-09 19:06 ` Romain Naour 0 siblings, 0 replies; 3+ messages in thread From: Romain Naour @ 2022-01-09 19:06 UTC (permalink / raw) To: Joachim Wiberg, Christian Stewart, buildroot Hello Christian, Le 31/12/2021 à 10:45, Joachim Wiberg a écrit : > 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 <christian@paral.in> >> --- > > Missing entry in DEVELOPERS. > >> 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.mk >> >> diff --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)? > >> 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 There is a configure.in, if possible patch this file instead and use DAEMONIZE_AUTORECONF = YES. > >> 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" is there any toolchain dependency ? glibc, uclibc-ng, musl ? Have you check with test-pkg ? >> + 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. Alse add the link to the project: http://software.clapper.org/daemonize (Use check-package) > >> 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.gz license file hash is missing. >> diff --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 Maybe we should add the getopt.c license: https://github.com/bmc/daemonize/blob/master/LICENSE.md#license-for-getoptc >> +DAEMONIZE_LICENSE_FILES = LICENSE.md >> + >> +$(eval $(autotools-package)) > > Other than my comments above, the packaging looks fine to me. > > Reviewed-by: Joachim Wiberg <troglobit@gmail.com> I have marked this patch as "Changes Requested". Best regards, Romain > > Best regards > /Joachim > > > _______________________________________________ > buildroot mailing list > buildroot@buildroot.org > https://lists.buildroot.org/mailman/listinfo/buildroot > _______________________________________________ buildroot mailing list buildroot@buildroot.org https://lists.buildroot.org/mailman/listinfo/buildroot ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2022-01-09 19:06 UTC | newest] Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-01-24 10:22 [Buildroot] [PATCH 1/1] package/daemonize: new package Christian Stewart 2021-12-31 9:45 ` Joachim Wiberg 2022-01-09 19:06 ` Romain Naour
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.