All of lore.kernel.org
 help / color / mirror / Atom feed
From: Luca Ceresoli <luca@lucaceresoli.net>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 4/4] toolchain/wrapper: add option to print one argument per line
Date: Thu, 05 Sep 2013 14:48:55 +0200	[thread overview]
Message-ID: <52287DB7.6080008@lucaceresoli.net> (raw)
In-Reply-To: <e1d03e22b10b073d5906a4b6df22c72d38bfd8c2.1378330371.git.yann.morin.1998@free.fr>

Yann E. MORIN wrote:
> From: "Yann E. MORIN" <yann.morin.1998@free.fr>
>
> In case there are many arguments passed to the tools, the command line
> can get very long, and difficult to parse visually.
>
> For example, the Linux kernel passes a lot of arguments to gcc (at least
> 45, which gives 53 with our hard-coded args). Looking at such a command
> line is daunting.
>
> So, add the possibility to print each argument on its own line.
>
> Also, enclose all args between single quotes, so the command line
> can be safely copy-pasted without special chars (spaces, $) being
> inrerpreted by the shell.
>
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>

This feature should be mentioned in the docs.

Apart for that:

Acked-by: Luca Ceresoli <luca@lucaceresoli.net>
Tested-by: Luca Ceresoli <luca@lucaceresoli.net>
[Tested on arm_foundationv8_defconfig]

Luca

  reply	other threads:[~2013-09-05 12:48 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-09-04 21:35 [Buildroot] [pull request] Pull request for branch yem/misc-updates Yann E. MORIN
2013-09-04 21:35 ` [Buildroot] [PATCH 1/4] package/rpi-firmware: bump version Yann E. MORIN
2013-09-05 13:42   ` Peter Korsgaard
2013-09-04 21:35 ` [Buildroot] [PATCH 2/4] configs: bump kernel use for RPi, use simpler defconfig Yann E. MORIN
2013-09-05 13:43   ` Peter Korsgaard
2013-09-04 21:35 ` [Buildroot] [PATCH 3/4] targets: move target options to their own sub-menu Yann E. MORIN
2013-09-05  2:25   ` Thomas De Schampheleire
2013-09-05 13:51     ` Yann E. MORIN
2013-09-05 13:44   ` Peter Korsgaard
2013-09-04 21:35 ` [Buildroot] [PATCH 4/4] toolchain/wrapper: add option to print one argument per line Yann E. MORIN
2013-09-05 12:48   ` Luca Ceresoli [this message]
2013-09-05 13:55     ` Yann E. MORIN

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=52287DB7.6080008@lucaceresoli.net \
    --to=luca@lucaceresoli.net \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.