From: Adrian Perez de Castro <aperez@igalia.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH v1 3/5] webkitgtk: remove GTK in WEBKITGTK_ARCH_SUPPORTS* variable names
Date: Mon, 4 Feb 2019 15:25:17 +0100 [thread overview]
Message-ID: <20190204152517.GF22033@momiji> (raw)
In-Reply-To: <20181231154915.32660-4-francois.perrad@gadz.org>
Hi,
On Mon, 31 Dec 2018 16:49:13 +0100, Francois Perrad <fperrad@gmail.com> wrote:
> there are common for the GTK & WPE ports of WebKit
>
> Signed-off-by: Francois Perrad <francois.perrad@gadz.org>
> ---
> package/midori/Config.in | 4 ++--
> package/webkitgtk/Config.in | 8 ++++----
> package/webkitgtk/webkitgtk.mk | 2 +-
> 3 files changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/package/midori/Config.in b/package/midori/Config.in
> index f0767c85f..460fc6fbf 100644
> --- a/package/midori/Config.in
> +++ b/package/midori/Config.in
> @@ -1,5 +1,5 @@
> comment "midori needs libgtk3 and a glibc toolchain w/ C++, gcc >= 6, host gcc >= 4.8"
> - depends on BR2_PACKAGE_WEBKITGTK_ARCH_SUPPORTS
> + depends on BR2_PACKAGE_WEBKIT_ARCH_SUPPORTS
In the end let's *not* rename this with this patch set, and unify the
configuration of both ?wpewebkit? and ?webkitgtk? packages in a different
patch set later.
> depends on !BR2_INSTALL_LIBSTDCPP || !BR2_PACKAGE_LIBGTK3 || \
> !BR2_HOST_GCC_AT_LEAST_4_8 || !BR2_TOOLCHAIN_GCC_AT_LEAST_6 || \
> !BR2_TOOLCHAIN_USES_GLIBC
> @@ -12,7 +12,7 @@ config BR2_PACKAGE_MIDORI
> depends on BR2_HOST_GCC_AT_LEAST_4_8 # webkitgtk -> icu
> depends on BR2_TOOLCHAIN_GCC_AT_LEAST_6 # webkitgtk
> depends on BR2_TOOLCHAIN_USES_GLIBC # webkitgtk
> - depends on BR2_PACKAGE_WEBKITGTK_ARCH_SUPPORTS
> + depends on BR2_PACKAGE_WEBKIT_ARCH_SUPPORTS
> # GCR can only be used with the X11 backend
> select BR2_PACKAGE_GCR if BR2_PACKAGE_LIBGTK3_X11
> select BR2_PACKAGE_GRANITE
> diff --git a/package/webkitgtk/Config.in b/package/webkitgtk/Config.in
> index 8714677b6..3e5e8d7b6 100644
> --- a/package/webkitgtk/Config.in
> +++ b/package/webkitgtk/Config.in
> @@ -1,4 +1,4 @@
> -config BR2_PACKAGE_WEBKITGTK_ARCH_SUPPORTS
> +config BR2_PACKAGE_WEBKIT_ARCH_SUPPORTS
> bool
> # ARM needs BLX, so v5t+, BE completely untested so disabled
> default y if BR2_arm && !BR2_ARM_CPU_ARMV4
> @@ -12,7 +12,7 @@ config BR2_PACKAGE_WEBKITGTK_ARCH_SUPPORTS
> depends on BR2_TOOLCHAIN_HAS_SYNC_4
> depends on BR2_PACKAGE_LIBGPG_ERROR_ARCH_SUPPORTS # libgcrypt
>
> -config BR2_PACKAGE_WEBKITGTK_ARCH_SUPPORTS_JIT
> +config BR2_PACKAGE_WEBKIT_ARCH_SUPPORTS_JIT
> bool
> # ARM needs NEON for JIT.
> default y if BR2_ARM_CPU_HAS_NEON
> @@ -29,7 +29,7 @@ config BR2_PACKAGE_WEBKITGTK_ARCH_SUPPORTS_JIT
> default y if BR2_mipsel && BR2_MIPS_CPU_MIPS32R5
>
> comment "webkitgtk needs libgtk3 and a glibc toolchain w/ C++, gcc >= 6, host gcc >= 4.8"
> - depends on BR2_PACKAGE_WEBKITGTK_ARCH_SUPPORTS
> + depends on BR2_PACKAGE_WEBKIT_ARCH_SUPPORTS
> depends on !BR2_PACKAGE_LIBGTK3 || !BR2_INSTALL_LIBSTDCPP || \
> !BR2_TOOLCHAIN_USES_GLIBC || \
> !BR2_HOST_GCC_AT_LEAST_4_8 || \
> @@ -43,7 +43,7 @@ config BR2_PACKAGE_WEBKITGTK
> depends on BR2_TOOLCHAIN_GCC_AT_LEAST_6
> depends on BR2_TOOLCHAIN_USES_GLIBC
> depends on BR2_PACKAGE_LIBGTK3
> - depends on BR2_PACKAGE_WEBKITGTK_ARCH_SUPPORTS
> + depends on BR2_PACKAGE_WEBKIT_ARCH_SUPPORTS
> select BR2_PACKAGE_CAIRO
> select BR2_PACKAGE_CAIRO_PNG
> select BR2_PACKAGE_ENCHANT
> diff --git a/package/webkitgtk/webkitgtk.mk b/package/webkitgtk/webkitgtk.mk
> index 80daadd15..53d6236ca 100644
> --- a/package/webkitgtk/webkitgtk.mk
> +++ b/package/webkitgtk/webkitgtk.mk
> @@ -27,7 +27,7 @@ WEBKITGTK_CONF_OPTS = \
> -DUSE_LIBHYPHEN=OFF \
> -DUSE_WOFF2=ON
>
> -ifeq ($(BR2_PACKAGE_WEBKITGTK_ARCH_SUPPORTS_JIT),y)
> +ifeq ($(BR2_PACKAGE_WEBKIT_ARCH_SUPPORTS_JIT),y)
> WEBKITGTK_CONF_OPTS += -DENABLE_JIT=ON
> else
> WEBKITGTK_CONF_OPTS += -DENABLE_JIT=OFF
> --
> 2.17.1
Cheers,
-Adri?n
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 195 bytes
Desc: not available
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20190204/426d113c/attachment.asc>
next prev parent reply other threads:[~2019-02-04 14:25 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-12-31 15:49 [Buildroot] [PATCH v1 0/5] WPE WebKit Francois Perrad
2018-12-31 15:49 ` [Buildroot] [PATCH v1 1/5] libwpe: new package Francois Perrad
2019-02-04 14:21 ` Adrian Perez de Castro
2018-12-31 15:49 ` [Buildroot] [PATCH v1 2/5] wpebackend-fdo: " Francois Perrad
2019-02-04 14:22 ` Adrian Perez de Castro
2018-12-31 15:49 ` [Buildroot] [PATCH v1 3/5] webkitgtk: remove GTK in WEBKITGTK_ARCH_SUPPORTS* variable names Francois Perrad
2019-02-04 14:25 ` Adrian Perez de Castro [this message]
2018-12-31 15:49 ` [Buildroot] [PATCH v1 4/5] wpewebkit: new package Francois Perrad
2019-02-04 14:28 ` Adrian Perez de Castro
2018-12-31 15:49 ` [Buildroot] [PATCH v1 5/5] cog: " Francois Perrad
2019-02-04 14:34 ` Adrian Perez de Castro
2019-02-04 14:18 ` [Buildroot] [PATCH v1 0/5] WPE WebKit Adrian Perez de Castro
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=20190204152517.GF22033@momiji \
--to=aperez@igalia.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