This is an automated email from the ASF dual-hosted git repository. davids5 pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git
commit 1653b295425df2a6cf92b30b8e97710eb1920611 Author: Xiang Xiao <xiaoxi...@xiaomi.com> AuthorDate: Fri Mar 20 01:35:04 2020 +0800 Revert "tools/configure.*: Add -s option to skip .config/Make.defs existence check." This reverts commit bf3ffddf2f29e3eef4b0f5c2c7b3a872fb53ff6c. Change-Id: Ia858ca1aab79a6c599cf93d2701515951bdc7b80 --- tools/configure.c | 18 ++---------------- tools/configure.sh | 9 ++------- 2 files changed, 4 insertions(+), 23 deletions(-) diff --git a/tools/configure.c b/tools/configure.c index 5e75c12..1698ffd 100644 --- a/tools/configure.c +++ b/tools/configure.c @@ -127,7 +127,6 @@ static char g_delim = '/'; /* Delimiter to use when forming pat static bool g_winpaths = false; /* False: POSIX style paths */ #endif static bool g_debug = false; /* Enable debug output */ -static bool g_skip = false; /* Skip .config/Make.defs existence check */ static const char *g_appdir = NULL; /* Relative path to the application directory */ static const char *g_archdir = NULL; /* Name of architecture subdirectory */ @@ -174,13 +173,11 @@ static const char *g_optfiles[] = static void show_usage(const char *progname, int exitcode) { - fprintf(stderr, "\nUSAGE: %s [-d] [-s] [-b] [-f] [-l|m|c|u|g|n] [-a <app-dir>] <board-name>:<config-name>\n", progname); + fprintf(stderr, "\nUSAGE: %s [-d] [-b] [-f] [-l|m|c|u|g|n] [-a <app-dir>] <board-name>:<config-name>\n", progname); fprintf(stderr, "\nUSAGE: %s [-h]\n", progname); fprintf(stderr, "\nWhere:\n"); fprintf(stderr, " -d:\n"); fprintf(stderr, " Enables debug output\n"); - fprintf(stderr, " -s:\n"); - fprintf(stderr, " Skip the .config/Make.defs existence check\n"); fprintf(stderr, " -b:\n"); #ifdef CONFIG_WINDOWS_NATIVE fprintf(stderr, " Informs the tool that it should use Windows style paths like C:\\Program Files\n"); @@ -249,7 +246,7 @@ static void parse_args(int argc, char **argv) g_debug = false; - while ((ch = getopt(argc, argv, "a:bcdfghlmnsu")) > 0) + while ((ch = getopt(argc, argv, "a:bcdfghlmnu")) > 0) { switch (ch) { @@ -297,10 +294,6 @@ static void parse_args(int argc, char **argv) g_windows = WINDOWS_NATIVE; break; - case 's' : - g_skip = true; - break; - case 'u' : g_host = HOST_WINDOWS; g_windows = WINDOWS_UBUNTU; @@ -746,13 +739,6 @@ static void check_configdir(void) static void check_configured(void) { - /* Skip to check .config/Make.defs? */ - - if (g_skip) - { - return; - } - /* If we are already configured then there will be a .config and a Make.defs * file in the top-level directory. */ diff --git a/tools/configure.sh b/tools/configure.sh index 6455fc4..87ca985 100755 --- a/tools/configure.sh +++ b/tools/configure.sh @@ -37,11 +37,10 @@ WD=`test -d ${0%/*} && cd ${0%/*}; pwd` TOPDIR="${WD}/.." USAGE=" -USAGE: ${0} [-d] [-s] [-l|m|c|u|g|n] [-a <app-dir>] <board-name>:<config-name> +USAGE: ${0} [-d] [-l|m|c|u|g|n] [-a <app-dir>] <board-name>:<config-name> Where: -d enables script debug output - -s skip the .config/Make.defs existence check -l selects the Linux (l) host environment. -m selects the macOS (m) host environment. -c selects the Windows host and Cygwin (c) environment. @@ -72,7 +71,6 @@ unset winnative unset appdir unset host unset debug -skip=0 while [ ! -z "$1" ]; do case "$1" in @@ -96,9 +94,6 @@ while [ ! -z "$1" ]; do echo "$USAGE" exit 0 ;; - -s ) - skip=1 - ;; *) if [ ! -z "${boardconfig}" ]; then echo "" @@ -174,7 +169,7 @@ if [ ! -r ${src_config} ]; then exit 5 fi -if [ ${skip} != 1 ] && [ -r ${dest_config} ]; then +if [ -r ${dest_config} ]; then echo "Already configured!" echo "Do 'make distclean' and try again." exit 6