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