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
next prev 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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).