ntp: fix build for no-MMU with libcap
authorBaruch Siach <baruch@tkos.co.il>
Sat, 13 Oct 2018 18:51:00 +0000 (21:51 +0300)
committerPeter Korsgaard <peter@korsgaard.com>
Sat, 13 Oct 2018 21:15:44 +0000 (23:15 +0200)
Commit 87d759ced5 (ntp: fix build for no-MMU) added a patch to make MMU
dependent code hidden behind HAVE_WORKING_FORK. It turns out that the
patch covers too much code. When libcap is enabled we pass
--enable-linuxcaps, which in turn enables HAVE_DROPROOT. This adds calls
to code that is covered by HAVE_WORKING_FORK.

Update the no-MMU fix so that HAVE_WORKING_FORK only covers the no-MMU
incompatible routine.

Fixes:
http://autobuild.buildroot.net/results/c5c/c5cf28bb969fec7c07864cdd094dedfa4d5439d2/

Cc: Artem Panfilov <apanfilov@spectracom.com>
Signed-off-by: Baruch Siach <baruch@tkos.co.il>
Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
package/ntp/0003-fix-nommu.patch

index 53bf4dfcf38bb8129ce47fa41d2cd9cdea6032b9..0959fa0ba522c9e7f6869cfaa69151644123841f 100644 (file)
@@ -10,13 +10,20 @@ Upstream status: http://bugs.ntp.org/show_bug.cgi?id=3538
 
 diff -Nuar ntp-4.2.8p12.orig/ntpd/ntpd.c ntp-4.2.8p12/ntpd/ntpd.c
 --- ntp-4.2.8p12.orig/ntpd/ntpd.c      2018-08-14 14:51:30.000000000 +0300
-+++ ntp-4.2.8p12/ntpd/ntpd.c   2018-10-12 08:19:01.291781773 +0300
-@@ -529,7 +529,7 @@
++++ ntp-4.2.8p12/ntpd/ntpd.c   2018-10-13 21:25:25.858261249 +0300
+@@ -534,6 +534,7 @@
+  * Detach from terminal (much like daemon())
+  * Nothe that this function calls exit()
+  */
++# ifdef HAVE_WORKING_FORK
+ static void
+ detach_from_terminal(
+       int pipe_fds[2],
+@@ -617,6 +618,7 @@
+       return;
  }
- #endif        /* !SIM */
++# endif /* HAVE_WORKING_FORK */
  
--#if !defined(SIM) && !defined(SYS_WINNT)
-+#if !defined(SIM) && !defined(SYS_WINNT) && defined(HAVE_WORKING_FORK)
+ #ifdef HAVE_DROPROOT
  /*
-  * Detach from terminal (much like daemon())
-  * Nothe that this function calls exit()