From: Matt Wilson <msw@amazon.com>
To: Jan Beulich <JBeulich@suse.com>
Cc: Ian Jackson <Ian.Jackson@eu.citrix.com>,
Ian Campbell <ian.campbell@citrix.com>,
xen-devel <xen-devel@lists.xen.org>
Subject: Re: [PATCH] tools: don't expand prefix and exec_prefix too early
Date: Wed, 8 Aug 2012 07:45:00 -0700 [thread overview]
Message-ID: <20120808144500.GD5592@US-SEA-R8XVZTX> (raw)
In-Reply-To: <502296310200007800093A35@nat28.tlf.novell.com>
On Wed, Aug 08, 2012 at 07:39:13AM -0700, Jan Beulich wrote:
> >>> On 08.08.12 at 16:26, "Jan Beulich" <JBeulich@suse.com> wrote:
> > A comment in tools/configure says that it is intended for these to be
> > command line overridable, so they shouldn't get expanded at configure
> > time.
>
> In addition, it would have been _very_ nice if it had been
> prominently announced that with (I believe) 25594:ad08cd8e7097
>
> it is now _required_ to configure with --libdir on x86-64, or else all
> the .so-s end up under /usr/lib. Figuring this out and getting the
> patch here in the right form to be able to use the most compatible
> form --libdir='${exec_prefix}'/lib64 has taken me a good part of
> the day, which could have been avoided if this whole configure
> adjustment (much like had apparently been missing already in
> earlier cases) had been done properly. I just can't imagine I'm
> the only one having used no options at all, and things working
> nevertheless despite .../lib not being in the library search paths
> used when running xl et al.
I'm sorry for the trouble it cause you today, Jan. I thought I called
it out sufficiently in the commit log:
With this change, packagers can supply the desired location for
shared libraries on the ./configure command line. Packagers need
to note that the default behaviour on 64-bit Linux systems will be
to install shared libraries in /usr/lib, not /usr/lib64, unless a
--libdir value is provided to ./configure.
The new behavior is consistent with all packages that use autoconf.
Would a separate email on the topic to xen-devel, apart from the patch
discussion, have helped raise awareness?
Matt
next prev parent reply other threads:[~2012-08-08 14:45 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-08-08 14:26 [PATCH] tools: don't expand prefix and exec_prefix too early Jan Beulich
2012-08-08 14:39 ` Jan Beulich
2012-08-08 14:45 ` Matt Wilson [this message]
2012-08-08 14:52 ` Jan Beulich
2012-08-08 14:55 ` Ian Campbell
2012-08-08 15:11 ` Olaf Hering
2012-08-08 15:18 ` Ian Campbell
2012-08-08 15:34 ` Ian Jackson
2012-08-08 15:10 ` Matt Wilson
2012-08-08 15:53 ` Jan Beulich
2012-08-08 16:06 ` Matt Wilson
2012-08-08 16:08 ` Jan Beulich
2012-08-13 13:50 ` Ian Jackson
2012-08-13 17:21 ` Matt Wilson
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=20120808144500.GD5592@US-SEA-R8XVZTX \
--to=msw@amazon.com \
--cc=Ian.Jackson@eu.citrix.com \
--cc=JBeulich@suse.com \
--cc=ian.campbell@citrix.com \
--cc=xen-devel@lists.xen.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.