Message ID | 1430141220-20123-1-git-send-email-mjt@msgid.tls.msk.ru |
---|---|
State | New |
Headers | show |
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" > >
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 > >
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
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 >
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"