From: Efraim Flashner <efraim@flashner.co.il>
To: 宋文武 <iyzsong@gmail.com>
Cc: guix-devel@gnu.org
Subject: Re: [PATCH 2/4] gnu: glib: Drop glib-tests-homedir.patch.
Date: Sun, 3 Apr 2016 13:09:34 +0300 [thread overview]
Message-ID: <20160403100934.GA6142@debian-netbook> (raw)
In-Reply-To: <1459650631-13117-2-git-send-email-iyzsong@gmail.com>
[-- Attachment #1: Type: text/plain, Size: 5702 bytes --]
On Sun, Apr 03, 2016 at 10:30:29AM +0800, 宋文武 wrote:
> * gnu/packages/glib.scm (glib)[source]: Remave glib-tests-homedir.patch.
s/Remave/remove
> [arguments]: Set HOME and disable '/gdbus/codegen-peer-to-peer' test.
> * gnu/packages/patches/glib-tests-homedir.patch: Remove file.
> * gnu-system.am (dist_patch_DATA): Remove it.
> ---
> gnu-system.am | 1 -
> gnu/packages/glib.scm | 7 +++-
> gnu/packages/patches/glib-tests-homedir.patch | 59 ---------------------------
> 3 files changed, 5 insertions(+), 62 deletions(-)
> delete mode 100644 gnu/packages/patches/glib-tests-homedir.patch
>
> diff --git a/gnu-system.am b/gnu-system.am
> index d45b3d1..293bda3 100644
> --- a/gnu-system.am
> +++ b/gnu-system.am
> @@ -486,7 +486,6 @@ dist_patch_DATA = \
> gnu/packages/patches/ghostscript-runpath.patch \
> gnu/packages/patches/glib-networking-ssl-cert-file.patch \
> gnu/packages/patches/glib-tests-desktop.patch \
> - gnu/packages/patches/glib-tests-homedir.patch \
> gnu/packages/patches/glib-tests-prlimit.patch \
> gnu/packages/patches/glib-tests-timer.patch \
> gnu/packages/patches/glib-tests-gapplication.patch \
> diff --git a/gnu/packages/glib.scm b/gnu/packages/glib.scm
> index 2fb601c..2b90c12 100644
> --- a/gnu/packages/glib.scm
> +++ b/gnu/packages/glib.scm
> @@ -139,8 +139,7 @@ shared NFS home directories.")
> (sha256
> (base32
> "1yzxr1ip3l0m9ydk5nq32piq70c9f17p5f0jyvlsghzbaawh67ss"))
> - (patches (list (search-patch "glib-tests-homedir.patch")
> - (search-patch "glib-tests-desktop.patch")
> + (patches (list (search-patch "glib-tests-desktop.patch")
> (search-patch "glib-tests-prlimit.patch")
> (search-patch "glib-tests-timer.patch")
> (search-patch "glib-tests-gapplication.patch")))))
> @@ -171,6 +170,7 @@ shared NFS home directories.")
> "/share/zoneinfo"))
>
> ;; Some tests want write access there.
> + (setenv "HOME" (getcwd))
> (setenv "XDG_CACHE_HOME" (getcwd))
>
> (substitute* '("glib/gspawn.c"
> @@ -179,6 +179,9 @@ shared NFS home directories.")
> (("/bin/sh")
> (string-append (assoc-ref inputs "bash") "/bin/sh")))
>
> + ;; Disable a test that requires /etc/machine-id.
> + (substitute* "gio/tests/gdbus-peer.c"
> + (("g_test_add_func.*/gdbus/codegen-peer-to-peer.*") ""))
> ;; Disable a test that requires dbus.
> (substitute* "gio/tests/gdbus-serialization.c"
> (("g_test_add_func \
> diff --git a/gnu/packages/patches/glib-tests-homedir.patch b/gnu/packages/patches/glib-tests-homedir.patch
> deleted file mode 100644
> index 0a2bcf1..0000000
> --- a/gnu/packages/patches/glib-tests-homedir.patch
> +++ /dev/null
> @@ -1,59 +0,0 @@
> -`g_get_home_dir' looks at /etc/passwd first, which fails in chroot builds.
> -The gdbus tests use it to lookup .dbus-keyrings, so they cannot run in our
> -chroot build environment. Thus, disable them.
> -
> ---- glib-2.34.3/gio/tests/gdbus-connection-flush.c 2013-01-16 17:29:46.000000000 +0100
> -+++ glib-2.34.3/gio/tests/gdbus-connection-flush.c 2013-01-16 17:29:47.000000000 +0100
> -@@ -373,10 +373,6 @@ main (int argc,
> - g_type_init ();
> - g_test_init (&argc, &argv, NULL);
> -
> -- g_test_add ("/gdbus/connection/flush/busy", Fixture, NULL,
> -- setup, test_flush_busy, teardown);
> -- g_test_add ("/gdbus/connection/flush/idle", Fixture, NULL,
> -- setup, test_flush_idle, teardown);
> -
> - ret = g_test_run();
> -
> ---- glib-2.38.0.orig/gio/tests/gdbus-peer.c 2013-08-08 12:00:40.000000000 +0200
> -+++ glib-2.38.0/gio/tests/gdbus-peer.c 2013-09-30 19:36:40.000000000 +0200
> -@@ -1746,11 +1746,6 @@
> -
> - g_test_add_func ("/gdbus/peer-to-peer", test_peer);
> - g_test_add_func ("/gdbus/delayed-message-processing", delayed_message_processing);
> -- g_test_add_func ("/gdbus/nonce-tcp", test_nonce_tcp);
> --
> -- g_test_add_func ("/gdbus/tcp-anonymous", test_tcp_anonymous);
> -- g_test_add_func ("/gdbus/credentials", test_credentials);
> -- g_test_add_func ("/gdbus/codegen-peer-to-peer", codegen_test_peer);
> -
> - ret = g_test_run();
> -
> ---- glib-2.37.1/gio/tests/gdbus-exit-on-close.c 2013-06-07 23:41:34.000000000 +0200
> -+++ glib-2.37.1/gio/tests/gdbus-exit-on-close.c 2013-06-07 23:41:40.000000000 +0200
> -@@ -211,6 +211,7 @@ main (int argc,
> -
> - g_test_init (&argc, &argv, NULL);
> -
> -+ return g_test_run();
> - for (i = 0; cases[i].name != NULL; i++)
> - {
> - gchar *name;
> -@@ -224,5 +225,4 @@ main (int argc,
> - g_free (name);
> - }
> -
> -- return g_test_run();
> - }
> -
> ---- glib-2.34.3/gio/tests/gdbus-non-socket.c 2013-01-16 18:13:25.000000000 +0100
> -+++ glib-2.34.3/gio/tests/gdbus-non-socket.c 2013-01-16 18:13:27.000000000 +0100
> -@@ -294,7 +294,6 @@ main (int argc,
> - g_type_init ();
> - g_test_init (&argc, &argv, NULL);
> -
> -- g_test_add_func ("/gdbus/non-socket", test_non_socket);
> -
> - ret = g_test_run();
> -
> -
> --
> 2.6.3
>
>
--
Efraim Flashner <efraim@flashner.co.il> אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D 14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
next prev parent reply other threads:[~2016-04-03 10:09 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-04-03 2:30 [PATCH 1/4] gnu: glib: Use modify-phases syntax 宋文武
2016-04-03 2:30 ` [PATCH 2/4] gnu: glib: Drop glib-tests-homedir.patch 宋文武
2016-04-03 10:09 ` Efraim Flashner [this message]
2016-04-03 17:00 ` Efraim Flashner
2016-04-03 2:30 ` [PATCH 3/4] gnu: glib: Disable tests in build phases instead of patches 宋文武
2016-04-03 2:55 ` 宋文武
2016-04-03 17:03 ` Efraim Flashner
2016-04-05 9:52 ` Ludovic Courtès
2016-04-03 17:11 ` Efraim Flashner
2016-04-03 18:00 ` Efraim Flashner
2016-04-04 5:26 ` 宋文武
2016-04-05 11:48 ` Efraim Flashner
2016-04-19 12:25 ` 宋文武
2016-04-03 2:30 ` [PATCH 4/4] gnu: glib: Update to 2.48.0 宋文武
2016-04-03 16:56 ` [PATCH 1/4] gnu: glib: Use modify-phases syntax Efraim Flashner
2016-04-07 12:03 ` Ludovic Courtès
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
List information: https://guix.gnu.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20160403100934.GA6142@debian-netbook \
--to=efraim@flashner.co.il \
--cc=guix-devel@gnu.org \
--cc=iyzsong@gmail.com \
/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.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/guix.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).