'Re: Subject: [PATCH] ash: make 'sleep' a builtin'

[prev in list] [next in list] [prev in thread] [next in thread] 

List:       busybox
Subject:    Re: Subject: [PATCH] ash: make 'sleep' a builtin
From:       shawnlandden () tutanota ! com
Date:       2022-09-08 21:35:08
Message-ID: NBU0mu8--3-2 () tutanota ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]

Thanks for merging this feature.

The builtin is exiting ash in two cases:

1.If no argument is given (also does not print usage of sleep, but ash---look at how \
printf handles his) 2.If xatou() interpretation of the number fails. This is a bit \
more involved to fix, which is why I didn't send my patch for #1.

30 ago 2022, 8:14 por vda.linux@googlemail.com:

> On Tue, Aug 30, 2022 at 10:10 AM David Laight <David.Laight@aculab.com> wrote:
> 
> > 
> > One question is why?
> > 
> > Shell builtins (mostly) exist to speed things up.
> > 
> > But sleep doesn't need speeding up.
> > 
> 
> Avoiding having another, possibly long-living process
> 

[Attachment #5 (text/html)]

<html>
  <head>
    <meta http-equiv="content-type" content="text/html; charset=UTF-8">
  </head>
  <body>
<div>Thanks for merging this feature.<br></div><div dir="auto"><br></div><div \
dir="auto">The builtin is exiting ash in two cases:<br></div><div \
dir="auto"><br></div><div dir="auto">1.If no argument is given (also does not print \
usage of sleep, but ash---look at how printf handles his)<br></div><div \
dir="auto">2.If xatou() interpretation of the number fails. This is a bit more \
involved to fix, which is why I didn't send my patch for \
#1.<br></div><div><br></div><div><br></div><div>30 ago 2022, 8:14 por \
vda.linux@googlemail.com:<br></div><blockquote class="tutanota_quote" \
style="border-left: 1px solid #93A3B8; padding-left: 10px; margin-left: 5px;"><div>On \
Tue, Aug 30, 2022 at 10:10 AM David Laight &lt;David.Laight@aculab.com&gt; \
wrote:<br></div><blockquote><div><br></div><div>One question is \
why?<br></div><div><br></div><div>Shell builtins (mostly) exist to speed things \
up.<br></div><div><br></div><div>But sleep doesn't need speeding \
up.<br></div></blockquote><div><br></div><div>Avoiding having another, possibly \
long-living process<br></div></blockquote><div dir="auto"><br></div>  </body> </html>

_______________________________________________
busybox mailing list
busybox@busybox.net
http://lists.busybox.net/mailman/listinfo/busybox

[prev in list] [next in list] [prev in thread] [next in thread] 