[PATCH] buildsys: make tailf linux-only (needs inotify)

[PATCH] buildsys: make tailf linux-only (needs inotify)

[Andreas Henriksson]

While tailf builds fine on other architectures, it relies
on inotify to work properly. So for example on
Debian architectures using Hurd or kFreeBSD tailf
would work like a regular tail (with a non-functional
-f option), which makes it quite redundant.
This was caught by the util-linux testsuite.

Signed-off-by: Andreas Henriksson <andreas@fatal.se>
---
 configure.ac | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/configure.ac b/configure.ac
index aae2456..eade79c 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1272,7 +1272,8 @@ AM_CONDITIONAL([BUILD_HEXDUMP], [test "x$build_hexdump" = xyes])
 UL_BUILD_INIT([rev], [yes])
 AM_CONDITIONAL([BUILD_REV], [test "x$build_rev" = xyes])
 
-UL_BUILD_INIT([tailf], [yes])
+UL_BUILD_INIT([tailf], [check])
+UL_REQUIRES_LINUX([tailf])
 AM_CONDITIONAL([BUILD_TAILF], [test "x$build_tailf" = xyes])
 
 
-- 
2.0.1

Re: [PATCH] buildsys: make tailf linux-only (needs inotify)

[Andreas Henriksson]

Hello all!

On Sat, Jul 26, 2014 at 07:01:11PM +0200, Andreas Henriksson wrote:
> While tailf builds fine on other architectures, it relies
> on inotify to work properly. So for example on
[...]

Please ignore this! I was to quick to judge. The program
has non-inotify fallback code which does work... this is
likely just a problem in the testsuite.
I'll follow up shortly.

Regards,
Andreas Henriksson