[Buildroot] [PATCH v2 1/1] qt5webengine: update chromium license file names

Thomas Petazzoni thomas.petazzoni at bootlin.com
Wed Aug 29 20:32:42 UTC 2018


Hello Laurent,

On Wed, 29 Aug 2018 12:51:24 +0200, Laurent Hartanerot wrote:
> From: Laurent Hartanerot <laurent.hartanerot at atos.net>
> 
> Add license file name for qt5webengine-5.6
> 
> Signed-off-by: Laurent Hartanerot <laurent.hartanerot at atos.net>

Thanks for this new iteration. Unfortunately, I still see a number of
problems, some of them dating back from the origin of the qt5webengine
package (so Gaël if you're around, your feedback/help would be nice).

Laurent, the first issue is that when your patch is applied, legal-info
still doesn't work when the build wasn't done, i.e if you do "make
clean; make legal-info", it will fail with:

cp: cannot stat '/home/thomas/projets/buildroot/output/build/qt5webengine-5.6.3/src/3rdparty/chromium/third_party/WebKit/Source/build/scripts/license.pyc': No such file or directory

This is because you have added the license.pyc file in the list of
license files, but this file only gets created at build time. And
anyway, since it's Python byte-code, it's not very useful.

How was this list of license files generated ? It seems like all files
containing "copying" or "license" in their name were taken. But does it
make sense to include those license.py files ?

Another problem is that the legal-info for qt5webengine in version 5.11
is also broken:

cp: cannot stat '/home/thomas/projets/buildroot/output/build/qt5webengine-5.11.1/src/3rdparty/chromium/buildtools/third_party/libc++abi/trunk/LICENSE.TXT': No such file or directory

> +CHROMIUM_LICENSE_FILES = \

Another problem is the name of this variable: if we ever have a package
called "chromium" (which is not unlikely, as it was submitted in
https://patchwork.ozlabs.org/patch/942284/), then this
CHROMIUM_LICENSE_FILES variable internal to qt5webengine will clash
with the CHROMIUM_LICENSE_FILES of the legitimate chromium package.

So the naming of this variable should be changed to
QT5WEBENGINE_CHROMIUM_LICENSE_FILES or something like this.

> +ifeq ($(BR2_PACKAGE_QT5_VERSION_LATEST),y)
>  include package/qt5/qt5webengine/chromium.inc

Perhaps this file should be renamed chromium-latest.inc

> +else
> +include package/qt5/qt5webengine/chromium.5.6.inc

and this one chromium-lts.inc ?

Could you send a couple of separate patches to fix those issues ?

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com



More information about the buildroot mailing list