From: Arnout Vandecappelle <arnout@mind.be>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 09/25 v6] docs/manual: do not hardcode name of the generated document
Date: Fri, 10 Oct 2014 00:22:01 +0200 [thread overview]
Message-ID: <54370A89.5070109@mind.be> (raw)
In-Reply-To: <20140929202629.GC6211@free.fr>
On 29/09/14 22:26, Yann E. MORIN wrote:
> Thomas, All,
[snip]
>>> But it does not really matter: we can quote messages, now. Or not.
>>
>> True. I don't understand though why we wouldn't
>> change all unnecessary quoting in the quest for symmetry and simplicity.
>
> If at all, we should change to always quote messages.
>
> The problem is two-fold:
>
> - no quoting: we can not have a comma in the message, otherwise make
> would split on it: $(call MESSAGE,This, and that, should foo)
"-quotes are completely transparent for make, so quoting doesn't help for
commas. You need $(call MESSAGE,This$(comma) and that$(comma) should foo).
Regards,
Arnout
> - always quoting: the MESSAGE definition already does the quoting to
> ensure it does not break on a semi-colon (which you noticed):
> $(call MESSAGE,"; echo shellshock")
>
> (He, shellshock is all the hype nowadays. Even superseeds heartbleed!)
>
> If MESSAGE would not do the quoting, and rely on the caller to do it,
> we'd be vulnerable to: $(call MESSAGE,; echo shellshock)
>
> So: we want MESSAGE to always quote its arguments (and no, single quotes
> do not cut it either, because a message might have single quotes too),
> while still allowing callers to do the quoting. Or not.
>
> Regards,
> Yann E. MORIN.
>
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286500
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
next prev parent reply other threads:[~2014-10-09 22:22 UTC|newest]
Thread overview: 55+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-09-27 10:16 [Buildroot] [pull request v6] gendoc infra: make it useable for other documents Yann E. MORIN
2014-09-27 10:16 ` [Buildroot] [PATCH 01/25 v6] gendoc infra: move manual build location into $(BUILD_DIR)/manual Yann E. MORIN
2014-09-27 10:16 ` [Buildroot] [PATCH 02/25 v6] gendoc infra: avoid a2x warning Yann E. MORIN
2014-09-27 10:16 ` [Buildroot] [PATCH 03/25 v6] gendoc infra: disable pdf manual generation if xsltproc is buggy Yann E. MORIN
2014-09-27 10:16 ` [Buildroot] [PATCH 04/25 v6] docs/manual: get rid of legacy comment in GENDOC Yann E. MORIN
2014-09-27 10:16 ` [Buildroot] [PATCH 05/25 v6] docs/manual: copying the manual sources is a common action Yann E. MORIN
2014-09-27 10:16 ` [Buildroot] [PATCH 06/25 v6] docs/manual: simplify generation dependencies Yann E. MORIN
2014-09-27 10:16 ` [Buildroot] [PATCH 07/25 v6] docs/manual: manual-update-lists is not a generic GENDOC rule Yann E. MORIN
2014-09-27 10:16 ` [Buildroot] [PATCH 08/25 v6] docs/manual: move manual-prepare-sources rule into GENDOC Yann E. MORIN
2014-09-27 10:16 ` [Buildroot] [PATCH 09/25 v6] docs/manual: do not hardcode name of the generated document Yann E. MORIN
2014-09-29 19:33 ` Thomas De Schampheleire
2014-09-29 20:01 ` Yann E. MORIN
2014-09-29 20:09 ` Thomas De Schampheleire
2014-09-29 20:26 ` Yann E. MORIN
2014-10-09 22:22 ` Arnout Vandecappelle [this message]
2014-09-27 10:16 ` [Buildroot] [PATCH 10/25 v6] docs/manual: allow a document to declare where its resources are Yann E. MORIN
2014-09-27 11:20 ` Samuel Martin
2014-09-29 19:35 ` Thomas De Schampheleire
2014-09-27 10:16 ` [Buildroot] [PATCH 11/25 v6] docs/manual: last pass at removing hard-coded path in GENDOC_INNER Yann E. MORIN
2014-09-27 11:20 ` Samuel Martin
2014-09-27 10:16 ` [Buildroot] [PATCH 12/25 v6] docs/manual: properly separate rules specific for our manual Yann E. MORIN
2014-09-27 11:20 ` Samuel Martin
2014-09-27 10:16 ` [Buildroot] [PATCH 13/25 v6] docs/manual: rename the generic dependency rules Yann E. MORIN
2014-09-27 10:16 ` [Buildroot] [PATCH 14/25 v6] docs/manual: allow documents to define some hooks, as for a package Yann E. MORIN
2014-10-01 19:58 ` Thomas De Schampheleire
2014-09-27 10:16 ` [Buildroot] [PATCH 15/25 v6] docs/manual: use the new hooks instead of gendoc rules Yann E. MORIN
2014-10-01 20:00 ` Thomas De Schampheleire
2014-09-27 10:16 ` [Buildroot] [PATCH 16/25 v6] docs/manual: always look for resources in destination directory Yann E. MORIN
2014-10-01 20:01 ` Thomas De Schampheleire
2014-09-27 10:16 ` [Buildroot] [PATCH 17/25 v6] docs: separate the GENDOC infra from our manual definition Yann E. MORIN
2014-09-27 11:20 ` Samuel Martin
2014-09-27 10:16 ` [Buildroot] [PATCH 18/25 v6] docs: rename the GENDOC infrastructure Yann E. MORIN
2014-10-01 20:02 ` Thomas De Schampheleire
2014-09-27 10:16 ` [Buildroot] [PATCH 19/25 v6] docs/asciidoc: call $(pkgname) and $(pkgdir) in a single place Yann E. MORIN
2014-10-01 20:03 ` Thomas De Schampheleire
2014-09-27 10:16 ` [Buildroot] [PATCH 20/25 v6] docs/asciidoc: make it possible to use $(@D) in hooks Yann E. MORIN
2014-10-02 9:45 ` Thomas De Schampheleire
2014-09-27 10:16 ` [Buildroot] [PATCH 21/25 v6] doc/asciidoc: allow documents to pass a global asciidoc configuration Yann E. MORIN
2014-09-27 11:20 ` Samuel Martin
2014-09-27 12:51 ` Yann E. MORIN
2014-10-02 9:50 ` Thomas De Schampheleire
2014-09-27 10:16 ` [Buildroot] [PATCH 22/25 v6] doc/asciidoc: always apply Buildroot's AsciiDoc config Yann E. MORIN
2014-09-27 11:21 ` Samuel Martin
2014-09-27 12:56 ` Yann E. MORIN
2014-10-02 10:17 ` Thomas De Schampheleire
2014-10-02 21:46 ` Yann E. MORIN
2014-09-27 10:16 ` [Buildroot] [PATCH 23/25 v6] doc/asciidoc: always use UTC Yann E. MORIN
2014-10-02 10:21 ` Thomas De Schampheleire
2014-10-02 19:14 ` Yann E. MORIN
2014-09-27 10:17 ` [Buildroot] [PATCH 24/25 v6] Makefile: enable generating documents from br2-external Yann E. MORIN
2014-09-27 10:17 ` [Buildroot] [PATCH 25/25 v6] docs/manual: document the asciidoc infra Yann E. MORIN
2014-09-27 11:21 ` Samuel Martin
2014-09-27 12:40 ` Yann E. MORIN
2014-10-02 10:27 ` Thomas De Schampheleire
2014-10-02 19:20 ` 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=54370A89.5070109@mind.be \
--to=arnout@mind.be \
--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