[Buildroot] [PATCH 14/51] core/legal-info: update the legal-info report header

Luca Ceresoli luca at lucaceresoli.net
Thu Dec 3 17:21:30 UTC 2015


Dear Yann,

thanks for having updated this forgotten file! See my comments below.

Yann E. MORIN wrote:
> In some cases, the toolchain sources are now recovered and available in
> the legal-info output. So, adapt the header to use conditional instead
> of an definitive negation.
>
> Also update the part about saving the sources: it's not the license list
> that defines whether sources are installed, but rather whether the
> package is reditributable or not. Update the header acoordingly.
>
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998 at free.fr>
> Cc: Luca Ceresoli <luca at lucaceresoli.net>
> ---
>   support/legal-info/README.header | 6 +++---
>   1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/support/legal-info/README.header b/support/legal-info/README.header
> index 90fc445..08aa74d 100644
> --- a/support/legal-info/README.header
> +++ b/support/legal-info/README.header
> @@ -12,11 +12,11 @@ This material is composed of the following items.
>    * The Buildroot configuration file; this has been saved in buildroot.config.
>    * The toolchain (cross-compiler and related tools) used to generate all the
>      compiled programs.
> -   Note: this has not been saved due to technical limitations, you must
> +   Note: this may not been saved due to technical limitations, you may need to

"may not been saved" -> "may have not been saved" I guess.

With that fixed:
  Acked-by: Luca Ceresoli <luca at lucaceresoli.net>

Since you're updating the README about features that are already in
master, this patch could be applied immediately. Except it probably
conflicts, so you might want to rework it and move it at the beginning
of your patchset (or submit independently) if you want it to be merged
more quickly.

-- 
Luca


More information about the buildroot mailing list