Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Nathanael Nerode <neroden@twcny.rr.com>
To: gcc-patches@gcc.gnu.org, gdb-patches@sources.redhat.com,
	binutils@sources.redhat.com, aoliva@redhat.com
Subject: Re: (top level patch) Autoconfiscate.  (Woo!)
Date: Fri, 27 Dec 2002 23:23:00 -0000	[thread overview]
Message-ID: <20021228070541.GA4340@doctormoo> (raw)

>> Canadian cross (host=i386-cygwin, target=powerpc-eabisim) produced a 
>> Makefile with no significant differences.  There was a set of 
>> spurious differences:  I didn't have all the "i386-cygwin-foo" and 
>> "powerpc-eabi-foo" tools in place.
>> Autoconf actually checks to see if they're present, and if they're 
>> not, defaults to the unprefixed 'foo' tools.
>
>This is a mistake.  You could end up thinking you built tools for one
>platform but that will actually work on your native platform.  Please
>arrange for host and target tools not to be too smart in this regard,
>and actually not go for unprefixed unless we know we don't really need
>a cross.
Will start working on this.  There are a couple of ways to do it, but 
the easiest is to switch over to the STRICT versions included in 
acx.m4.  Heh....  I'll probably have that patch submitted by 
tomorrow; it's simple enough.

>> +# _NCN_TOOL_PREFIXES:  Some stuff that oughtta be done in 
>AC_CANONICAL_SYSTEM 
>
>Would you mind renaming the _NCN macros to say _TOPLEV or so?  Nothing
>personal, it's just...  Well...  You understand, don't you? :-)

Heh.  Well, there were CYG_ macros, and DJ_ macros, and...

Anyway, once I get them in their 'happiest form', I'm going to submit 
them to the autoconf maintainers to become AC_* (and I expect to get 
them in, since they're pretty simple).  So, that will take care of that 
in the long run.

>> +# clear some things potentially inherited from environment.
>> +
>> +enable_threads=no
>> +enable_shared=no
>> +enable_libstdcxx_v3=yes
>
>I'm not sure this is actually necessary.  Traditionally, configure
>scripts have taken --with/--enable arguments from the environment too,
>even if it's just because such variables are not cleaned up.  I don't
>really see a reason to override this behavior.

OK, will strip.  I have to make sure the defaults are correct anyway,
so this will probably come as part of a larger patch to handle 
enable_ and with_ stuff in a more autoconfy manner.

>> +        PATH=$PATH:${PWD=`${PWDCMD-pwd}`} ; export PATH
>
>Huh?  Any idea why we need this?  (I realize it's inherited from the
>old configure, but it would be nice if we could get rid of this at
>some point.  It is *so* wrong!)

I have absolutely no idea why we need this; I don't think we run any
programs in the toplevel working directory.  I'll submit a patch to
delete that.

--Nathanael


             reply	other threads:[~2002-12-28  7:07 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-12-27 23:23 Nathanael Nerode [this message]
  -- strict thread matches above, loose matches on Subject: below --
2002-12-27 20:03 Nathanael Nerode
2002-12-27 23:07 ` Alexandre Oliva
2002-12-28  7:06 ` Mark Kettenis
2002-12-28  8:24   ` Alexandre Oliva
2002-12-28 10:05     ` Alexandre Oliva
2002-12-28 10:08       ` Daniel Jacobowitz
2002-12-28 10:34         ` Alexandre Oliva

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=20021228070541.GA4340@doctormoo \
    --to=neroden@twcny.rr.com \
    --cc=aoliva@redhat.com \
    --cc=binutils@sources.redhat.com \
    --cc=gcc-patches@gcc.gnu.org \
    --cc=gdb-patches@sources.redhat.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox