Re: [PATCH] scripts/ver_linux : correct printing of binutils version

2007-08-28 Thread Sam Ravnborg
On Thu, Aug 02, 2007 at 01:21:15AM +0200, Jesper Juhl wrote:
> Hi,
> 
> Currently scripts/ver_linux prints "Binutils" or other random 
> information for the version number in the "binutils" output line 
> on some distributions. This patch corrects that.
> 
> When I initially submitted a patch to correct that, I was not aware 
> that the output from "ld -v" could differ as much as it turned out 
> it can, so my original fix turned out to not cover all bases.
> This patch works correctly with all the different "ld -v" output 
> that people posted in replys to my first patch, so it should be a 
> clear win over what we have currently.
> 
> Please apply.

Done,

Sam
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH] scripts/ver_linux : correct printing of binutils version

2007-08-28 Thread Sam Ravnborg
On Thu, Aug 02, 2007 at 01:21:15AM +0200, Jesper Juhl wrote:
 Hi,
 
 Currently scripts/ver_linux prints Binutils or other random 
 information for the version number in the binutils output line 
 on some distributions. This patch corrects that.
 
 When I initially submitted a patch to correct that, I was not aware 
 that the output from ld -v could differ as much as it turned out 
 it can, so my original fix turned out to not cover all bases.
 This patch works correctly with all the different ld -v output 
 that people posted in replys to my first patch, so it should be a 
 clear win over what we have currently.
 
 Please apply.

Done,

Sam
-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH] scripts/ver_linux : correct printing of binutils version

2007-08-01 Thread Jesper Juhl
Hi,

Currently scripts/ver_linux prints "Binutils" or other random 
information for the version number in the "binutils" output line 
on some distributions. This patch corrects that.

When I initially submitted a patch to correct that, I was not aware 
that the output from "ld -v" could differ as much as it turned out 
it can, so my original fix turned out to not cover all bases.
This patch works correctly with all the different "ld -v" output 
that people posted in replys to my first patch, so it should be a 
clear win over what we have currently.

Please apply.


Signed-off-by: Jesper Juhl <[EMAIL PROTECTED]>
---

 scripts/ver_linux |4 +---
 1 files changed, 1 insertions(+), 3 deletions(-)

diff --git a/scripts/ver_linux b/scripts/ver_linux
index 8f8df93..27a5a21 100755
--- a/scripts/ver_linux
+++ b/scripts/ver_linux
@@ -21,9 +21,7 @@ gcc --version 2>&1| grep gcc | awk \
 make --version 2>&1 | awk -F, '{print $1}' | awk \
   '/GNU Make/{print "Gnu make  ",$NF}'
 
-ld -v | awk -F\) '{print $1}' | awk \
-'/BFD/{print "binutils  ",$NF} \
-/^GNU/{print "binutils  ",$4}'
+echo "binutils   $(ld -v | egrep -o '[0-9]+\.[0-9\.]+')"
 
 echo -n "util-linux "
 fdformat --version | awk '{print $NF}' | sed -e s/^util-linux-// -e s/\)$//




-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH] scripts/ver_linux : correct printing of binutils version

2007-08-01 Thread Jesper Juhl
Hi,

Currently scripts/ver_linux prints Binutils or other random 
information for the version number in the binutils output line 
on some distributions. This patch corrects that.

When I initially submitted a patch to correct that, I was not aware 
that the output from ld -v could differ as much as it turned out 
it can, so my original fix turned out to not cover all bases.
This patch works correctly with all the different ld -v output 
that people posted in replys to my first patch, so it should be a 
clear win over what we have currently.

Please apply.


Signed-off-by: Jesper Juhl [EMAIL PROTECTED]
---

 scripts/ver_linux |4 +---
 1 files changed, 1 insertions(+), 3 deletions(-)

diff --git a/scripts/ver_linux b/scripts/ver_linux
index 8f8df93..27a5a21 100755
--- a/scripts/ver_linux
+++ b/scripts/ver_linux
@@ -21,9 +21,7 @@ gcc --version 21| grep gcc | awk \
 make --version 21 | awk -F, '{print $1}' | awk \
   '/GNU Make/{print Gnu make  ,$NF}'
 
-ld -v | awk -F\) '{print $1}' | awk \
-'/BFD/{print binutils  ,$NF} \
-/^GNU/{print binutils  ,$4}'
+echo binutils   $(ld -v | egrep -o '[0-9]+\.[0-9\.]+')
 
 echo -n util-linux 
 fdformat --version | awk '{print $NF}' | sed -e s/^util-linux-// -e s/\)$//




-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/