From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH v2 1/1] webkitgtk: Add patch for properly picking GL flags when building
Date: Thu, 3 Aug 2017 17:58:10 +0200 [thread overview]
Message-ID: <20170803175810.71d1fcca@windsurf.lan> (raw)
In-Reply-To: <20170803181034.GB17397@momiji>
Hello,
On Thu, 3 Aug 2017 18:10:34 +0300, Adrian Perez de Castro wrote:
> On Thu, 3 Aug 2017 15:41:56 +0200, Thomas Petazzoni <thomas.petazzoni@free-electrons.com> wrote:
>
> > On Thu, 3 Aug 2017 14:31:09 +0300, Adrian Perez de Castro wrote:
> > > This solves build failures caused by WebKit trying to include X11 headers
> > > when support for X11 is disabled in Mesa3D. A common situation is when
> > > configuring both GTK+ and WebKitGTK+ only with Wayland support.
> >
> > Have such errors occurred in the autobuilders? If so, we want to have a
> > reference to such autobuilder failures in the commit log.
>
> Unfortunately, I have not seen this kind of build failure in the autobuilder
> logs, sorry. But the issue certainly exists, and I can experience it myself,
> and I am aware of a couple of work mates who have hit this ? one of them
> Carlos Lopez, who made the patch for the WebKit CMake files.
In this case, ideally having a defconfig that exhibits the issue is
always nice. Don't bother adding it now, but next time, when you fix a
build failure, adding some kind of minimal defconfig is very good. For
trivial build issues, it's not needed. But for build issues that
involve multiple packages in specific configurations (such as a
condition of a given OpenGL provider and WebkitGtk), having a defconfig
is good.
Thanks!
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
next prev parent reply other threads:[~2017-08-03 15:58 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-08-03 11:31 [Buildroot] [PATCH v2 1/1] webkitgtk: Add patch for properly picking GL flags when building Adrian Perez de Castro
2017-08-03 13:41 ` Thomas Petazzoni
2017-08-03 15:10 ` Adrian Perez de Castro
2017-08-03 15:58 ` Thomas Petazzoni [this message]
2017-08-10 9:19 ` Arnout Vandecappelle
2017-09-04 7:10 ` Peter Korsgaard
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20170803175810.71d1fcca@windsurf.lan \
--to=thomas.petazzoni@free-electrons.com \
--cc=buildroot@busybox.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox