public inbox for gentoo-commits@lists.gentoo.org
 help / color / mirror / Atom feed
From: "Gilles Dartiguelongue" <eva@gentoo.org>
To: gentoo-commits@lists.gentoo.org
Subject: [gentoo-commits] proj/gnome:master commit in: net-libs/webkit-gtk/files/, net-libs/webkit-gtk/
Date: Wed, 27 Nov 2013 23:32:00 +0000 (UTC)	[thread overview]
Message-ID: <1385594828.9cf5cf2220ee1a5de7575441a0b817b948ca5aa9.eva@gentoo> (raw)

commit:     9cf5cf2220ee1a5de7575441a0b817b948ca5aa9
Author:     Gilles Dartiguelongue <eva <AT> gentoo <DOT> org>
AuthorDate: Wed Nov 27 22:39:11 2013 +0000
Commit:     Gilles Dartiguelongue <eva <AT> gentoo <DOT> org>
CommitDate: Wed Nov 27 23:27:08 2013 +0000
URL:        http://git.overlays.gentoo.org/gitweb/?p=proj/gnome.git;a=commit;h=9cf5cf22

net-libs/webkit-gtk: do not build unittests unless needed

---
 .../files/webkit-gtk-1.8.1-tests-xvfb.patch        | 32 ---------------
 .../files/webkit-gtk-2.2.2-unittests-build.patch   | 45 ++++++++++++++++++++++
 net-libs/webkit-gtk/webkit-gtk-2.2.2.ebuild        |  3 ++
 3 files changed, 48 insertions(+), 32 deletions(-)

diff --git a/net-libs/webkit-gtk/files/webkit-gtk-1.8.1-tests-xvfb.patch b/net-libs/webkit-gtk/files/webkit-gtk-1.8.1-tests-xvfb.patch
deleted file mode 100644
index e310c65..0000000
--- a/net-libs/webkit-gtk/files/webkit-gtk-1.8.1-tests-xvfb.patch
+++ /dev/null
@@ -1,32 +0,0 @@
-Rely on virtualx.eclass instead of manually spawning Xvfb. Fixes occasional
-test failures.
-
---- a/Tools/Scripts/run-gtk-tests
-+++ b/Tools/Scripts/run-gtk-tests
-@@ -147,18 +147,9 @@
- 
-     def _setup_testing_environment(self):
-         self._test_env = os.environ
--        self._test_env["DISPLAY"] = self._options.display
-         self._test_env["WEBKIT_INSPECTOR_PATH"] = os.path.abspath(os.path.join(self._programs_path, 'resources', 'inspector'))
-         self._test_env['GSETTINGS_BACKEND'] = 'memory'
- 
--        try:
--            self._xvfb = self._create_process(["Xvfb", self._options.display, "-screen", "0", "800x600x24", "-nolisten", "tcp"],
--                                              stdout=subprocess.PIPE, stderr=subprocess.PIPE)
--        except Exception as e:
--            sys.stderr.write("Failed to run Xvfb: %s\n", e)
--            sys.stderr.flush()
--            return False
--
-         # If we cannot start the accessibility daemons, we can just skip the accessibility tests.
-         if not self._start_accessibility_daemons():
-             print "Could not start accessibility bus, so skipping TestWebKitAccessibility"
-@@ -170,7 +161,6 @@
-             self._spi_registryd.terminate()
-         if self._spi_bus_launcher:
-             self._spi_bus_launcher.terminate()
--        self._xvfb.kill();
- 
-     def _remove_skipped_tests(self):
-         tests_to_remove = []

diff --git a/net-libs/webkit-gtk/files/webkit-gtk-2.2.2-unittests-build.patch b/net-libs/webkit-gtk/files/webkit-gtk-2.2.2-unittests-build.patch
new file mode 100644
index 0000000..14fa30f
--- /dev/null
+++ b/net-libs/webkit-gtk/files/webkit-gtk-2.2.2-unittests-build.patch
@@ -0,0 +1,45 @@
+From: Gilles Dartiguelongue <eva@gentoo.org>
+Date: Wed, 27 Nov 2013 23:36:41 +0100
+Subject: [PATCH] Build unittests on demand
+
+--- a/GNUmakefile.am	2013-11-27 23:52:56.148735433 +0100
++++ b/GNUmakefile.am	2013-11-27 23:51:25.551590806 +0100
+@@ -51,6 +51,7 @@
+ # Libraries and support components
+ bin_PROGRAMS :=
+ noinst_PROGRAMS :=
++check_PROGRAMS :=
+ libexec_PROGRAMS :=
+ noinst_DATA :=
+ noinst_HEADERS :=
+--- a/Source/WebKit/gtk/GNUmakefile.am
++++ b/Source/WebKit/gtk/GNUmakefile.am
+@@ -446,7 +446,7 @@ webkit_tests_ldflags = \
+ 	-no-fast-install
+ 
+ if ENABLE_WEBKIT1
+-noinst_PROGRAMS += \
++check_PROGRAMS += \
+ 	Programs/unittests/testapplicationcache \
+ 	Programs/unittests/testcontextmenu \
+ 	Programs/unittests/testdomdocument \
+--- a/Source/WebKit2/UIProcess/API/gtk/tests/GNUmakefile.am
++++ b/Source/WebKit2/UIProcess/API/gtk/tests/GNUmakefile.am
+@@ -25,12 +25,12 @@ TEST_PROGS += \
+ 	Programs/WebKit2APITests/TestWebKitWebViewGroup \
+ 	Programs/WebKit2APITests/TestWebViewEditor
+ 
+-noinst_PROGRAMS += $(TEST_PROGS)
++check_PROGRAMS += $(TEST_PROGS)
+ 
+ if HAVE_ATSPI2
+ TEST_PROGS += Programs/WebKit2APITests/TestWebKitAccessibility
+ 
+-noinst_PROGRAMS += Programs/WebKit2APITests/AccessibilityTestServer
++check_PROGRAMS += Programs/WebKit2APITests/AccessibilityTestServer
+ endif
+ 
+ webkit2_tests_cppflags = \
+-- 
+1.8.3.2
+

diff --git a/net-libs/webkit-gtk/webkit-gtk-2.2.2.ebuild b/net-libs/webkit-gtk/webkit-gtk-2.2.2.ebuild
index 7e1b36d..6afe3a6 100644
--- a/net-libs/webkit-gtk/webkit-gtk-2.2.2.ebuild
+++ b/net-libs/webkit-gtk/webkit-gtk-2.2.2.ebuild
@@ -178,6 +178,9 @@ src_prepare() {
 	# bug #459978, upstream bug #113397
 	epatch "${FILESDIR}/${PN}-1.11.90-gtk-docize-fix.patch"
 
+	# Do not build unittests unless requested
+	epatch "${FILESDIR}"/${PN}-2.2.2-unittests-build.patch
+
 	# Prevent maintainer mode from being triggered during make
 	AT_M4DIR=Source/autotools eautoreconf
 }


             reply	other threads:[~2013-11-27 23:32 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-11-27 23:32 Gilles Dartiguelongue [this message]
  -- strict thread matches above, loose matches on Subject: below --
2014-04-21 14:40 [gentoo-commits] proj/gnome:master commit in: net-libs/webkit-gtk/files/, net-libs/webkit-gtk/ Gilles Dartiguelongue
2013-03-03  0:57 Priit Laes
2013-02-13 14:05 Priit Laes
2013-01-15  9:39 Priit Laes
2012-11-04  6:25 Alexandre Rostovtsev
2012-10-27  8:43 Priit Laes
2012-10-10 17:34 Priit Laes
2012-09-13  5:35 Alexandre Rostovtsev
2012-07-12 12:33 Priit Laes
2012-04-14  6:10 Alexandre Restovtsev
2012-04-06  2:36 Alexandre Restovtsev
2012-03-04 21:11 Alexandre Restovtsev
2012-02-26 19:20 Alexandre Restovtsev
2012-01-17  9:08 Priit Laes
2011-12-20 17:37 Priit Laes
2011-11-28  5:35 Priit Laes
2011-11-26  8:43 Priit Laes
2011-10-30  0:13 Alexandre Restovtsev
2011-06-11  1:44 Nirbheek Chauhan
2011-06-05 20:12 Priit Laes
2011-05-07 19:18 Priit Laes
2011-03-22 13:22 Priit Laes
2011-03-04 10:23 Nirbheek Chauhan
2011-02-25 11:54 Priit Laes

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=1385594828.9cf5cf2220ee1a5de7575441a0b817b948ca5aa9.eva@gentoo \
    --to=eva@gentoo.org \
    --cc=gentoo-commits@lists.gentoo.org \
    --cc=gentoo-dev@lists.gentoo.org \
    /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