All of 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 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.