tools: env: allow to print U-Boot version
PATCH 1/1 tools: env: allow to print U-Boot version
From: Stefan Agner <[email protected]>
The fw_env utility family has a default environment compiled in which ties it quite strongly to the U-Boot source/config it has been built with. Allow to display the U-Boot version it has been built with using the -v/--version argument.
---
tools/env/fw_env_main.c | 14 +++++++++++--- 1 file changed, 11 insertions(+), 3 deletions(-)
tools/env/fw_env_main.c +11 -3
@@ -34,6 +34,7 @@#include <stdlib.h> #include <sys/file.h> #include <unistd.h> +#include <version.h> #include "fw_env_private.h" #include "fw_env.h"
@@ -48,6 +49,7 @@ static struct option long_options[] = { {"script", required_argument, NULL, 's'}, {"noheader", required_argument, NULL, 'n'}, {"lock", required_argument, NULL, 'l'}, + {"version", no_argument, NULL, 'v'}, {NULL, 0, NULL, 0} };
@@ -67,6 +69,7 @@ void usage_printenv(void)"Print variables from U-Boot environment\n" "\n" " -h, --help print this help.\n" +" -v, --version display version\n" #ifdef CONFIG_ENV_AES" -a, --aes aes key to access environment\n" #endif
@@ -85,6 +88,7 @@ void usage_env_set(void)"Modify variables in U-Boot environment\n" "\n" " -h, --help print this help.\n" +" -v, --version display version\n" #ifdef CONFIG_ENV_AES" -a, --aes aes key to access environment\n" #endif
@@ -123,7 +127,7 @@ static void parse_common_args(int argc, char *argv[]) env_opts.config_file =CONFIG_FILE ;#endif -while ((c =getopt_long (argc, argv,":a:c:l:h" , long_options,NULL )) != +while ((c =getopt_long (argc, argv,":a:c:l:h:v" , long_options,NULL )) !=EOF ) {switch (c) {case 'a' :
@@ -145,6 +149,10 @@ static void parse_common_args(int argc, char *argv[]) do_printenv ?usage_printenv () :usage_env_set ();exit (EXIT_SUCCESS );break ; +case 'v' : +fprintf (stderr,"Compiled with " U_BOOT_VERSION "\n" ); +exit (EXIT_SUCCESS ); +break ;default :/* ignore unknown options */ break ;
@@ -162,7 +170,7 @@ int parse_printenv_args(int argc, char *argv[])parse_common_args (argc, argv); -while ((c =getopt_long (argc, argv,"a:c:ns:l:h" , long_options,NULL )) +while ((c =getopt_long (argc, argv,"a:c:ns:l:h:v" , long_options,NULL )) !=EOF ) {switch (c) {case 'n' :
@@ -189,7 +197,7 @@ int parse_setenv_args(int argc, char *argv[])parse_common_args (argc, argv); -while ((c =getopt_long (argc, argv,"a:c:ns:l:h" , long_options,NULL )) +while ((c =getopt_long (argc, argv,"a:c:ns:l:h:v" , long_options,NULL )) !=EOF ) {switch (c) {case 's' :
Signature · 1 line
2.14.2
PATCH 1/1 tools: env: allow to print U-Boot version
Quoted 90 lines · On Thu, 26 Oct 2017 14:57:53 +0200 Stefan Agner <[email protected]> wrote:
From: Stefan Agner <[email protected]>
The
fw_envutility family has a default environment compiled in which ties it quite strongly to the U-Boot source/config it has been built with. Allow to display the U-Boot version it has been built with using the -v/--version argument.Signed-off-by Stefan Agner <[email protected]>---
tools/env/fw_env_main.c | 14 +++++++++++--- 1 file changed, 11 insertions(+), 3 deletions(-)tools/env/fw_env_main.c +3
-0@@ -34,6 +34,7 @@#include <stdlib.h> #include <sys/file.h> #include <unistd.h> +#include <version.h> #include "fw_env_private.h" #include "fw_env.h" @@ -48,6 +49,7 @@ static struct option long_options[] = { {"script", required_argument, NULL, 's'}, {"noheader", required_argument, NULL, 'n'}, {"lock", required_argument, NULL, 'l'}, + {"version", no_argument, NULL, 'v'}, {NULL, 0, NULL, 0} };@@ -67,6 +69,7 @@ void usage_printenv(void)"Print variables from U-Boot environment\n" "\n" " -h, --help print this help.\n" +" -v, --version display version\n" #ifdef CONFIG_ENV_AES " -a, --aes aes key to accessenvironment\n" #endif
tools/env/fw_env_main.c +1
-0@@ -85,6 +88,7 @@ void usage_env_set(void)"Modify variables in U-Boot environment\n" "\n" " -h, --help print this help.\n" +" -v, --version display version\n" #ifdef CONFIG_ENV_AES " -a, --aes aes key to accessenvironment\n" #endif
tools/env/fw_env_main.c +0
-1@@ -123,7 +127,7 @@ static void parse_common_args(int argc, char *argv[]) env_opts.config_file = CONFIG_FILE;#endif - while ((c = getopt_long(argc, argv, ":a:c:l:h",
long_options, NULL)) !=tools/env/fw_env_main.c +1
-0+ while ((c = getopt_long(argc, argv, ":a:c:l:h:v",
long_options, NULL)) != EOF) {tools/env/fw_env_main.c +4
-0switch (c) {case 'a' :@@ -145,6 +149,10 @@ static void parse_common_args(int argc, char *argv[]) do_printenv ? usage_printenv() : usage_env_set();exit (EXIT_SUCCESS );break ; +case 'v' : +fprintf (stderr,"Compiled with " U_BOOT_VERSION "\n" ); +exit (EXIT_SUCCESS ); +break ;default :/* ignore unknown options */ tools/env/fw_env_main.c +1
-1break ;@@ -162,7 +170,7 @@ int parse_printenv_args(int argc, char *argv[])parse_common_args (argc, argv); -while ((c =getopt_long (argc, argv,"a:c:ns:l:h" , long_options, NULL)) +while ((c =getopt_long (argc, argv,"a:c:ns:l:h:v" , long_options,NULL )) !=EOF ) {switch (c) {tools/env/fw_env_main.c +1
-1case 'n' :@@ -189,7 +197,7 @@ int parse_setenv_args(int argc, char *argv[])parse_common_args (argc, argv); -while ((c =getopt_long (argc, argv,"a:c:ns:l:h" , long_options, NULL)) +while ((c =getopt_long (argc, argv,"a:c:ns:l:h:v" , long_options,NULL )) !=EOF ) {switch (c) {
Quoted 1 line
case 's':
Best regards,
Lukasz Majewski
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Quoted 9 lines · Stefan Agner
From: Stefan Agner <[email protected]>
The
fw_envutility family has a default environment compiled in which ties it quite strongly to the U-Boot source/config it has been built with. Allow to display the U-Boot version it has been built with using the -v/--version argument.Signed-off-by Stefan Agner <[email protected]>Reviewed-by Lukasz Majewski <[email protected]>
Applied to u-boot/master, thanks!
Signature · 8 lines
Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: not available URL: <http://lists.denx.de/pipermail/u-boot/attachments/20171106/bd94378c/attachment.sig>