[Buildroot] [PATCH] This upstream patch fixes chromium compile issues where standard includes are added that break compilation.

Arnout Vandecappelle arnout at mind.be
Thu Jun 27 21:59:46 UTC 2019


 Hi Nathan,

 Thanks for this patch.

 The subject of you commit message isn't very good. It should contain the
package name and it should be a bit shorter, e.g.

package/qt5webengine: add upstream chromium patch to fix include path

On 26/06/2019 17:04, Nathan Ford wrote:
> See the upstream commit for more detail:
>   https://chromium.googlesource.com/chromium/src.git/+/a8c8396fd20d98666d517c45b358c63736e345ef
> 
> Signed-off-by: Nathan Ford <nford at westpond.com>
> ---
>  .../5.12.2/0001-fix_compile_regression.patch         | 20 ++++++++++++++++++++
>  1 file changed, 20 insertions(+)
>  create mode 100644 package/qt5/qt5webengine/5.12.2/0001-fix_compile_regression.patch
> 
> diff --git a/package/qt5/qt5webengine/5.12.2/0001-fix_compile_regression.patch b/package/qt5/qt5webengine/5.12.2/0001-fix_compile_regression.patch
> new file mode 100644
> index 0000000..974e8c0
> --- /dev/null
> +++ b/package/qt5/qt5webengine/5.12.2/0001-fix_compile_regression.patch
> @@ -0,0 +1,20 @@
> +--- a/src/3rdparty/chromium/build/config/linux/pkg_config.gni	2019-06-06 08:53:17.000000000 -0400
> ++++ b/src/3rdparty/chromium/build/config/linux/pkg_config.gni	2019-05-23 08:39:34.000000000 -0400

 The patch should be git formatted since upstream is git-formatted. If
definitely needs a complete description - you can just take the upstream one. It
should also have your Signed-off-by. This is a short way for you to
assert that you are entitled to contribute the patch under the upstream license.
 See  http://elinux.org/Developer_Certificate_Of_Origin for more details.


> +@@ -103,11 +103,15 @@
> +     pkgresult = exec_script(pkg_config_script, args, "value")
> +     cflags = pkgresult[1]
> + 
> +-    # We want the system include paths to use -isystem instead of -I to suppress
> +-    # warnings in those headers.
> +     foreach(include, pkgresult[0]) {
> ++      if (use_sysroot) {
> ++        # We want the system include paths to use -isystem instead of -I to
> ++        # suppress warnings in those headers.

 I don't understand how this patch can change anything... I would expect
use_sysroot to be true, since we do have a sysroot...

 That said, if this patch fixes things (and BTW it has been included upstream in
qt5webengine 5.13) we should probably just apply it.

 Regards,
 Arnout

> +       include_relativized = rebase_path(include, root_build_dir)
> +       cflags += [ "-isystem$include_relativized" ]
> ++      } else {
> ++        cflags += [ "-I$include" ]
> ++      }
> +     }
> + 
> +     if (!defined(invoker.ignore_libs) || !invoker.ignore_libs) {
> 


More information about the buildroot mailing list