[Buildroot] [PATCH 05/13 v7] core/legal-info: add package version to license directory
Yann E. MORIN
yann.morin.1998 at free.fr
Fri May 13 20:11:35 UTC 2016
Thomas, All,
On 2016-05-11 23:43 +0200, Thomas Petazzoni spake thusly:
> On Sat, 7 May 2016 22:01:28 +0200, Arnout Vandecappelle wrote:
>
> > > + @$$(call legal-warning-pkg,$$($(2)_RAW_BASE_NAME),cannot save license ($(2)_LICENSE_FILES not defined))
> >
> > I think this should stay $(2)_RAWNAME, or either it should change as well for
> > the legal-warning-source below.
> >
> > Other than that:
> > Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout at mind.be>
>
> I agree with Arnout here (we should keep using $(2)_RAWNAME for the
> warning). Yann, what do you think?
1. With this patch (and the following one), legal-info now gets saved in
sub-directories named after the paclage name and version, i.e.:
legal-info/licenses/pkg-version/
legal-info/sources/pkg-version/
2. With this patch, we also store the package name and version in the
manifest.
3. The warnings are stored in the legal-info report.
So, I think it is better that the info about packages is the same
everywhere: sources/ and licenses/ sub-dirs, entries in the manifest and
warnings alike.
Regards,
Yann E. MORIN.
--
.-----------------.--------------------.------------------.--------------------.
| 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. |
'------------------------------^-------^------------------^--------------------'
More information about the buildroot
mailing list