All of lore.kernel.org
 help / color / mirror / Atom feed
From: Karel Zak <kzak@redhat.com>
To: Andreas Henriksson <andreas@fatal.se>
Cc: util-linux@vger.kernel.org
Subject: Re: [PATCH 0/4] Patches for kFreeBSD support from Debian
Date: Wed, 17 Sep 2014 11:00:10 +0200	[thread overview]
Message-ID: <20140917090010.GI7867@x2.net.home> (raw)
In-Reply-To: <1408896284-14467-1-git-send-email-andreas@fatal.se>

On Sun, Aug 24, 2014 at 06:04:40PM +0200, Andreas Henriksson wrote:
> This is the patch set that has been shipping since forever
> in Debian (minus the last one from me, which is new).

 Applied, thanks.

> Christoph Egger (1):
>   Fixing FTBFS on !linux (Debian util-linux 2.20.1-1.2)

 -lutil everywhere seems like overkill as login_tty() is probably 
 necessary for BSD only, so I have applied the patch below too.

    Karel


commit 8026fa9bc7525c4cef02c17099e2db79284078f8
Author: Karel Zak <kzak@redhat.com>
Date:   Wed Sep 17 10:56:46 2014 +0200

    build-sys: use -lutil for BSD only
    
    Signed-off-by: Karel Zak <kzak@redhat.com>

diff --git a/configure.ac b/configure.ac
index 5b558ec..387372f 100644
--- a/configure.ac
+++ b/configure.ac
@@ -136,9 +136,17 @@ PKG_PROG_PKG_CONFIG
 GTK_DOC_CHECK([1.10])
 AC_PATH_PROG([XSLTPROC], [xsltproc])
 
+
 linux_os=no
-AS_CASE([${host_os}], [*linux*], [linux_os=yes])
+bsd_os=no
+AS_CASE([${host_os}],
+  [*linux*],
+     [linux_os=yes],
+  [*bsd*],
+     [bsd_os=yes])
 AM_CONDITIONAL([LINUX], [test "x$linux_os" = xyes])
+AM_CONDITIONAL([BSD], [test "x$bsd_os" = xyes])
+
 
 dnl define ARCH_<NAME> conditionals
 UL_SET_ARCH([I86], [i?86-*])
diff --git a/term-utils/Makemodule.am b/term-utils/Makemodule.am
index 627aca0..e7ac707 100644
--- a/term-utils/Makemodule.am
+++ b/term-utils/Makemodule.am
@@ -23,7 +23,9 @@ sbin_PROGRAMS += agetty
 dist_man_MANS += term-utils/agetty.8
 agetty_SOURCES = term-utils/agetty.c
 agetty_LDADD = $(LDADD) libcommon.la
+if BSD
 agetty_LDADD += -lutil
+endif
 endif # BUILD_AGETTY
 
 

-- 
 Karel Zak  <kzak@redhat.com>
 http://karelzak.blogspot.com

  parent reply	other threads:[~2014-09-17  9:00 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-08-24 16:04 [PATCH 0/4] Patches for kFreeBSD support from Debian Andreas Henriksson
2014-08-24 16:04 ` [PATCH 1/4] Report correct disk size on GNU/kFreeBSD. Thanks Tuco Andreas Henriksson
2014-08-24 16:04 ` [PATCH 2/4] remaining kFreeBSD hackery for building Andreas Henriksson
2014-08-24 16:04 ` [PATCH 3/4] Fixing FTBFS on !linux (Debian util-linux 2.20.1-1.2) Andreas Henriksson
2014-08-24 16:04 ` [PATCH 4/4] kFreeBSD: add hacks in ipcrm to avoid FTBFS Andreas Henriksson
2014-09-17  9:00 ` Karel Zak [this message]
2014-09-17 14:43   ` [PATCH 0/4] Patches for kFreeBSD support from Debian Andreas Henriksson

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20140917090010.GI7867@x2.net.home \
    --to=kzak@redhat.com \
    --cc=andreas@fatal.se \
    --cc=util-linux@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.