From patchwork Mon Feb 18 17:17:17 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Adrian Perez de Castro X-Patchwork-Id: 1044195 Return-Path: X-Original-To: incoming-buildroot@patchwork.ozlabs.org Delivered-To: patchwork-incoming-buildroot@bilbo.ozlabs.org Authentication-Results: ozlabs.org; spf=pass (mailfrom) smtp.mailfrom=busybox.net (client-ip=140.211.166.133; helo=hemlock.osuosl.org; envelope-from=buildroot-bounces@busybox.net; receiver=) Authentication-Results: ozlabs.org; dmarc=none (p=none dis=none) header.from=igalia.com Authentication-Results: ozlabs.org; dkim=fail reason="signature verification failed" (2048-bit key; unprotected) header.d=igalia.com header.i=@igalia.com header.b="kpMlaVGy"; dkim-atps=neutral Received: from hemlock.osuosl.org (smtp2.osuosl.org [140.211.166.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 4439YD5HRqz9s7h for ; Tue, 19 Feb 2019 04:17:52 +1100 (AEDT) Received: from localhost (localhost [127.0.0.1]) by hemlock.osuosl.org (Postfix) with ESMTP id 2CBB98774C; Mon, 18 Feb 2019 17:17:48 +0000 (UTC) X-Virus-Scanned: amavisd-new at osuosl.org Received: from hemlock.osuosl.org ([127.0.0.1]) by localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id YE0ZcxZOFFiL; Mon, 18 Feb 2019 17:17:44 +0000 (UTC) Received: from ash.osuosl.org (ash.osuosl.org [140.211.166.34]) by hemlock.osuosl.org (Postfix) with ESMTP id 7FA3B8756C; Mon, 18 Feb 2019 17:17:43 +0000 (UTC) X-Original-To: buildroot@lists.busybox.net Delivered-To: buildroot@osuosl.org Received: from whitealder.osuosl.org (smtp1.osuosl.org [140.211.166.138]) by ash.osuosl.org (Postfix) with ESMTP id 04D3B1BF2A0 for ; Mon, 18 Feb 2019 17:17:41 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by whitealder.osuosl.org (Postfix) with ESMTP id 00E4E85335 for ; Mon, 18 Feb 2019 17:17:41 +0000 (UTC) X-Virus-Scanned: amavisd-new at osuosl.org Received: from whitealder.osuosl.org ([127.0.0.1]) by localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id QSusYkMWMldK for ; Mon, 18 Feb 2019 17:17:38 +0000 (UTC) X-Greylist: from auto-whitelisted by SQLgrey-1.7.6 Received: from fanzine.igalia.com (fanzine.igalia.com [91.117.99.155]) by whitealder.osuosl.org (Postfix) with ESMTPS id 949978528B for ; Mon, 18 Feb 2019 17:17:36 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=igalia.com; s=20170329; h=Content-Transfer-Encoding:MIME-Version:References:In-Reply-To:Message-Id:Date:Subject:To:From; bh=baVIZUtAxcji006iK0fr1r4jradcCQECWF0FPvvtzyM=; b=kpMlaVGyq5BiC+qnBw8OyVosfGQ4jP4JZLUbXJLJo4WHhqqgJhlXBNS0sS3c4dpMQiy/q1qmnqFW4VrMmM82FMJEL2Z3oTfPbUHeSMTRp6GK3+ygEDg9S43Ar0vfYUfk7RcC5XO9/Qz/YV82kUaqj2pyCmAt2CKWQcL7QC3XQmI1BzNcLgkk4gxWsS2xW+mEfdu3rw2QBWju6B80cAhD9Ni/16EYzNLmbl3eELiZIdCRDtEy4jCxlt5hN/agaAqFUcWZLPDSiYVv63PBX3b845VLnGDH+Gerh5LGc+dDUVur09SttpX2cLfqCJupVoBN0pJnAXEYRfVw9CbO+kZ5hA==; Received: from 62-165-147-230.co.dnainternet.fi ([62.165.147.230] helo=kodama) by fanzine.igalia.com with esmtpsa (Cipher TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim) id 1gvmXu-0003Dg-2K for ; Mon, 18 Feb 2019 18:17:34 +0100 Received: from localhost (kodama [local]) by kodama (OpenSMTPD) with ESMTPA id ba91ebcd for ; Mon, 18 Feb 2019 17:17:19 +0000 (UTC) From: Adrian Perez de Castro To: buildroot@buildroot.org Date: Mon, 18 Feb 2019 19:17:17 +0200 Message-Id: <20190218171719.18721-3-aperez@igalia.com> X-Mailer: git-send-email 2.20.1 In-Reply-To: <20190218171719.18721-1-aperez@igalia.com> References: <20190218171719.18721-1-aperez@igalia.com> MIME-Version: 1.0 Subject: [Buildroot] [PATCH RFC/next 2/4] package/webkit: unify *_ARCH_SUPPORTS{, _JIT} X-BeenThere: buildroot@busybox.net X-Mailman-Version: 2.1.29 Precedence: list List-Id: Discussion and development of buildroot List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: buildroot-bounces@busybox.net Sender: "buildroot" Instead of having BR2_PACKAGE_{WPEWEBKIT,WEBKITGTK}_ARCH_SUPPORTS and BR2_PACKAGE_{WPEWEBKIT,WEBKITGTK}_ARCH_SUPPORTS_JIT, which have the same conditions for both WebKit packages, unify them into common configuration variables BR2_WEBKIT_ARCH_SUPPORTS{,_JIT} which get used by both. Also, propagate the needed changes to the .mk files. Signed-off-by: Adrian Perez de Castro --- package/Config.in | 1 + package/midori/Config.in | 4 +-- package/webkit/Config.in | 29 +++++++++++++++++++++ package/webkit/webkitgtk/Config.in | 34 ++----------------------- package/webkit/webkitgtk/webkitgtk.mk | 2 +- package/webkit/wpewebkit/Config.in | 36 +++------------------------ package/webkit/wpewebkit/wpewebkit.mk | 2 +- 7 files changed, 39 insertions(+), 69 deletions(-) create mode 100644 package/webkit/Config.in diff --git a/package/Config.in b/package/Config.in index 65faf1fcd6..6801637f33 100644 --- a/package/Config.in +++ b/package/Config.in @@ -1320,6 +1320,7 @@ menu "Graphics" source "package/wayland-protocols/Config.in" source "package/waylandpp/Config.in" source "package/webkit/webkitgtk/Config.in" + source "package/webkit/Config.in" source "package/webkit/libwpe/Config.in" source "package/webkit/wpebackend-fdo/Config.in" source "package/webkit/wpewebkit/Config.in" diff --git a/package/midori/Config.in b/package/midori/Config.in index f0767c85f9..2bd82b448a 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_WEBKIT_ARCH_SUPPORTS 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_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/webkit/Config.in b/package/webkit/Config.in new file mode 100644 index 0000000000..5e4660d59a --- /dev/null +++ b/package/webkit/Config.in @@ -0,0 +1,29 @@ +config BR2_WEBKIT_ARCH_SUPPORTS + bool + # ARM needs BLX, so v5t+, BE completely untested so disabled + default y if BR2_arm && !BR2_ARM_CPU_ARMV4 + default y if BR2_aarch64 || BR2_aarch64_be + default y if BR2_i386 || BR2_x86_64 + # Disabled on MIPS big endian due to sigbus + default y if BR2_mipsel || BR2_mips64el + # Disabled on PowerPC pending runtime testing + # Disabled on SuperH because of segfault + depends on BR2_USE_MMU # libglib2 + depends on BR2_TOOLCHAIN_HAS_SYNC_4 + depends on BR2_PACKAGE_LIBGPG_ERROR_ARCH_SUPPORTS # libgcrypt + +config BR2_WEBKIT_ARCH_SUPPORTS_JIT + bool + # ARM needs NEON for JIT. + default y if BR2_ARM_CPU_HAS_NEON + # AArch64 is supported upstream but not well tested on big-endian mode. + default y if BR2_aarch64 + # i386 & x86_64 don't have any special requirements. + default y if BR2_i386 + default y if BR2_x86_64 + # JIT is known not to work on MIPS64. + # Plain MIPS32 (pre R2) is not well tested and likely broken, and R6 + # is unsupported, see https://bugs.webkit.org/show_bug.cgi?id=191258 + # The MIPS support is completely untested in big-endian mode. + default y if BR2_mipsel && BR2_MIPS_CPU_MIPS32R2 + default y if BR2_mipsel && BR2_MIPS_CPU_MIPS32R5 diff --git a/package/webkit/webkitgtk/Config.in b/package/webkit/webkitgtk/Config.in index 8714677b6a..09c6848723 100644 --- a/package/webkit/webkitgtk/Config.in +++ b/package/webkit/webkitgtk/Config.in @@ -1,35 +1,5 @@ -config BR2_PACKAGE_WEBKITGTK_ARCH_SUPPORTS - bool - # ARM needs BLX, so v5t+, BE completely untested so disabled - default y if BR2_arm && !BR2_ARM_CPU_ARMV4 - default y if BR2_aarch64 || BR2_aarch64_be - default y if BR2_i386 || BR2_x86_64 - # Disabled on MIPS big endian due to sigbus - default y if BR2_mipsel || BR2_mips64el - # Disabled on PowerPC pending runtime testing - # Disabled on SuperH because of segfault - depends on BR2_USE_MMU # libglib2 - depends on BR2_TOOLCHAIN_HAS_SYNC_4 - depends on BR2_PACKAGE_LIBGPG_ERROR_ARCH_SUPPORTS # libgcrypt - -config BR2_PACKAGE_WEBKITGTK_ARCH_SUPPORTS_JIT - bool - # ARM needs NEON for JIT. - default y if BR2_ARM_CPU_HAS_NEON - # AArch64 is supported upstream but not well tested on big-endian mode. - default y if BR2_aarch64 - # i386 & x86_64 don't have any special requirements. - default y if BR2_i386 - default y if BR2_x86_64 - # JIT is known not to work on MIPS64. - # Plain MIPS32 (pre R2) is not well tested and likely broken, and R6 - # is unsupported, see https://bugs.webkit.org/show_bug.cgi?id=191258 - # The MIPS support is completely untested in big-endian mode. - default y if BR2_mipsel && BR2_MIPS_CPU_MIPS32R2 - 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_WEBKIT_ARCH_SUPPORTS depends on !BR2_PACKAGE_LIBGTK3 || !BR2_INSTALL_LIBSTDCPP || \ !BR2_TOOLCHAIN_USES_GLIBC || \ !BR2_HOST_GCC_AT_LEAST_4_8 || \ @@ -43,7 +13,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_WEBKIT_ARCH_SUPPORTS select BR2_PACKAGE_CAIRO select BR2_PACKAGE_CAIRO_PNG select BR2_PACKAGE_ENCHANT diff --git a/package/webkit/webkitgtk/webkitgtk.mk b/package/webkit/webkitgtk/webkitgtk.mk index c17bbc8028..1834108bcc 100644 --- a/package/webkit/webkitgtk/webkitgtk.mk +++ b/package/webkit/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_WEBKIT_ARCH_SUPPORTS_JIT),y) WEBKITGTK_CONF_OPTS += -DENABLE_JIT=ON else WEBKITGTK_CONF_OPTS += -DENABLE_JIT=OFF diff --git a/package/webkit/wpewebkit/Config.in b/package/webkit/wpewebkit/Config.in index 64e81a95f9..dbcab39ebe 100644 --- a/package/webkit/wpewebkit/Config.in +++ b/package/webkit/wpewebkit/Config.in @@ -1,42 +1,12 @@ -config BR2_PACKAGE_WPEWEBKIT_ARCH_SUPPORTS - bool - # ARM needs BLX, so v5t+, BE completely untested so disabled - default y if BR2_arm && !BR2_ARM_CPU_ARMV4 - default y if BR2_aarch64 || BR2_aarch64_be - default y if BR2_i386 || BR2_x86_64 - # Disabled on MIPS big endian due to sigbus - default y if BR2_mipsel || BR2_mips64el - # Disabled on PowerPC pending runtime testing - # Disabled on SuperH because of segfault - depends on BR2_USE_MMU # libglib2 - depends on BR2_TOOLCHAIN_HAS_SYNC_4 - depends on BR2_PACKAGE_LIBGPG_ERROR_ARCH_SUPPORTS # libgcrypt - -config BR2_PACKAGE_WPEWEBKIT_ARCH_SUPPORTS_JIT - bool - # ARM needs NEON for JIT. - default y if BR2_ARM_CPU_HAS_NEON - # AArch64 is supported upstream but not well tested on big-endian mode. - default y if BR2_aarch64 - # i386 & x86_64 don't have any special requirements. - default y if BR2_i386 - default y if BR2_x86_64 - # JIT is known not to work on MIPS64. - # Plain MIPS32 (pre R2) is not well tested and likely broken, and R6 - # is unsupported, see https://bugs.webkit.org/show_bug.cgi?id=191258 - # The MIPS support is completely untested in big-endian mode. - default y if BR2_mipsel && BR2_MIPS_CPU_MIPS32R2 - default y if BR2_mipsel && BR2_MIPS_CPU_MIPS32R5 - comment "wpewebkit needs a toolchain w/ C++, wchar, threads, dynamic library, gcc >= 4.8, host gcc >= 4.8" - depends on BR2_PACKAGE_WPEWEBKIT_ARCH_SUPPORTS + depends on BR2_WEBKIT_ARCH_SUPPORTS depends on !BR2_BINFMT_FLAT depends on !BR2_INSTALL_LIBSTDCPP || !BR2_USE_WCHAR \ || !BR2_TOOLCHAIN_HAS_THREADS || BR2_STATIC_LIBS \ || !BR2_TOOLCHAIN_GCC_AT_LEAST_4_8 || BR2_HOST_GCC_AT_LEAST_4_8 comment "wpewebkit needs an OpenGL ES w/ EGL backend" - depends on BR2_PACKAGE_WPEWEBKIT_ARCH_SUPPORTS + depends on BR2_WEBKIT_ARCH_SUPPORTS depends on !BR2_BINFMT_FLAT depends on !BR2_PACKAGE_HAS_LIBGLES || !BR2_PACKAGE_HAS_LIBEGL @@ -51,7 +21,7 @@ config BR2_PACKAGE_WPEWEBKIT depends on BR2_USE_WCHAR # icu, libsoup depends on BR2_PACKAGE_HAS_LIBGLES # libepoxy depends on BR2_PACKAGE_HAS_LIBEGL # libepoxy - depends on BR2_PACKAGE_WPEWEBKIT_ARCH_SUPPORTS + depends on BR2_WEBKIT_ARCH_SUPPORTS select BR2_PACKAGE_CAIRO select BR2_PACKAGE_CAIRO_PNG select BR2_PACKAGE_HARFBUZZ diff --git a/package/webkit/wpewebkit/wpewebkit.mk b/package/webkit/wpewebkit/wpewebkit.mk index 89df9b725c..3877dcfbb0 100644 --- a/package/webkit/wpewebkit/wpewebkit.mk +++ b/package/webkit/wpewebkit/wpewebkit.mk @@ -21,7 +21,7 @@ WPEWEBKIT_CONF_OPTS = \ -DENABLE_API_TESTS=OFF \ -DENABLE_MINIBROWSER=OFF -ifeq ($(BR2_PACKAGE_WPEWEBKIT_ARCH_SUPPORTS_JIT),y) +ifeq ($(BR2_WEBKIT_ARCH_SUPPORTS_JIT),y) WPEWEBKIT_CONF_OPTS += -DENABLE_JIT=ON else WPEWEBKIT_CONF_OPTS += -DENABLE_JIT=OFF