[ipxe-devel] [PATCH ipxe v2 3/4] [uuid] parse uuid

[Michael Brown]

On 20/01/17 18:29, David Decotigny wrote:
>   src/core/settings.c       | 28 ++++++++++++++++++++++
>   src/core/uuid.c           | 60 +++++++++++++++++++++++++++++++++++++++++++++++
>   src/include/ipxe/uuid.h   |  1 +
>   src/tests/settings_test.c |  4 ++++

This would add a reasonable amount of extra code to the standard ROM 
builds, which I don't think can easily be justified by the benefit.

You can work around the absence of parse_uuid_setting() by setting 
uuid:hexraw, e.g.

   set uuid:hexraw 1a6a749d0eda461aa87a7cfe4fca4a57

One possible option would be to allow hex_decode() to treat the 
separator character as optional, and then treat :uuid as a variation on 
:hexhyp for parsing purposes.

Another potential problem is that there is no universal agreement on the 
endianness of UUIDs.  I suspect that the use of :hexraw might be the 
cleanest overall solution.

Thanks,

Michael