public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Marcelo Tosatti <marcelo.tosatti@cyclades.com>
To: joshk@triplehelix.org,
	linux-kernel mailing list <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 2.4] scripts: Support output of new ld
Date: Mon, 20 Sep 2004 13:19:13 -0300	[thread overview]
Message-ID: <20040920161913.GC4501@logos.cnet> (raw)
In-Reply-To: <20040918065733.GA4549@darjeeling.triplehelix.org>

[-- Attachment #1: Type: text/plain, Size: 1466 bytes --]


Hi Joshua, 

(We met at Palo Alto in a Debian dinner on a chinese restaurant, remember? :))


Unfortunately this patch doenst apply cleanly

[marcelo@logos linux-2.4]$ fp /tmp/gnu
bk import -tpatch -r -CR -yscripts: Support output of new ld /tmp/patch4572 .
Patching...
Patch failed.  **** patch log follows ****
Patching file scripts/ver_linux
1 out of 1 hunk FAILED -- saving rejects to file scripts/ver_linux.rej

Its easy enough to be applied by hand but I prefer if you generate
a clean patch instead

reject file attached



On Fri, Sep 17, 2004 at 11:57:33PM -0700, Joshua Kwan wrote:
> Hello,
> 
> This is one in a handful of small patches that I'll be sending along in
> the near future. This patch allows scripts/ver_linux to find out the ld
> version for versions of ld that have different output on 'ld -v' (new
> ones have "GNU" at the beginning.)
> 
> Marcelo, please apply. 
> 
> Signed-off-by: Joshua Kwan <joshk@triplehelix.org>
> 
> -- 
> Joshua Kwan
> 
> --- a/scripts/ver_linux	2004-09-05 01:31:23.000000000 -0700
> +++ b/scripts/ver_linux	2004-09-05 01:31:47.000000000 -0700
> @@ -22,7 +22,8 @@
>        '/GNU Make/{print "Gnu make              ",$NF}'
>  
>  ld -v 2>&1 | awk -F\) '{print $1}' | awk \
> -      '/BFD/{print "binutils              ",$NF}'
> +      '/BFD/{print "binutils              ",$NF}
> +       /^GNU/{print "binutils              ",$4}'
>  
>  fdformat --version | awk -F\- '{print "util-linux            ", $NF}'
>  



[-- Attachment #2: ver_linux.rej --]
[-- Type: text/plain, Size: 579 bytes --]

***************
*** 22,28 ****
        '/GNU Make/{print "Gnu make              ",$NF}'
  20
  ld -v 2>&1 | awk -F\) '{print $1}' | awk \
-       '/BFD/{print "binutils              ",$NF}'
  20
  fdformat --version | awk -F\- '{print "util-linux            ", $NF}'
  20
--- 22,29 ----
        '/GNU Make/{print "Gnu make              ",$NF}'
  20
  ld -v 2>&1 | awk -F\) '{print $1}' | awk \
+       '/BFD/{print "binutils              ",$NF}
+        /^GNU/{print "binutils              ",$4}'
  20
  fdformat --version | awk -F\- '{print "util-linux            ", $NF}'
  20

  reply	other threads:[~2004-09-20 17:46 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-09-18  6:57 [PATCH 2.4] scripts: Support output of new ld Joshua Kwan
2004-09-20 16:19 ` Marcelo Tosatti [this message]
2004-09-20 23:56   ` Joshua Kwan

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=20040920161913.GC4501@logos.cnet \
    --to=marcelo.tosatti@cyclades.com \
    --cc=joshk@triplehelix.org \
    --cc=linux-kernel@vger.kernel.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