qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH 0/2] qga: vss-win32: Fix build failure
@ 2014-03-26 18:28 Tomoki Sekiyama
  2014-03-26 18:28 ` [Qemu-devel] [PATCH 1/2] Makefile: add qga-vss-dll-obj-y to nested variables Tomoki Sekiyama
  2014-03-26 18:28 ` [Qemu-devel] [PATCH 2/2] vss-win32: Fix build with mingw64-headers-3.1.0 Tomoki Sekiyama
  0 siblings, 2 replies; 3+ messages in thread
From: Tomoki Sekiyama @ 2014-03-26 18:28 UTC (permalink / raw)
  To: qemu-devel; +Cc: pbonzini, famz, mdroth, mitsuhiro.tanino

Hello,

These patches are to fix recent build failures of qemu-guest-agent for
Windows when fsfreeze using VSS is enabled by:

  ./configure --cross-prefix=i686-w64-mingw32- --with-vss-sdk=yes

---
Tomoki Sekiyama (2):
      Makefile: add qga-vss-dll-obj-y to nested variables
      vss-win32: Fix build with mingw64-headers-3.1.0


 Makefile                  |    1 +
 qga/vss-win32/install.cpp |    3 +++
 2 files changed, 4 insertions(+)

-- 
Tomoki Sekiyama

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [Qemu-devel] [PATCH 1/2] Makefile: add qga-vss-dll-obj-y to nested variables
  2014-03-26 18:28 [Qemu-devel] [PATCH 0/2] qga: vss-win32: Fix build failure Tomoki Sekiyama
@ 2014-03-26 18:28 ` Tomoki Sekiyama
  2014-03-26 18:28 ` [Qemu-devel] [PATCH 2/2] vss-win32: Fix build with mingw64-headers-3.1.0 Tomoki Sekiyama
  1 sibling, 0 replies; 3+ messages in thread
From: Tomoki Sekiyama @ 2014-03-26 18:28 UTC (permalink / raw)
  To: qemu-devel; +Cc: pbonzini, famz, mdroth, mitsuhiro.tanino

The build rule for qga/vss-win32/qga-vss.dll is broken by commit
ba1183da9a10b94611cad88c44a5c6df005f9b55, because it misses
qga-vss-dll-obj-y in the list of nested variables.
This fixes build of qga-vss.dll by adding qga-vss-dll-obj-y to the list.

Signed-off-by: Tomoki Sekiyama <tomoki.sekiyama@hds.com>
---
 Makefile |    1 +
 1 file changed, 1 insertion(+)

diff --git a/Makefile b/Makefile
index ec74039..b6b09a9 100644
--- a/Makefile
+++ b/Makefile
@@ -133,6 +133,7 @@ dummy := $(call unnest-vars,, \
                 stub-obj-y \
                 util-obj-y \
                 qga-obj-y \
+                qga-vss-dll-obj-y \
                 block-obj-y \
                 block-obj-m \
                 common-obj-y \

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [Qemu-devel] [PATCH 2/2] vss-win32: Fix build with mingw64-headers-3.1.0
  2014-03-26 18:28 [Qemu-devel] [PATCH 0/2] qga: vss-win32: Fix build failure Tomoki Sekiyama
  2014-03-26 18:28 ` [Qemu-devel] [PATCH 1/2] Makefile: add qga-vss-dll-obj-y to nested variables Tomoki Sekiyama
@ 2014-03-26 18:28 ` Tomoki Sekiyama
  1 sibling, 0 replies; 3+ messages in thread
From: Tomoki Sekiyama @ 2014-03-26 18:28 UTC (permalink / raw)
  To: qemu-devel; +Cc: pbonzini, famz, mdroth, mitsuhiro.tanino

In mingw64-headers-3.1.0, definition of _com_issue_error() is added, which
conflicts with definition in install.cpp. This adds version checking for
mingw headers to disable the definition when the headers>=3.1 is used.

Signed-off-by: Tomoki Sekiyama <tomoki.sekiyama@hds.com>
---
 qga/vss-win32/install.cpp |    3 +++
 1 file changed, 3 insertions(+)

diff --git a/qga/vss-win32/install.cpp b/qga/vss-win32/install.cpp
index b791a6c..b0e4426 100644
--- a/qga/vss-win32/install.cpp
+++ b/qga/vss-win32/install.cpp
@@ -75,10 +75,13 @@ static void errmsg_dialog(DWORD err, const char *text, const char *opt = "")
 
 #define chk(status) _chk(hr, status, "Failed to " #status, out)
 
+#if !defined(__MINGW64_VERSION_MAJOR) || !defined(__MINGW64_VERSION_MINOR) || \
+    __MINGW64_VERSION_MAJOR * 100 + __MINGW64_VERSION_MINOR < 301
 void __stdcall _com_issue_error(HRESULT hr)
 {
     errmsg(hr, "Unexpected error in COM");
 }
+#endif
 
 template<class T>
 HRESULT put_Value(ICatalogObject *pObj, LPCWSTR name, T val)

^ permalink raw reply related	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2014-03-26 18:29 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-03-26 18:28 [Qemu-devel] [PATCH 0/2] qga: vss-win32: Fix build failure Tomoki Sekiyama
2014-03-26 18:28 ` [Qemu-devel] [PATCH 1/2] Makefile: add qga-vss-dll-obj-y to nested variables Tomoki Sekiyama
2014-03-26 18:28 ` [Qemu-devel] [PATCH 2/2] vss-win32: Fix build with mingw64-headers-3.1.0 Tomoki Sekiyama

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).