]> rtime.felk.cvut.cz Git - coffee/buildroot.git/commitdiff
docs/manual: size of tab in package description
authorRicardo Martincoski <ricardo.martincoski@gmail.com>
Sun, 19 Feb 2017 22:17:22 +0000 (19:17 -0300)
committerThomas Petazzoni <thomas.petazzoni@free-electrons.com>
Sat, 8 Apr 2017 14:15:36 +0000 (16:15 +0200)
Explicitly state that one tab counts for 8 columns in package
description, leaving 62 characters to the text itself.
Update the text and the example in the two places where the Config.in
format is described.
Also mention a newline is expected between the help text itself and the
upstream URL.

This blob can help developers to understand the expected formatting.
Also, it can be referenced by reviewers.

http://patchwork.ozlabs.org/patch/611289/
http://patchwork.ozlabs.org/patch/606866/
http://patchwork.ozlabs.org/patch/459960/

Signed-off-by: Ricardo Martincoski <ricardo.martincoski@gmail.com>
Cc: Arnout Vandecappelle <arnout@mind.be>
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Cc: Romain Naour <romain.naour@gmail.com>
Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
docs/manual/adding-packages-directory.txt
docs/manual/writing-rules.txt

index 5dba962bd45fa00103e760d0a0d893cc3a0f82f5..08f5d42f91bd3eca2a938272ef6bb38ae02819f3 100644 (file)
@@ -28,7 +28,8 @@ contain:
 config BR2_PACKAGE_LIBFOO
        bool "libfoo"
        help
 config BR2_PACKAGE_LIBFOO
        bool "libfoo"
        help
-         This is a comment that explains what libfoo is.
+         This is a comment that explains what libfoo is. The help text
+         should be wrapped.
 
          http://foosoftware.org/libfoo/
 ---------------------------
 
          http://foosoftware.org/libfoo/
 ---------------------------
@@ -36,8 +37,9 @@ config BR2_PACKAGE_LIBFOO
 The +bool+ line, +help+ line and other metadata information about the
 configuration option must be indented with one tab. The help text
 itself should be indented with one tab and two spaces, lines should
 The +bool+ line, +help+ line and other metadata information about the
 configuration option must be indented with one tab. The help text
 itself should be indented with one tab and two spaces, lines should
-not be longer than 72 columns, and it must mention the upstream URL
-of the project.
+be wrapped to fit 72 columns, where tab counts for 8, so 62 characters
+in the text itself. The help text must mention the upstream URL of the
+project after an empty line.
 
 As a convention specific to Buildroot, the ordering of the attributes
 is as follows:
 
 As a convention specific to Buildroot, the ordering of the attributes
 is as follows:
index ec1ddb1915caa6b337fe3325d7ca6898b726aa09..e2ad41ebc3e3da6581e36449ffb87c0d809e7555 100644 (file)
@@ -29,7 +29,8 @@ config BR2_PACKAGE_LIBFOO
        depends on BR2_PACKAGE_LIBBAZ
        select BR2_PACKAGE_LIBBAR
        help
        depends on BR2_PACKAGE_LIBBAZ
        select BR2_PACKAGE_LIBBAR
        help
-         This is a comment that explains what libfoo is.
+         This is a comment that explains what libfoo is. The help text
+         should be wrapped.
 
          http://foosoftware.org/libfoo/
 ---------------------
 
          http://foosoftware.org/libfoo/
 ---------------------
@@ -40,7 +41,8 @@ config BR2_PACKAGE_LIBFOO
 * The help text itself should be indented with one tab and two
   spaces.
 
 * The help text itself should be indented with one tab and two
   spaces.
 
-* The help text should be wrapped to fit 72 columns.
+* The help text should be wrapped to fit 72 columns, where tab counts
+  for 8, so 62 characters in the text itself.
 
 The +Config.in+ files are the input for the configuration tool
 used in Buildroot, which is the regular _Kconfig_. For further
 
 The +Config.in+ files are the input for the configuration tool
 used in Buildroot, which is the regular _Kconfig_. For further