Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Alexandre Oliva <aoliva@redhat.com>
To: Nathanael Nerode <neroden@doctormoo.dyndns.org>
Cc: gcc-patches@gcc.gnu.org, gdb-patches@sources.redhat.com,
	binutils@sources.redhat.com, dj@redhat.com
Subject: Re: toplevel Makefile.in: generate with autogen
Date: Sun, 22 Sep 2002 13:17:00 -0000	[thread overview]
Message-ID: <ork7le6m6n.fsf@free.redhat.lsd.ic.unicamp.br> (raw)
In-Reply-To: <20020922021604.GA32268@doctormoo.dyndns.org>

On Sep 21, 2002, Nathanael Nerode <neroden@doctormoo.dyndns.org> wrote:

> This change causes toplevel Makefile.in to be generated by autogen from
> Makefile.tpl and Makefile.def; consensus was reached a while back that
> this was a reasonable thing to do.

Hmm...  I thought this would also move into Makefile.def the
dependencies of each module (configure, build and perhaps install
dependencies).  I think this would be a nice improvement in the
future, since the information would all be collected in a single
place.

Also, I've noticed that parallel make recently hasn't been as
efficient as it was in the past as to processing multiple directories
simultaneously.  It appears that these days it only processes
directories sequentially, even though they're not using as many
processes as specified for the top-level make -j.  I thought this
might have been caused by changes in make itself, but last week I've
been working with parallel makes on an old branch and I found out it
was not the case: it's actually something that changed in the GCC
build infrastructure.  Could I perhaps convince you to have a look?
:-)

I'd approve this patch (with the same caveats) if it had a ChangeLog
entry and it accompanied documentation changes that introduced the
dependency on autogen and explained how to rebuild Makefile.in from
its new dependencies.  A new rule to that end in Makefile.in would
definitely be welcome.  config/gcc_update should also be adjusted.

-- 
Alexandre Oliva   Enjoy Guarana', see http://www.ic.unicamp.br/~oliva/
Red Hat GCC Developer                 aoliva@{redhat.com, gcc.gnu.org}
CS PhD student at IC-Unicamp        oliva@{lsd.ic.unicamp.br, gnu.org}
Free Software Evangelist                Professional serial bug killer


      reply	other threads:[~2002-09-22 20:17 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-09-21 19:20 Nathanael Nerode
2002-09-22 13:17 ` Alexandre Oliva [this message]

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=ork7le6m6n.fsf@free.redhat.lsd.ic.unicamp.br \
    --to=aoliva@redhat.com \
    --cc=binutils@sources.redhat.com \
    --cc=dj@redhat.com \
    --cc=gcc-patches@gcc.gnu.org \
    --cc=gdb-patches@sources.redhat.com \
    --cc=neroden@doctormoo.dyndns.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