LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
From: Oleg Verych <olecom@flower.upol.cz>
To: Tim Schmielau <tim@physik3.uni-rostock.de>
Cc: Rob Landley <rob@landley.net>, Andrew Morton <akpm@osdl.org>,
	Sam Ravnborg <sam@ravnborg.org>,
	Roman Zippel <zippel@linux-m68k.org>,
	LKML <linux-kernel@vger.kernel.org>
Subject: scripts/makelst: bc -> shell Re: sed _s_gnu_alternatives_ (Re: [rft] (g)awk substitution)
Date: Sat, 27 Jan 2007 06:38:36 +0000	[thread overview]
Message-ID: <20070127063836.GA15178@flower.upol.cz> (raw)
In-Reply-To: <Pine.LNX.4.63.0701270113400.4567@gockel.physik3.uni-rostock.de>

Hallo.

On Sat, Jan 27, 2007 at 01:18:30AM +0100, Tim Schmielau wrote:
> On Fri, 26 Jan 2007, Oleg Verych wrote:
> 
> > In Debian's version it has no `bc', while native `bc' is producing
> > crap like that:
> > ,-*- bash -*-
> > |olecom@flower:~$ printf "%u" $(( -1 )) ; echo
> > |18446744073709551615
> > |olecom@flower:~$ echo -1 | bc
> > |-1
> > |olecom@flower:~$ printf "%u" $(( 0xFF )) ; echo
> > |255
> > |olecom@flower:~$ echo FF | bc
> > |99
> > |olecom@flower:~$ echo F | bc
> > |15
> > |olecom@flower:~$
> > `-*-
> > 
> > i can't understand the result... I think linux/scripts/makelst may be
> > updated to have sh's version of address calculation ;)
> 
> No, IMHO that should not be a reason to change anything in the kernel.
> Just set ibase=16 or read the fine manual:

Thank you very much ;) Maybe i must use XSLT with 0.99 kernel, or
what?

I'm talking about patch posted here:
Message-ID: <E1H9rZ3-0002T5-Po@flower>
Archived-At: <http://permalink.gmane.org/gmane.linux.kernel/485999>

,-*- diff snip -*-
|-    t4=`echo $t3 | gawk '{ print $1 }'`
|-    t5=`echo $t1 | gawk '{ print $1 }'`
|+    t4=`pos_param 1 $t3`
|+    t5=`pos_param 1 $t1`
5     t6=`echo $t4 - $t5 | tr a-f A-F`
6     t7=`( echo  ibase=16 ; echo $t6 ) | bc`
`-*-

I've just noticed, that things on lines 5 and 6 may be optimized.

t7=`printf "%lu" $(( 0x$t4 - 0x$t5 ))`

No tr, no bc, no pipes -- magic!
Better know and use our *fine shell*.

And if you jouined us, Tim, maybe can share your experience and test
patch above and this one, please?
Message-ID: <E1H9qr4-0002Nu-RL@flower>
Archived-At: <http://permalink.gmane.org/gmane.linux.kernel/485984>

It's just my optimization hint. I have no bc, and i'm sure
things, like that will cause silent misinformation.

Anyway, what is so bad in `sh'?

Thanks.

-*- OT -*-
>   info '(bc)About Expressions and Special Variables'
GFDL. Enough of that bull, even Documentation/ is *free* from it.

Good bye.
--
-o--=O`C  info emacs : not found  /. .\ ( is there any reason to live? )
 #oo'L O  info make  : not found      o (    yes --- R.I.P. FSF+RMS    )
<___=E M  man gcc    : not found    `-- ( viva Debian Operating System )

  reply	other threads:[~2007-01-27  6:30 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-01-15 21:24 [PATCH] sed s/gawk/awk/ scripts/gen_init_ramfs.sh Rob Landley
2007-01-15 22:11 ` Sam Ravnborg
2007-01-24  0:49   ` Andrew Morton
2007-01-24 17:30     ` Rob Landley
2007-01-24 21:03       ` [rft] (g)awk substitution (Re: [PATCH] sed s/gawk/awk/scripts/gen_init_ramfs.sh) Oleg Verych
2007-01-24 22:44         ` [patch, rft] scripts/gen_initramfs_list.sh: replace gawk with shell, whitespace cleanup Oleg Verych
2007-01-24 23:30         ` [patch, rft] scripts/makelst: replace gawk with shell, update Oleg Verych
2007-01-25  1:51         ` [rft] (g)awk substitution (Re: [PATCH] sed s/gawk/awk/scripts/gen_init_ramfs.sh) Rob Landley
2007-01-25  9:40           ` sed _s_gnu_alternatives_ (Re: [rft] (g)awk substitution) Oleg Verych
2007-01-25 18:03             ` Rob Landley
2007-01-25 19:14               ` Oleg Verych
2007-01-25 19:38                 ` Rob Landley
2007-01-25 21:18                   ` Oleg Verych
2007-01-26 17:47                     ` Rob Landley
2007-01-26 20:14                       ` Oleg Verych
2007-01-27  0:18                         ` Tim Schmielau
2007-01-27  6:38                           ` Oleg Verych [this message]
2007-02-03 11:24                             ` scripts/makelst: bc -> shell " David Weinehall
2007-02-03 13:10                               ` Oleg Verych
2007-02-03 11:13                 ` David Weinehall
2007-02-02 16:36     ` [PATCH] sed s/gawk/awk/ scripts/gen_init_ramfs.sh Adrian Bunk
2007-02-02 21:01       ` Oleg Verych

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=20070127063836.GA15178@flower.upol.cz \
    --to=olecom@flower.upol.cz \
    --cc=akpm@osdl.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=rob@landley.net \
    --cc=sam@ravnborg.org \
    --cc=tim@physik3.uni-rostock.de \
    --cc=zippel@linux-m68k.org \
    --subject='scripts/makelst: bc -> shell Re: sed _s_gnu_alternatives_ (Re: [rft] (g)awk substitution)' \
    /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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).