[ipxe-devel] missing commands

[Andrew Bobulsky]

Hrmm:

> Anyone have any strong thoughts on whether or not "reboot" should be included
> by default?  Total cost is only 65 bytes, so if at least one person thinks
> it's a good idea, I'll do it!  :)

While it would probably be a good idea for the sake of completeness,
one /could/ argue its necessity.  It'd be handiest in scripts, but the
kind of scripts that I use (server-side) would have ready access to
reboot.c32, which I've used fairly often.

For manual cases, there's always the three finger salute. ;)

However (and this might be more relevant), when I *first* started
using gPXE, I would have unequivocally said "yes" to the proposition
;)

Cheers,
Andrew Bobulsky

On Wed, Oct 26, 2011 at 11:36 AM, Michael Brown <mbrown at fensystems.co.uk> wrote:
> On Wednesday 26 Oct 2011 15:41:16 Andrew Bobulsky wrote:
>> While iPXE does contain and support those commands, they're often left
>> /#undef/ined in src/config/general.h.  I believe that the reason for
>> doing so is to keep compiled size down for ROM-based iPXE deployments.
>>
>> "time" and "reboot" are two of these such commands, though browsing
>> the git log, I see that Michael has put "sleep" in by default; cheers
>> to that!
>>
>> You can get a quick idea of what is and isn't available here, with the
>> commands you're interested in starting at line 123:
>> https://git.ipxe.org/ipxe.git/blob/bf2da3122b988cd277da435ee76bc35ad438e8af
>> :/src/config/general.h
>>
>> On a related note, the documentation doesn't state that these commands
>> need to be enabled, and perhaps it should.  I remember being confused
>> the first time I tried to "sleep" and got an error back from the CLI
>
> I've updated the documentation for the commands that aren't enabled by default
> to mention the build options in config/general.h:
>
>    http://ipxe.org/cmd/reboot
>    http://ipxe.org/cmd/time
>    (and others)
>

>
> Michael
>