[Buildroot] [PATCH] manual: clarify that Config.in comments should have correct dependencies
s.martin49 at gmail.com
Mon Nov 4 21:26:11 UTC 2013
2013/11/4 Thomas De Schampheleire <patrickdepinguin at gmail.com>
> Signed-off-by: Thomas De Schampheleire <thomas.de.schampheleire at gmail.com>
> Note: this documentation patch accompanies my earlier patch of today to add
> missing dependencies on such comments . These two should have been sent
> as one series, sorry for that.
>  http://patchwork.ozlabs.org/patch/288125/
> docs/manual/adding-packages-directory.txt | 9 +++++++++
> 1 files changed, 9 insertions(+), 0 deletions(-)
> diff --git a/docs/manual/adding-packages-directory.txt
> --- a/docs/manual/adding-packages-directory.txt
> +++ b/docs/manual/adding-packages-directory.txt
> @@ -171,6 +171,7 @@ C library, C++ support, largefile suppor
> IPv6 support, wchar support, or dynamic library support. Some packages
> can only be built on certain target architectures, or if an MMU is
> available in the processor.
> These dependencies have to expressed with the appropriate 'depends on'
s/have to expressed/have to be expressed/
statements in the Config.in file. Additionally, for dependencies on
> toolchain options, a +comment+ should be displayed when the option is
> @@ -180,6 +181,14 @@ made visible in a comment: since it is u
> freely choose another target, it makes little sense to show these
> dependencies explicitly.
> +The +comment+ should only be visible if the +config+ option itself would
> +be visible when the toolchain option dependencies are met. This means
> +that all other dependencies of the package (including dependencies on
> +target architecture and MMU support) have to be repeated on the
> ++comment+ definition. To keep it clear, the +depends on+ statement for
> +these non-toolchain option should be kept separate from the +depends on+
> +statement for the toolchain options.
> The general format of a dependency +comment+ for package foo is:
> foo needs a toolchain w/ featA, featB, featC
> buildroot mailing list
> buildroot at busybox.net
Acked-by: Samuel Martin <s.martin49 at gmail.com>
(with this little fix ;))
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the buildroot