From: Sam Ravnborg <sam@ravnborg.org>
To: Tom Rini <trini@kernel.crashing.org>
Cc: Linus Torvalds <torvalds@osdl.org>,
Sam Ravnborg <sam@ravnborg.org>,
Kernel Mailing List <linux-kernel@vger.kernel.org>,
Carsten Milling <cmil@hashtable.de>
Subject: Re: [PATCH][PPC32] Fix the 'checkbin' target
Date: Wed, 1 Sep 2004 21:20:54 +0200 [thread overview]
Message-ID: <20040901192054.GA7219@mars.ravnborg.org> (raw)
In-Reply-To: <20040901180356.GD19730@smtp.west.cox.net>
On Wed, Sep 01, 2004 at 11:03:56AM -0700, Tom Rini wrote:
> Hello.
>
> Currently, the checkbin target on PPC32 isn't quite right. First, one
> of the tests (to ensure that some instructions are known to gas) is
> never actually invoked because 'checkbin' doesn't know about stuff set
> in .config, so we always have the 'else' case run. This changes to
> always running the test and telling the user to upgrade to at least
> binutils 2.12.1. The next problem is that we were doing $(AS) -o
> /dev/null ... in both that test, as well as another. The problem here
> is that the checkbin target is run on the install targets, meaning that
> /dev/null will get unlinked when the test passes. To get around this we
> use .tmp_gas_check as the output file instead.
>
> Assuming Sam doesn't object, I hope this can go in quickly. Thanks.
>
> Signed-off-by: Tom Rini <trini@kernel.crashing.org>
Looks good to me - and it solves the critical /dev/null
bug in ppc.
Linus please apply.
Sam
prev parent reply other threads:[~2004-09-01 19:18 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-09-01 18:03 [PATCH][PPC32] Fix the 'checkbin' target Tom Rini
2004-09-01 19:20 ` Sam Ravnborg [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=20040901192054.GA7219@mars.ravnborg.org \
--to=sam@ravnborg.org \
--cc=cmil@hashtable.de \
--cc=linux-kernel@vger.kernel.org \
--cc=torvalds@osdl.org \
--cc=trini@kernel.crashing.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.