On 02/ 4/11 06:47 AM, Dermot McCluskey wrote:
Niall,

Is there a file that defines all the supported GUIDs?
/usr/include/sys/efi_partition.h seems to have at least the Solaris
value you are defaulting to.

My preference would be to either require the user to enter the guid
hex string, but refer them to a local file that defines all the supported
values (in which case I call the attrib "guid", not "guid_type"?

or switch to using meaning names, but have a list of the supported
values hard-coded in the DTD, eg:
     <!ATTLIST gpt_slice gpt_partition_type (solaris|esp|....) "solaris">
plus a comment that explains that these values are mapped to GUIDs
internally.


I think that's unnecessarily restrictive. New types are liable to be defined from time to time, and allowing GUIDs as a fallback when we have not defined a token (we may also not care to bother defining tokens for all possible usages) seems to have little cost and gain flexibility and robustness[1].

Dave

[1] http://en.wikipedia.org/wiki/Robustness_principle
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss

Reply via email to