This is the mail archive of the
binutils@sources.redhat.com
mailing list for the binutils project.
Re: PR/17383: fix gcc in-srcdir build (take 2)
- From: Alexandre Oliva <aoliva at redhat dot com>
- To: neroden at twcny dot rr dot com (Nathanael Nerode)
- Cc: gcc-patches at gcc dot gnu dot org, paolo dot bonzini at lu dot unisi dot ch, binutils at sources dot redhat dot com, gdb-patches at sources dot redhat dot com, dj at delorie dot com
- Date: 26 Feb 2005 23:47:22 -0300
- Subject: Re: PR/17383: fix gcc in-srcdir build (take 2)
- Organization: Red Hat Global Engineering Services Compiler Team
- References: <20050226193558.GA5098@twcny.rr.com>
On Feb 26, 2005, neroden@twcny.rr.com (Nathanael Nerode) wrote:
> I've been wanting to do this for yonks. Approved for mainline, unless DJ or
> Alex objects. Give them a day or two.
DJ has objections to the change, unfortunately. Personally, I don't
have any problem with this change: it will only affect the ./configure
case, that has long been recommended against, and even then, will only
make a difference for someone for whom the change would actually make
some difference, e.g., not someone who's only half-mindlessly building
and installing the package. If you need to enter some sub-package
build-for-host directory, you'll probably be able to figure out what's
the right thing to do. If you have scripts to automate some of these
steps, then you probably do this often enough that you probably don't
want to pollute your source tree with builds and already follow the
recommended procedure to build out of the source tree. So, I'd
approve this change if it wasn't for DJ's objection.
DJ, given my arguments above, would you be willing to withdraw your
objection?
Any comments for binutils or GDB folks? Does anyone object to moving
the directories in which we build tools for the host to a host subdir
when srcdir=.?
--
Alexandre Oliva http://www.ic.unicamp.br/~oliva/
Red Hat Compiler Engineer aoliva@{redhat.com, gcc.gnu.org}
Free Software Evangelist oliva@{lsd.ic.unicamp.br, gnu.org}