All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] strtonum() is gawk specific extension
@ 2014-06-15 13:56 Vasili Galka
  2014-06-18  6:10 ` Simon Glass
  0 siblings, 1 reply; 3+ messages in thread
From: Vasili Galka @ 2014-06-15 13:56 UTC (permalink / raw)
  To: u-boot

The following patch "Check that u-boot.bin size looks correct":
http://patchwork.ozlabs.org/patch/355151/
Uses strtonum() function which is gawk specific extension:
https://www.gnu.org/software/gawk/manual/html_node/Strtonum-Function.html

As a result, currenlty, u-boot/master generates build errors on systems
with alternative awk implementations e.g. I get:

awk: line 2: function strtonum never defined

Either gawk shall be required and explicitly used or the
implementation changed.

Best,
Vasili

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [U-Boot] strtonum() is gawk specific extension
  2014-06-15 13:56 [U-Boot] strtonum() is gawk specific extension Vasili Galka
@ 2014-06-18  6:10 ` Simon Glass
  2014-06-18  6:32   ` Vasili Galka
  0 siblings, 1 reply; 3+ messages in thread
From: Simon Glass @ 2014-06-18  6:10 UTC (permalink / raw)
  To: u-boot

Hi,

On 15 June 2014 06:56, Vasili Galka <vvv444@gmail.com> wrote:
> The following patch "Check that u-boot.bin size looks correct":
> http://patchwork.ozlabs.org/patch/355151/
> Uses strtonum() function which is gawk specific extension:
> https://www.gnu.org/software/gawk/manual/html_node/Strtonum-Function.html
>
> As a result, currenlty, u-boot/master generates build errors on systems
> with alternative awk implementations e.g. I get:
>
> awk: line 2: function strtonum never defined
>
> Either gawk shall be required and explicitly used or the
> implementation changed.

I thought of an ugly way and sent a patch using bc, but please let me
know if you have something better.

Regards,
Simon

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [U-Boot] strtonum() is gawk specific extension
  2014-06-18  6:10 ` Simon Glass
@ 2014-06-18  6:32   ` Vasili Galka
  0 siblings, 0 replies; 3+ messages in thread
From: Vasili Galka @ 2014-06-18  6:32 UTC (permalink / raw)
  To: u-boot

Hi Simon,

On Wed, Jun 18, 2014 at 9:10 AM, Simon Glass <sjg@chromium.org> wrote:
> I thought of an ugly way and sent a patch using bc, but please let me
> know if you have something better.

I honestly don't know. Is bc considered "standard" and always comes on
Linux systems?
Maybe there shall be a list of prerequisites in README file?
For example some configurations require openssl on the host.

Best,
Vasili

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2014-06-18  6:32 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-06-15 13:56 [U-Boot] strtonum() is gawk specific extension Vasili Galka
2014-06-18  6:10 ` Simon Glass
2014-06-18  6:32   ` Vasili Galka

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.