From: Eric Sandeen <sandeen@sandeen.net>
To: Christoph Hellwig <hch@infradead.org>
Cc: xfs@oss.sgi.com
Subject: Re: [PATCH] xfsprogs: update configure defaults
Date: Wed, 07 Oct 2009 14:01:40 -0500 [thread overview]
Message-ID: <4ACCE594.3050207@sandeen.net> (raw)
In-Reply-To: <20091006151458.GA2560@infradead.org>
Christoph Hellwig wrote:
> Currenly xfsprogs has two different sets for configurating it, those
> in the configure.in template, and those passed to configure by the
> toplevel Makefile when just running make.
>
> Those in the makefile make a lot more sense as they install xfsprogs
> into the / vs /usr split used by most distributions, and enable multi-libs
> when detected.
>
> This patch changes the build system to also use the same defaults when
> running ./configure. To do this we get rid of some of the hacky
> autoconf abuse we had in the past in xfsprogs and just use the normal
> default sbindir and libdir variables for /usr/sbin an /usr/lib respecticly
> and define new root_sbindir and root_libdir variables for those few
> binaries installed into /. The installation to / is disabled if the
> user chooses a non-default exex prefix.
>
> Signed-off-by: Christoph Hellwig <hch@lst.de>
>
This almost works for me in fedora, except that fedora's rpm uses a %configure
which sets --libdir=/usr/lib64 and then I end up with stuff installed in
/usr/lib6464 \o/
With this patch things are ok again if I add --disable-lib64 :
diff --git a/configure.in b/configure.in
index 35a1e6f..78ac522 100644
--- a/configure.in
+++ b/configure.in
@@ -58,7 +58,14 @@ NONE:NONE | NONE:/usr | /usr:*)
root_sbindir='/sbin'
AC_SUBST([root_sbindir])
- root_libdir='/lib'
+ case $libdir in
+ *64)
+ root_libdir='/lib64'
+ ;;
+ *)
+ root_libdir='/lib'
+ ;;
+ esac
AC_SUBST([root_libdir])
esac
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
next prev parent reply other threads:[~2009-10-07 19:00 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-10-06 15:14 [PATCH] xfsprogs: update configure defaults Christoph Hellwig
2009-10-07 19:01 ` Eric Sandeen [this message]
2009-10-07 20:11 ` Christoph Hellwig
2009-10-07 22:06 ` [PATCH v2] " Christoph Hellwig
2009-10-07 23:24 ` Eric Sandeen
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=4ACCE594.3050207@sandeen.net \
--to=sandeen@sandeen.net \
--cc=hch@infradead.org \
--cc=xfs@oss.sgi.com \
/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.