* [Qemu-devel] [PATCH v2] libcacard: stop including qemu-common.h
@ 2015-04-27 13:27 Michael Tokarev
2015-04-27 13:29 ` Paolo Bonzini
2015-05-06 9:23 ` Laurent Desnogues
0 siblings, 2 replies; 5+ messages in thread
From: Michael Tokarev @ 2015-04-27 13:27 UTC (permalink / raw)
To: qemu-devel; +Cc: qemu-trivial, Paolo Bonzini, Michael Tokarev
From: Paolo Bonzini <pbonzini@redhat.com>
This is a small step towards making libcacard standalone.
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
---
v2: update vscclient.c too, use unistd.h on !WIN32 and getopt.h on *nix
libcacard/cac.c | 5 ++++-
libcacard/card_7816.c | 4 +++-
libcacard/event.c | 2 +-
libcacard/vcard.c | 4 +++-
libcacard/vcard_emul_nss.c | 2 +-
libcacard/vreader.c | 4 +++-
libcacard/vscclient.c | 8 +++++++-
7 files changed, 22 insertions(+), 7 deletions(-)
diff --git a/libcacard/cac.c b/libcacard/cac.c
index f38fdce..bc84534 100644
--- a/libcacard/cac.c
+++ b/libcacard/cac.c
@@ -5,7 +5,10 @@
* See the COPYING.LIB file in the top-level directory.
*/
-#include "qemu-common.h"
+#include "glib-compat.h"
+
+#include <string.h>
+#include <stdbool.h>
#include "cac.h"
#include "vcard.h"
diff --git a/libcacard/card_7816.c b/libcacard/card_7816.c
index 814fa16..22fd334 100644
--- a/libcacard/card_7816.c
+++ b/libcacard/card_7816.c
@@ -5,7 +5,9 @@
* See the COPYING.LIB file in the top-level directory.
*/
-#include "qemu-common.h"
+#include "glib-compat.h"
+
+#include <string.h>
#include "vcard.h"
#include "vcard_emul.h"
diff --git a/libcacard/event.c b/libcacard/event.c
index 4c551e4..63f4057 100644
--- a/libcacard/event.c
+++ b/libcacard/event.c
@@ -5,7 +5,7 @@
* See the COPYING.LIB file in the top-level directory.
*/
-#include "qemu-common.h"
+#include "glib-compat.h"
#include "vcard.h"
#include "vreader.h"
diff --git a/libcacard/vcard.c b/libcacard/vcard.c
index d140a8e..1a87208 100644
--- a/libcacard/vcard.c
+++ b/libcacard/vcard.c
@@ -5,7 +5,9 @@
* See the COPYING.LIB file in the top-level directory.
*/
-#include "qemu-common.h"
+#include "glib-compat.h"
+
+#include <string.h>
#include "vcard.h"
#include "vcard_emul.h"
diff --git a/libcacard/vcard_emul_nss.c b/libcacard/vcard_emul_nss.c
index 950edee..6955f69 100644
--- a/libcacard/vcard_emul_nss.c
+++ b/libcacard/vcard_emul_nss.c
@@ -25,7 +25,7 @@
#include <prthread.h>
#include <secerr.h>
-#include "qemu-common.h"
+#include "glib-compat.h"
#include "vcard.h"
#include "card_7816t.h"
diff --git a/libcacard/vreader.c b/libcacard/vreader.c
index 0315dd8..9725f46 100644
--- a/libcacard/vreader.c
+++ b/libcacard/vreader.c
@@ -10,7 +10,9 @@
#endif
#define G_LOG_DOMAIN "libcacard"
-#include "qemu-common.h"
+#include "glib-compat.h"
+
+#include <string.h>
#include "vcard.h"
#include "vcard_emul.h"
diff --git a/libcacard/vscclient.c b/libcacard/vscclient.c
index fa6041d..0652684 100644
--- a/libcacard/vscclient.c
+++ b/libcacard/vscclient.c
@@ -10,14 +10,20 @@
* See the COPYING.LIB file in the top-level directory.
*/
+#include <stdio.h>
+#include <stdlib.h>
+#include <string.h>
#ifndef _WIN32
#include <sys/socket.h>
#include <netinet/in.h>
#include <netdb.h>
+#include <unistd.h>
#define closesocket(x) close(x)
+#else
+#include <getopt.h>
#endif
-#include "qemu-common.h"
+#include "glib-compat.h"
#include "vscard_common.h"
--
2.1.4
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [Qemu-devel] [PATCH v2] libcacard: stop including qemu-common.h
2015-04-27 13:27 [Qemu-devel] [PATCH v2] libcacard: stop including qemu-common.h Michael Tokarev
@ 2015-04-27 13:29 ` Paolo Bonzini
2015-05-06 9:23 ` Laurent Desnogues
1 sibling, 0 replies; 5+ messages in thread
From: Paolo Bonzini @ 2015-04-27 13:29 UTC (permalink / raw)
To: Michael Tokarev, qemu-devel; +Cc: qemu-trivial
On 27/04/2015 15:27, Michael Tokarev wrote:
> From: Paolo Bonzini <pbonzini@redhat.com>
>
> This is a small step towards making libcacard standalone.
>
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
> ---
> v2: update vscclient.c too, use unistd.h on !WIN32 and getopt.h on *nix
Thanks very much Michael. The vscclient.c parts look good, you may want
to add your changes before your S-o-b line.
Paolo
>
> libcacard/cac.c | 5 ++++-
> libcacard/card_7816.c | 4 +++-
> libcacard/event.c | 2 +-
> libcacard/vcard.c | 4 +++-
> libcacard/vcard_emul_nss.c | 2 +-
> libcacard/vreader.c | 4 +++-
> libcacard/vscclient.c | 8 +++++++-
> 7 files changed, 22 insertions(+), 7 deletions(-)
>
> diff --git a/libcacard/cac.c b/libcacard/cac.c
> index f38fdce..bc84534 100644
> --- a/libcacard/cac.c
> +++ b/libcacard/cac.c
> @@ -5,7 +5,10 @@
> * See the COPYING.LIB file in the top-level directory.
> */
>
> -#include "qemu-common.h"
> +#include "glib-compat.h"
> +
> +#include <string.h>
> +#include <stdbool.h>
>
> #include "cac.h"
> #include "vcard.h"
> diff --git a/libcacard/card_7816.c b/libcacard/card_7816.c
> index 814fa16..22fd334 100644
> --- a/libcacard/card_7816.c
> +++ b/libcacard/card_7816.c
> @@ -5,7 +5,9 @@
> * See the COPYING.LIB file in the top-level directory.
> */
>
> -#include "qemu-common.h"
> +#include "glib-compat.h"
> +
> +#include <string.h>
>
> #include "vcard.h"
> #include "vcard_emul.h"
> diff --git a/libcacard/event.c b/libcacard/event.c
> index 4c551e4..63f4057 100644
> --- a/libcacard/event.c
> +++ b/libcacard/event.c
> @@ -5,7 +5,7 @@
> * See the COPYING.LIB file in the top-level directory.
> */
>
> -#include "qemu-common.h"
> +#include "glib-compat.h"
>
> #include "vcard.h"
> #include "vreader.h"
> diff --git a/libcacard/vcard.c b/libcacard/vcard.c
> index d140a8e..1a87208 100644
> --- a/libcacard/vcard.c
> +++ b/libcacard/vcard.c
> @@ -5,7 +5,9 @@
> * See the COPYING.LIB file in the top-level directory.
> */
>
> -#include "qemu-common.h"
> +#include "glib-compat.h"
> +
> +#include <string.h>
>
> #include "vcard.h"
> #include "vcard_emul.h"
> diff --git a/libcacard/vcard_emul_nss.c b/libcacard/vcard_emul_nss.c
> index 950edee..6955f69 100644
> --- a/libcacard/vcard_emul_nss.c
> +++ b/libcacard/vcard_emul_nss.c
> @@ -25,7 +25,7 @@
> #include <prthread.h>
> #include <secerr.h>
>
> -#include "qemu-common.h"
> +#include "glib-compat.h"
>
> #include "vcard.h"
> #include "card_7816t.h"
> diff --git a/libcacard/vreader.c b/libcacard/vreader.c
> index 0315dd8..9725f46 100644
> --- a/libcacard/vreader.c
> +++ b/libcacard/vreader.c
> @@ -10,7 +10,9 @@
> #endif
> #define G_LOG_DOMAIN "libcacard"
>
> -#include "qemu-common.h"
> +#include "glib-compat.h"
> +
> +#include <string.h>
>
> #include "vcard.h"
> #include "vcard_emul.h"
> diff --git a/libcacard/vscclient.c b/libcacard/vscclient.c
> index fa6041d..0652684 100644
> --- a/libcacard/vscclient.c
> +++ b/libcacard/vscclient.c
> @@ -10,14 +10,20 @@
> * See the COPYING.LIB file in the top-level directory.
> */
>
> +#include <stdio.h>
> +#include <stdlib.h>
> +#include <string.h>
> #ifndef _WIN32
> #include <sys/socket.h>
> #include <netinet/in.h>
> #include <netdb.h>
> +#include <unistd.h>
> #define closesocket(x) close(x)
> +#else
> +#include <getopt.h>
> #endif
>
> -#include "qemu-common.h"
> +#include "glib-compat.h"
>
> #include "vscard_common.h"
>
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [Qemu-devel] [PATCH v2] libcacard: stop including qemu-common.h
2015-04-27 13:27 [Qemu-devel] [PATCH v2] libcacard: stop including qemu-common.h Michael Tokarev
2015-04-27 13:29 ` Paolo Bonzini
@ 2015-05-06 9:23 ` Laurent Desnogues
2015-05-06 10:05 ` Michael Tokarev
1 sibling, 1 reply; 5+ messages in thread
From: Laurent Desnogues @ 2015-05-06 9:23 UTC (permalink / raw)
To: Michael Tokarev; +Cc: qemu-trivial, Paolo Bonzini, qemu-devel@nongnu.org
Hello,
On Mon, Apr 27, 2015 at 3:27 PM, Michael Tokarev <mjt@tls.msk.ru> wrote:
> From: Paolo Bonzini <pbonzini@redhat.com>
>
> This is a small step towards making libcacard standalone.
on my system the removal of qemu-common.h inclusion broke compilation
due to assert being used in glib-compat.h. A fix might be to include
assert.h in glib-compat.h.
Thanks,
Laurent
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
> ---
> v2: update vscclient.c too, use unistd.h on !WIN32 and getopt.h on *nix
>
> libcacard/cac.c | 5 ++++-
> libcacard/card_7816.c | 4 +++-
> libcacard/event.c | 2 +-
> libcacard/vcard.c | 4 +++-
> libcacard/vcard_emul_nss.c | 2 +-
> libcacard/vreader.c | 4 +++-
> libcacard/vscclient.c | 8 +++++++-
> 7 files changed, 22 insertions(+), 7 deletions(-)
>
> diff --git a/libcacard/cac.c b/libcacard/cac.c
> index f38fdce..bc84534 100644
> --- a/libcacard/cac.c
> +++ b/libcacard/cac.c
> @@ -5,7 +5,10 @@
> * See the COPYING.LIB file in the top-level directory.
> */
>
> -#include "qemu-common.h"
> +#include "glib-compat.h"
> +
> +#include <string.h>
> +#include <stdbool.h>
>
> #include "cac.h"
> #include "vcard.h"
> diff --git a/libcacard/card_7816.c b/libcacard/card_7816.c
> index 814fa16..22fd334 100644
> --- a/libcacard/card_7816.c
> +++ b/libcacard/card_7816.c
> @@ -5,7 +5,9 @@
> * See the COPYING.LIB file in the top-level directory.
> */
>
> -#include "qemu-common.h"
> +#include "glib-compat.h"
> +
> +#include <string.h>
>
> #include "vcard.h"
> #include "vcard_emul.h"
> diff --git a/libcacard/event.c b/libcacard/event.c
> index 4c551e4..63f4057 100644
> --- a/libcacard/event.c
> +++ b/libcacard/event.c
> @@ -5,7 +5,7 @@
> * See the COPYING.LIB file in the top-level directory.
> */
>
> -#include "qemu-common.h"
> +#include "glib-compat.h"
>
> #include "vcard.h"
> #include "vreader.h"
> diff --git a/libcacard/vcard.c b/libcacard/vcard.c
> index d140a8e..1a87208 100644
> --- a/libcacard/vcard.c
> +++ b/libcacard/vcard.c
> @@ -5,7 +5,9 @@
> * See the COPYING.LIB file in the top-level directory.
> */
>
> -#include "qemu-common.h"
> +#include "glib-compat.h"
> +
> +#include <string.h>
>
> #include "vcard.h"
> #include "vcard_emul.h"
> diff --git a/libcacard/vcard_emul_nss.c b/libcacard/vcard_emul_nss.c
> index 950edee..6955f69 100644
> --- a/libcacard/vcard_emul_nss.c
> +++ b/libcacard/vcard_emul_nss.c
> @@ -25,7 +25,7 @@
> #include <prthread.h>
> #include <secerr.h>
>
> -#include "qemu-common.h"
> +#include "glib-compat.h"
>
> #include "vcard.h"
> #include "card_7816t.h"
> diff --git a/libcacard/vreader.c b/libcacard/vreader.c
> index 0315dd8..9725f46 100644
> --- a/libcacard/vreader.c
> +++ b/libcacard/vreader.c
> @@ -10,7 +10,9 @@
> #endif
> #define G_LOG_DOMAIN "libcacard"
>
> -#include "qemu-common.h"
> +#include "glib-compat.h"
> +
> +#include <string.h>
>
> #include "vcard.h"
> #include "vcard_emul.h"
> diff --git a/libcacard/vscclient.c b/libcacard/vscclient.c
> index fa6041d..0652684 100644
> --- a/libcacard/vscclient.c
> +++ b/libcacard/vscclient.c
> @@ -10,14 +10,20 @@
> * See the COPYING.LIB file in the top-level directory.
> */
>
> +#include <stdio.h>
> +#include <stdlib.h>
> +#include <string.h>
> #ifndef _WIN32
> #include <sys/socket.h>
> #include <netinet/in.h>
> #include <netdb.h>
> +#include <unistd.h>
> #define closesocket(x) close(x)
> +#else
> +#include <getopt.h>
> #endif
>
> -#include "qemu-common.h"
> +#include "glib-compat.h"
>
> #include "vscard_common.h"
>
> --
> 2.1.4
>
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [Qemu-devel] [PATCH v2] libcacard: stop including qemu-common.h
2015-05-06 9:23 ` Laurent Desnogues
@ 2015-05-06 10:05 ` Michael Tokarev
2015-05-06 12:01 ` Laurent Desnogues
0 siblings, 1 reply; 5+ messages in thread
From: Michael Tokarev @ 2015-05-06 10:05 UTC (permalink / raw)
To: Laurent Desnogues; +Cc: qemu-trivial, Paolo Bonzini, qemu-devel@nongnu.org
06.05.2015 12:23, Laurent Desnogues wrote:
> Hello,
>
> On Mon, Apr 27, 2015 at 3:27 PM, Michael Tokarev <mjt@tls.msk.ru> wrote:
>> From: Paolo Bonzini <pbonzini@redhat.com>
>>
>> This is a small step towards making libcacard standalone.
>
> on my system the removal of qemu-common.h inclusion broke compilation
> due to assert being used in glib-compat.h.
Interesting. What kind of build environment is that? I compile-tested on
several platforms, all went fine.. ;)
> A fix might be to include
> assert.h in glib-compat.h.
I prefer s/assert/g_assert/ in glib-compat.h.
Thanks,
/mjt
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [Qemu-devel] [PATCH v2] libcacard: stop including qemu-common.h
2015-05-06 10:05 ` Michael Tokarev
@ 2015-05-06 12:01 ` Laurent Desnogues
0 siblings, 0 replies; 5+ messages in thread
From: Laurent Desnogues @ 2015-05-06 12:01 UTC (permalink / raw)
To: Michael Tokarev; +Cc: qemu-trivial, Paolo Bonzini, qemu-devel@nongnu.org
On Wed, May 6, 2015 at 12:05 PM, Michael Tokarev <mjt@tls.msk.ru> wrote:
> 06.05.2015 12:23, Laurent Desnogues wrote:
>> Hello,
>>
>> On Mon, Apr 27, 2015 at 3:27 PM, Michael Tokarev <mjt@tls.msk.ru> wrote:
>>> From: Paolo Bonzini <pbonzini@redhat.com>
>>>
>>> This is a small step towards making libcacard standalone.
>>
>> on my system the removal of qemu-common.h inclusion broke compilation
>> due to assert being used in glib-compat.h.
>
> Interesting. What kind of build environment is that? I compile-tested on
> several platforms, all went fine.. ;)
That's a CentOS 6.6 machine with glib2 2.28.8.
>> A fix might be to include
>> assert.h in glib-compat.h.
>
> I prefer s/assert/g_assert/ in glib-compat.h.
That indeed looks better :-)
Thanks,
Laurent
> Thanks,
>
> /mjt
>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2015-05-06 12:01 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-04-27 13:27 [Qemu-devel] [PATCH v2] libcacard: stop including qemu-common.h Michael Tokarev
2015-04-27 13:29 ` Paolo Bonzini
2015-05-06 9:23 ` Laurent Desnogues
2015-05-06 10:05 ` Michael Tokarev
2015-05-06 12:01 ` Laurent Desnogues
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).