diff mbox series

toolchain/wrapper: also dump args it was called with

Message ID 20190817124817.8859-1-yann.morin.1998@free.fr
State Accepted
Headers show
Series toolchain/wrapper: also dump args it was called with | expand

Commit Message

Yann E. MORIN Aug. 17, 2019, 12:48 p.m. UTC
Signed-off-by: Yann E. MORIN <yann.morin.1998@free.fr>
Cc: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
Cc: Thomas De Schampheleire <patrickdepinguin@gmail.com>
---
 toolchain/toolchain-wrapper.c | 49 +++++++++++++++++++++--------------
 1 file changed, 29 insertions(+), 20 deletions(-)

Comments

Thomas De Schampheleire Aug. 17, 2019, 6:25 p.m. UTC | #1
El sáb., 17 ago. 2019 a las 14:48, Yann E. MORIN
(<yann.morin.1998@free.fr>) escribió:
>
> Signed-off-by: Yann E. MORIN <yann.morin.1998@free.fr>
> Cc: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
> Cc: Thomas De Schampheleire <patrickdepinguin@gmail.com>
> ---
>  toolchain/toolchain-wrapper.c | 49 +++++++++++++++++++++--------------
>  1 file changed, 29 insertions(+), 20 deletions(-)
>
> diff --git a/toolchain/toolchain-wrapper.c b/toolchain/toolchain-wrapper.c
> index 7a4b9c4007..b7705a4cbf 100644
> --- a/toolchain/toolchain-wrapper.c
> +++ b/toolchain/toolchain-wrapper.c
> @@ -240,7 +240,24 @@ int main(int argc, char **argv)
>         char *env_debug;
>         char *paranoid_wrapper;
>         int paranoid;
> -       int ret, i, count = 0, debug, found_shared = 0;
> +       int ret, i, count = 0, debug = 0, found_shared = 0;
> +
> +       /* Debug the wrapper to see arguments it was called with.
> +        * If environment variable BR2_DEBUG_WRAPPER is:
> +        * unset, empty, or 0: do not trace
> +        * set to 1          : trace all arguments on a single line
> +        * set to 2          : trace one argument per line
> +        */
> +       if ((env_debug = getenv("BR2_DEBUG_WRAPPER"))) {
> +               debug = atoi(env_debug);
> +       }
> +       if (debug > 0) {
> +               fprintf(stderr, "Toolchain wrapper was called with:");
> +               for (i = 0; i < argc; i++)
> +                       fprintf(stderr, "%s'%s'",
> +                               (debug == 2) ? "\n    " : " ", argv[i]);
> +               fprintf(stderr, "\n");
> +       }
>
>         /* Calculate the relative paths */
>         basename = strrchr(progpath, '/');
> @@ -483,29 +500,21 @@ int main(int argc, char **argv)
>                 exec_args++;
>  #endif
>
> -       /* Debug the wrapper to see actual arguments passed to
> -        * the compiler:
> -        * unset, empty, or 0: do not trace
> -        * set to 1          : trace all arguments on a single line
> -        * set to 2          : trace one argument per line
> -        */
> -       if ((env_debug = getenv("BR2_DEBUG_WRAPPER"))) {
> -               debug = atoi(env_debug);
> -               if (debug > 0) {
> -                       fprintf(stderr, "Toolchain wrapper executing:");
> +       /* Debug the wrapper to see final arguments passed to the real compiler. */
> +       if (debug > 0) {
> +               fprintf(stderr, "Toolchain wrapper executing:");
>  #ifdef BR_CCACHE_HASH
> -                       fprintf(stderr, "%sCCACHE_COMPILERCHECK='string:" BR_CCACHE_HASH "'",
> -                               (debug == 2) ? "\n    " : " ");
> +               fprintf(stderr, "%sCCACHE_COMPILERCHECK='string:" BR_CCACHE_HASH "'",
> +                       (debug == 2) ? "\n    " : " ");
>  #endif
>  #ifdef BR_CCACHE_BASEDIR
> -                       fprintf(stderr, "%sCCACHE_BASEDIR='" BR_CCACHE_BASEDIR "'",
> -                               (debug == 2) ? "\n    " : " ");
> +               fprintf(stderr, "%sCCACHE_BASEDIR='" BR_CCACHE_BASEDIR "'",
> +                       (debug == 2) ? "\n    " : " ");
>  #endif
> -                       for (i = 0; exec_args[i]; i++)
> -                               fprintf(stderr, "%s'%s'",
> -                                       (debug == 2) ? "\n    " : " ", exec_args[i]);
> -                       fprintf(stderr, "\n");
> -               }
> +               for (i = 0; exec_args[i]; i++)
> +                       fprintf(stderr, "%s'%s'",
> +                               (debug == 2) ? "\n    " : " ", exec_args[i]);
> +               fprintf(stderr, "\n");
>         }
>
>  #ifdef BR_CCACHE_HASH

Reviewed-by: Thomas De Schampheleire <thomas.de_schampheleire@nokia.com>
Tested-by: Thomas De Schampheleire <thomas.de_schampheleire@nokia.com>
Thomas Petazzoni Aug. 17, 2019, 10:20 p.m. UTC | #2
On Sat, 17 Aug 2019 14:48:17 +0200
"Yann E. MORIN" <yann.morin.1998@free.fr> wrote:

> Signed-off-by: Yann E. MORIN <yann.morin.1998@free.fr>
> Cc: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
> Cc: Thomas De Schampheleire <patrickdepinguin@gmail.com>
> ---
>  toolchain/toolchain-wrapper.c | 49 +++++++++++++++++++++--------------
>  1 file changed, 29 insertions(+), 20 deletions(-)

Applied to next, thanks.

Thomas
diff mbox series

Patch

diff --git a/toolchain/toolchain-wrapper.c b/toolchain/toolchain-wrapper.c
index 7a4b9c4007..b7705a4cbf 100644
--- a/toolchain/toolchain-wrapper.c
+++ b/toolchain/toolchain-wrapper.c
@@ -240,7 +240,24 @@  int main(int argc, char **argv)
 	char *env_debug;
 	char *paranoid_wrapper;
 	int paranoid;
-	int ret, i, count = 0, debug, found_shared = 0;
+	int ret, i, count = 0, debug = 0, found_shared = 0;
+
+	/* Debug the wrapper to see arguments it was called with.
+	 * If environment variable BR2_DEBUG_WRAPPER is:
+	 * unset, empty, or 0: do not trace
+	 * set to 1          : trace all arguments on a single line
+	 * set to 2          : trace one argument per line
+	 */
+	if ((env_debug = getenv("BR2_DEBUG_WRAPPER"))) {
+		debug = atoi(env_debug);
+	}
+	if (debug > 0) {
+		fprintf(stderr, "Toolchain wrapper was called with:");
+		for (i = 0; i < argc; i++)
+			fprintf(stderr, "%s'%s'",
+				(debug == 2) ? "\n    " : " ", argv[i]);
+		fprintf(stderr, "\n");
+	}
 
 	/* Calculate the relative paths */
 	basename = strrchr(progpath, '/');
@@ -483,29 +500,21 @@  int main(int argc, char **argv)
 		exec_args++;
 #endif
 
-	/* Debug the wrapper to see actual arguments passed to
-	 * the compiler:
-	 * unset, empty, or 0: do not trace
-	 * set to 1          : trace all arguments on a single line
-	 * set to 2          : trace one argument per line
-	 */
-	if ((env_debug = getenv("BR2_DEBUG_WRAPPER"))) {
-		debug = atoi(env_debug);
-		if (debug > 0) {
-			fprintf(stderr, "Toolchain wrapper executing:");
+	/* Debug the wrapper to see final arguments passed to the real compiler. */
+	if (debug > 0) {
+		fprintf(stderr, "Toolchain wrapper executing:");
 #ifdef BR_CCACHE_HASH
-			fprintf(stderr, "%sCCACHE_COMPILERCHECK='string:" BR_CCACHE_HASH "'",
-				(debug == 2) ? "\n    " : " ");
+		fprintf(stderr, "%sCCACHE_COMPILERCHECK='string:" BR_CCACHE_HASH "'",
+			(debug == 2) ? "\n    " : " ");
 #endif
 #ifdef BR_CCACHE_BASEDIR
-			fprintf(stderr, "%sCCACHE_BASEDIR='" BR_CCACHE_BASEDIR "'",
-				(debug == 2) ? "\n    " : " ");
+		fprintf(stderr, "%sCCACHE_BASEDIR='" BR_CCACHE_BASEDIR "'",
+			(debug == 2) ? "\n    " : " ");
 #endif
-			for (i = 0; exec_args[i]; i++)
-				fprintf(stderr, "%s'%s'",
-					(debug == 2) ? "\n    " : " ", exec_args[i]);
-			fprintf(stderr, "\n");
-		}
+		for (i = 0; exec_args[i]; i++)
+			fprintf(stderr, "%s'%s'",
+				(debug == 2) ? "\n    " : " ", exec_args[i]);
+		fprintf(stderr, "\n");
 	}
 
 #ifdef BR_CCACHE_HASH