Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Yann E. MORIN <yann.morin.1998@free.fr>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 7/9] manual: Remove dollar before command example
Date: Wed, 19 Nov 2014 23:29:57 +0100	[thread overview]
Message-ID: <20141119222957.GI3779@free.fr> (raw)
In-Reply-To: <1416068964-22529-7-git-send-email-maxime.hadjinlian@gmail.com>

Maxime, All,

On 2014-11-15 17:29 +0100, Maxime Hadjinlian spake thusly:
> In order to harmonize the manual, in some places there was a dollar
> before and other place did not have that dollar.
> 
> Made the choice to remove the dollar in order to facilitate copy/paste
> from the manual.

Well, I actually prefer when there is a dollar. It makes it rather
obvious this is a comman to type, as opposed to the output of a
command. For us long-bearded old-timers, it is just obvious. ;-)

That's a standard practice, used for example in (POSIX-related?) man
pages.

I agree that we mst be consistent throughtout the manual, but I'd prefer
we choose the other consistency. ;-)

However, I won't be blindly supporting my position, so in case the
maintainers prefer without $, this still gets my:
-- Reviewed-by: "Yann E. MORIN" <yann.morin.1998@free.fr>

Regards,
Yann E. MORIN.

> Signed-off-by: Maxime Hadjinlian <maxime.hadjinlian@gmail.com>
> ---
>  docs/manual/common-usage.txt        | 12 ++++++------
>  docs/manual/make-tips.txt           | 12 ++++++------
>  docs/manual/quickstart.txt          | 10 +++++-----
>  docs/manual/rebuilding-packages.txt |  2 +-
>  4 files changed, 18 insertions(+), 18 deletions(-)
> 
> diff --git a/docs/manual/common-usage.txt b/docs/manual/common-usage.txt
> index 89cd9fe..6ce97f1 100644
> --- a/docs/manual/common-usage.txt
> +++ b/docs/manual/common-usage.txt
> @@ -14,7 +14,7 @@ all sources that you previously selected in the configurator
>  ('menuconfig', 'nconfig', 'xconfig' or 'gconfig'), then issue:
>  
>  --------------------
> - $ make source
> +make source
>  --------------------
>  
>  You can now disconnect or copy the content of your +dl+
> @@ -30,13 +30,13 @@ the Linux kernel. To use it, add +O=<directory>+ to the make command
>  line:
>  
>  --------------------
> - $ make O=/tmp/build
> +make O=/tmp/build
>  --------------------
>  
>  Or:
>  
>  --------------------
> - $ cd /tmp/build; make O=$PWD -C path/to/buildroot
> +cd /tmp/build; make O=$PWD -C path/to/buildroot
>  --------------------
>  
>  All the output files will be located under +/tmp/build+. If the +O+
> @@ -57,7 +57,7 @@ directory - so after the first run, you no longer need to pass +O=<...>+
>  and +-C <...>+, simply run (in the output directory):
>  
>  --------------------
> - $ make <target>
> +make <target>
>  --------------------
>  
>  [[env-vars]]
> @@ -103,14 +103,14 @@ An example that uses config files located in the toplevel directory and
>  in your $HOME:
>  
>  --------------------
> - $ make UCLIBC_CONFIG_FILE=uClibc.config BUSYBOX_CONFIG_FILE=$HOME/bb.config
> +make UCLIBC_CONFIG_FILE=uClibc.config BUSYBOX_CONFIG_FILE=$HOME/bb.config
>  --------------------
>  
>  If you want to use a compiler other than the default +gcc+
>  or +g+++ for building helper-binaries on your host, then do
>  
>  --------------------
> - $ make HOSTCXX=g++-4.3-HEAD HOSTCC=gcc-4.3-HEAD
> +make HOSTCXX=g++-4.3-HEAD HOSTCC=gcc-4.3-HEAD
>  --------------------
>  
>  === Dealing efficiently with filesystem images
> diff --git a/docs/manual/make-tips.txt b/docs/manual/make-tips.txt
> index 027eddf..f078eea 100644
> --- a/docs/manual/make-tips.txt
> +++ b/docs/manual/make-tips.txt
> @@ -9,13 +9,13 @@ This is a collection of tips that help you make the most of Buildroot.
>  .Display all commands executed by make:
>  
>  --------------------
> - $ make V=1 <target>
> +make V=1 <target>
>  --------------------
>  
>  .Display all available targets:
>  
>  --------------------
> - $ make help
> +make help
>  --------------------
>  
>  Not all targets are always available,
> @@ -38,7 +38,7 @@ To delete all build products (including build directories, host, staging
>  and target trees, the images and the toolchain):
>  
>  --------------------
> - $ make clean
> +make clean
>  --------------------
>  
>  .Generating the manual:
> @@ -47,8 +47,8 @@ The present manual sources are located in the 'docs/manual' directory.
>  To generate the manual:
>  
>  ---------------------------------
> - $ make manual-clean
> - $ make manual
> +make manual-clean
> +make manual
>  ---------------------------------
>  
>  The manual outputs will be generated in 'output/docs/manual'.
> @@ -62,7 +62,7 @@ The manual outputs will be generated in 'output/docs/manual'.
>  To delete all build products as well as the configuration:
>  
>  --------------------
> - $ make distclean
> +make distclean
>  --------------------
>  
>  .Notes
> diff --git a/docs/manual/quickstart.txt b/docs/manual/quickstart.txt
> index 975fffc..24b18de 100644
> --- a/docs/manual/quickstart.txt
> +++ b/docs/manual/quickstart.txt
> @@ -16,25 +16,25 @@ http://www.busybox.net/[BusyBox].
>  From the buildroot directory, run
>  
>  --------------------
> - $ make menuconfig
> +make menuconfig
>  --------------------
>  
>  for the original curses-based configurator, or
>  
>  --------------------
> - $ make nconfig
> +make nconfig
>  --------------------
>  
>  for the new curses-based configurator, or
>  
>  --------------------
> - $ make xconfig
> +make xconfig
>  --------------------
>  
>  for the Qt-based configurator, or
>  
>  --------------------
> - $ make gconfig
> +make gconfig
>  --------------------
>  
>  for the GTK-based configurator.
> @@ -57,7 +57,7 @@ read by the top-level Makefile.
>  To start the build process, simply run:
>  
>  --------------------
> - $ make
> +make
>  --------------------
>  
>  You *should never* use +make -jN+ with Buildroot: top-level parallel
> diff --git a/docs/manual/rebuilding-packages.txt b/docs/manual/rebuilding-packages.txt
> index 6faa67a..00423e4 100644
> --- a/docs/manual/rebuilding-packages.txt
> +++ b/docs/manual/rebuilding-packages.txt
> @@ -77,7 +77,7 @@ necessary, and you will save more and more time.
>  For reference, a full rebuild is achieved by running:
>  
>  ---------------
> -$ make clean all
> +make clean all
>  ---------------
>  
>  [[rebuild-pkg]]
> -- 
> 2.1.3
> 
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

  reply	other threads:[~2014-11-19 22:29 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-15 16:29 [Buildroot] [PATCH 1/9] package/*/*.mk: Fix indent Maxime Hadjinlian
2014-11-15 16:29 ` [Buildroot] [PATCH 2/9] makedevs: Rework README Maxime Hadjinlian
2014-11-19 22:05   ` Yann E. MORIN
2014-11-24  9:34     ` Maxime Hadjinlian
2014-11-15 16:29 ` [Buildroot] [PATCH 3/9] manual: Rework makedevs syntax Maxime Hadjinlian
2014-11-19 22:14   ` Yann E. MORIN
2014-11-22 19:15   ` Thomas Petazzoni
2014-11-15 16:29 ` [Buildroot] [PATCH 4/9] manual: Remove trailing space Maxime Hadjinlian
2014-11-19 22:15   ` Yann E. MORIN
2014-11-22 19:15   ` Thomas Petazzoni
2014-11-15 16:29 ` [Buildroot] [PATCH 5/9] manual: Add missing vim header Maxime Hadjinlian
2014-11-19 22:17   ` Yann E. MORIN
2014-11-22 19:15   ` Thomas Petazzoni
2014-11-15 16:29 ` [Buildroot] [PATCH 6/9] manual: Add newline before block code Maxime Hadjinlian
2014-11-19 22:21   ` Yann E. MORIN
2014-11-22 19:15   ` Thomas Petazzoni
2014-11-15 16:29 ` [Buildroot] [PATCH 7/9] manual: Remove dollar before command example Maxime Hadjinlian
2014-11-19 22:29   ` Yann E. MORIN [this message]
2014-11-22 19:16   ` Thomas Petazzoni
2014-11-24  8:14     ` Maxime Hadjinlian
2014-11-15 16:29 ` [Buildroot] [PATCH 8/9] manual: Harmonize " Maxime Hadjinlian
2014-11-19 22:32   ` Yann E. MORIN
2014-11-15 16:29 ` [Buildroot] [PATCH 9/9] manual: Block code needs only 4 dashes Maxime Hadjinlian
2014-11-19 22:37   ` Yann E. MORIN
2014-11-22 19:16   ` Thomas Petazzoni
2014-11-19 22:10 ` [Buildroot] [PATCH 1/9] package/*/*.mk: Fix indent Yann E. MORIN
2014-11-19 22:25 ` Thomas Petazzoni

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=20141119222957.GI3779@free.fr \
    --to=yann.morin.1998@free.fr \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox