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
next prev parent 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).