qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Paolo Bonzini <pbonzini@redhat.com>
To: Laszlo Ersek <lersek@redhat.com>
Cc: libaiqing@huawei.com, mdroth@linux.vnet.ibm.com,
	stefanha@gmail.com, qemu-devel@nongnu.org,
	lcapitulino@redhat.com, vrozenfe@redhat.com,
	Tomoki Sekiyama <tomoki.sekiyama@hds.com>,
	seiji.aguchi@hds.com, areis@redhat.com
Subject: Re: [Qemu-devel] [PATCH v4 04/10] Add a script to extract VSS SDK headers on POSIX system
Date: Tue, 25 Jun 2013 17:01:05 +0200	[thread overview]
Message-ID: <51C9B0B1.7000508@redhat.com> (raw)
In-Reply-To: <51C9B0E5.6040306@redhat.com>

[-- Attachment #1: Type: text/plain, Size: 907 bytes --]

Il 25/06/2013 17:01, Laszlo Ersek ha scritto:
> On 06/06/13 17:06, Tomoki Sekiyama wrote:
> 
>> +if ! command -v msiextract > /dev/null; then
>> +  echo 'msiextract not found. Please install msitools.' >&2
>> +  exit 1
>> +fi
> 
> (This is not a review comment -- I'm trying to test it:)
> 
> What msiextract version (and dependencies, like libgcab, libgsf etc) are
> you using? I'm unable to extract "vsssdk.msi"; msiextract spews a bunch
> of assertion failures.
> 
> 2e39646b7850a12673bc66ade85fece3  setup.exe
> 433eb024ed0c669dd1563d952ca41091  vsssdk.msi
> 
> My versions (RHEL-6.4.z distro):
> - msitools-0.92 (built from source)
> - gcab-0.4 (built from source)
> - libgsf-1.14.15-5.el6 (patch in [1] doesn't seem to help, it only
> changes the kinds of asserts that fail)
> - glib2-2.22.5-7.el6
> - libuuid-2.17.2-12.9.el6_4.3

The attached patch may help building a newer libgsf on RHEL6.

Paolo


[-- Attachment #2: 0001-remove-need-for-glib-2.26.patch --]
[-- Type: text/x-patch, Size: 1794 bytes --]

>From 0eee72d26d06d7087d5d83efbc37564f67489019 Mon Sep 17 00:00:00 2001
From: Paolo Bonzini <pbonzini@redhat.com>
Date: Wed, 16 Jan 2013 19:27:20 +0100
Subject: [PATCH] remove need for glib 2.26

---
 configure           |    2 +-
 configure.in        |    2 +-
 gsf/gsf-timestamp.c |   16 +++++++++-------
 3 files changed, 11 insertions(+), 9 deletions(-)

diff --git a/configure b/configure
index 30aa84d..029f5f9 100755
--- a/configure
+++ b/configure
@@ -12740,7 +12740,7 @@ _ACEOF
 
 libgsf_reqs="
     gobject-2.0 >= 2.16.0
-    glib-2.0 >= 2.26.0
+    glib-2.0 >= 2.22.0
     libxml-2.0 >= 2.4.16
 "
 
diff --git a/configure.in b/configure.in
index a830360..c6c7254 100644
--- a/configure.in
+++ b/configure.in
@@ -71,7 +71,7 @@ ifelse([
 dnl Modules required for libgsf
 libgsf_reqs="
     gobject-2.0 >= 2.16.0
-    glib-2.0 >= 2.26.0
+    glib-2.0 >= 2.22.0
     libxml-2.0 >= 2.4.16
 "
 
diff --git a/gsf/gsf-timestamp.c b/gsf/gsf-timestamp.c
index 9d45f9a..a8e0a44 100644
--- a/gsf/gsf-timestamp.c
+++ b/gsf/gsf-timestamp.c
@@ -136,20 +136,22 @@ int
 gsf_timestamp_load_from_string (GsfTimestamp *stamp, char const *spec)
 {
 	int year, month, day, hour, minute, second;
-	GDateTime *dt;
+	time_t t;
 
 	/* 'YYYY-MM-DDThh:mm:ss' */
 	if (6 != sscanf (spec, "%d-%d-%dT%d:%d:%d",
 			 &year, &month, &day, &hour, &minute, &second))
 		return FALSE;
 
-	dt = g_date_time_new_utc (year, month, day, hour, minute, second);
-	if (!dt)
-		return FALSE;
-
-	stamp->timet = g_date_time_to_unix (dt);
+	if (month < 3) {
+		month += 12;
+		year--;
+	}
+	t = 86400ULL * (day + (153 * month - 457) / 5 + 365 * year + year / 4 - year / 100 + 
+		     year / 400 - 719469);
+	t += 3600 * hour + 60 * minute + second;
 
-	g_date_time_unref (dt);
+	stamp->timet = t;
 	return TRUE;
 }
 
-- 
1.7.1


  reply	other threads:[~2013-06-25 15:01 UTC|newest]

Thread overview: 70+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-06-06 15:06 [Qemu-devel] [PATCH v4 00/10] qemu-ga: fsfreeze on Windows using VSS Tomoki Sekiyama
2013-06-06 15:06 ` [Qemu-devel] [PATCH v4 01/10] configure: Support configuring c++ compiler Tomoki Sekiyama
2013-06-18 10:17   ` Paolo Bonzini
2013-06-25  8:15   ` Laszlo Ersek
2013-06-06 15:06 ` [Qemu-devel] [PATCH v4 02/10] Add c++ keywords to QAPI helper script Tomoki Sekiyama
2013-06-25  8:16   ` Laszlo Ersek
2013-06-06 15:06 ` [Qemu-devel] [PATCH v4 03/10] checkpatch.pl: check .cpp files Tomoki Sekiyama
2013-06-25  8:17   ` Laszlo Ersek
2013-06-06 15:06 ` [Qemu-devel] [PATCH v4 04/10] Add a script to extract VSS SDK headers on POSIX system Tomoki Sekiyama
2013-06-25  8:30   ` Laszlo Ersek
2013-06-25 15:01   ` Laszlo Ersek
2013-06-25 15:01     ` Paolo Bonzini [this message]
2013-06-26 10:52       ` Laszlo Ersek
2013-06-06 15:06 ` [Qemu-devel] [PATCH v4 05/10] qemu-ga: Add configure options to specify path to Windows/VSS SDK Tomoki Sekiyama
2013-06-25 11:16   ` Laszlo Ersek
2013-06-28 10:43     ` Daniel P. Berrange
2013-06-28 10:54       ` Paolo Bonzini
2013-06-28 11:01         ` Daniel P. Berrange
2013-06-28 11:18           ` Paolo Bonzini
2013-06-28 11:30             ` Daniel P. Berrange
2013-06-28 12:18               ` Paolo Bonzini
2013-06-28 11:05       ` Laszlo Ersek
2013-06-06 15:06 ` [Qemu-devel] [PATCH v4 06/10] qemu-ga: Add Windows VSS provider to quiesce applications on fsfreeze Tomoki Sekiyama
2013-06-25 16:03   ` Laszlo Ersek
2013-06-25 16:19     ` Paolo Bonzini
2013-06-25 18:23     ` Tomoki Sekiyama
2013-06-25 18:59       ` Paolo Bonzini
2013-06-25 19:28         ` Tomoki Sekiyama
2013-06-25 19:52       ` Laszlo Ersek
2013-06-26 14:32     ` Laszlo Ersek
2013-06-26 21:27       ` Paolo Bonzini
2013-06-26 22:09       ` Tomoki Sekiyama
2013-06-27 15:01       ` Laszlo Ersek
2013-06-27 22:25         ` Tomoki Sekiyama
2013-06-28  7:05           ` Paolo Bonzini
2013-06-28 10:40             ` Laszlo Ersek
2013-06-28 10:40               ` Paolo Bonzini
2013-06-28 17:18                 ` Tomoki Sekiyama
2013-06-28 10:44           ` Laszlo Ersek
2013-06-25 21:15   ` Paolo Bonzini
2013-06-25 22:31     ` Tomoki Sekiyama
2013-06-26  5:59       ` Paolo Bonzini
2013-06-26 14:13         ` Tomoki Sekiyama
2013-06-30 13:16       ` Gal Hammer
2013-07-01 18:57         ` Tomoki Sekiyama
2013-07-02 12:36           ` Gal Hammer
2013-06-06 15:06 ` [Qemu-devel] [PATCH v4 07/10] qemu-ga: Add Windows VSS requester to quiesce applications and filesystems Tomoki Sekiyama
2013-06-28 18:01   ` Laszlo Ersek
2013-06-28 18:34     ` Laszlo Ersek
2013-06-30  1:21     ` Tomoki Sekiyama
2013-07-01  8:31       ` Laszlo Ersek
2013-06-06 15:06 ` [Qemu-devel] [PATCH v4 08/10] qemu-ga: call Windows VSS requester in fsfreeze command handler Tomoki Sekiyama
2013-07-01 13:29   ` Laszlo Ersek
2013-07-01 17:48     ` Eric Blake
2013-07-01 17:59     ` Tomoki Sekiyama
2013-07-02  6:36       ` Laszlo Ersek
2013-07-02 14:09         ` Luiz Capitulino
2013-07-02 14:44           ` Laszlo Ersek
2013-07-02 15:16             ` Luiz Capitulino
2013-07-02 15:28               ` Laszlo Ersek
2013-07-02 14:58         ` Michael Roth
2013-06-06 15:06 ` [Qemu-devel] [PATCH v4 09/10] qemu-ga: install Windows VSS provider on `qemu-ga -s install' Tomoki Sekiyama
2013-07-01 14:50   ` Laszlo Ersek
2013-07-01 17:59     ` Tomoki Sekiyama
2013-06-06 15:07 ` [Qemu-devel] [PATCH v4 10/10] QMP/qemu-ga-client: make timeout longer for guest-fsfreeze-freeze command Tomoki Sekiyama
2013-06-18 10:17   ` Paolo Bonzini
2013-07-01 15:02   ` Laszlo Ersek
2013-06-10  9:26 ` [Qemu-devel] [PATCH v4 00/10] qemu-ga: fsfreeze on Windows using VSS Stefan Hajnoczi
2013-06-24 19:30 ` Tomoki Sekiyama
2013-06-24 21:13   ` Laszlo Ersek

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=51C9B0B1.7000508@redhat.com \
    --to=pbonzini@redhat.com \
    --cc=areis@redhat.com \
    --cc=lcapitulino@redhat.com \
    --cc=lersek@redhat.com \
    --cc=libaiqing@huawei.com \
    --cc=mdroth@linux.vnet.ibm.com \
    --cc=qemu-devel@nongnu.org \
    --cc=seiji.aguchi@hds.com \
    --cc=stefanha@gmail.com \
    --cc=tomoki.sekiyama@hds.com \
    --cc=vrozenfe@redhat.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.
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 NNTP newsgroup(s).