On 02/06/2019 11:03 AM, Geert Stappers wrote:
On Wed, Feb 06, 2019 at 10:16:44AM -0800, aaron.yo...@oracle.com wrote:
On 02/06/2019 05:42 AM, Geert Stappers wrote:
On Tue, Feb 05, 2019, Aaron Young wrote:
Implement scaffolding for new efimap command.
...
index 3c14a2c..a902934 100644
--- a/src/config/general.h
+++ b/src/config/general.h
@@ -154,6 +154,7 @@ FILE_LICENCE ( GPL2_OR_LATER_OR_UBDL );
//#define PROFSTAT_CMD /* Profiling commands */
//#define NTP_CMD /* NTP commands */
//#define CERT_CMD /* Certificate management commands */
+#define EFIMAP_CMD /* EFI Map command */
/*
* ROM-specific options
The new command is default enabled.
Please advise or clarify your comment.
That I would expect
--- a/src/config/general.h
+++ b/src/config/general.h
@@ -154,6 +154,7 @@ FILE_LICENCE ( GPL2_OR_LATER_OR_UBDL );
//#define PROFSTAT_CMD /* Profiling commands */
//#define NTP_CMD /* NTP commands */
//#define CERT_CMD /* Certificate management commands */
+ //#define EFIMAP_CMD /* EFI Map command */
/*
* ROM-specific options
Or
--- a/src/config/general.h
+++ b/src/config/general.h
@@ -136,6 +136,7 @@ FILE_LICENCE ( GPL2_OR_LATER_OR_UBDL );
#define LOGIN_CMD /* Login command */
#define SYNC_CMD /* Sync command */
#define SHELL_CMD /* Shell command */
+#define EFIMAP_CMD /* EFI Map command */
//#define NSLOOKUP_CMD /* DNS resolving command */
//#define TIME_CMD /* Time commands */
//#define DIGEST_CMD /* Image crypto digest commands */
Yes, I should have raised clearly the question:
Should the efimap command be default enabled?
Yes, it should. The functionality to boot a local UEFI disk is
enabled by default and the efimap command is an integral part of that
functionality.
P.S.
Other ideas how to prevent that patches get lost in email traffic are welcome.
I'm trying to care about the cool iPXE project.
It does happen that submitted patches are unanswered for weeks.
Patch submitters will think: "Why should I bother again?" Or "Does
really no-one care?"
My simple
The new command is default enabled.
was for saying "Some one does care".
OK. Thank you for your review comments!
-Aaron
Cheers
Geert Stappers
DevOps Engineer at Hendrikx-ITC
_______________________________________________
ipxe-devel mailing list
ipxe-devel@lists.ipxe.org
https://lists.ipxe.org/mailman/listinfo.cgi/ipxe-devel
_______________________________________________
ipxe-devel mailing list
ipxe-devel@lists.ipxe.org
https://lists.ipxe.org/mailman/listinfo.cgi/ipxe-devel
