From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:42210) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Za5x9-0004RB-Eu for qemu-devel@nongnu.org; Thu, 10 Sep 2015 13:48:08 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Za5x5-0008JJ-6X for qemu-devel@nongnu.org; Thu, 10 Sep 2015 13:48:07 -0400 Received: from mail.codeweavers.com ([216.251.189.131]:43018) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Za5x5-0008Go-19 for qemu-devel@nongnu.org; Thu, 10 Sep 2015 13:48:03 -0400 Received: from jwhite.mn.codeweavers.com ([10.69.137.101]) by mail.codeweavers.com with esmtpsa (TLS1.2:DHE_RSA_AES_128_CBC_SHA1:128) (Exim 4.80) (envelope-from ) id 1Za5wy-0006UR-66 for qemu-devel@nongnu.org; Thu, 10 Sep 2015 12:47:56 -0500 Message-ID: <55F1C24C.1050101@codeweavers.com> Date: Thu, 10 Sep 2015 12:47:56 -0500 From: Jeremy White MIME-Version: 1.0 References: <1440934175-9971-1-git-send-email-marcandre.lureau@redhat.com> In-Reply-To: <1440934175-9971-1-git-send-email-marcandre.lureau@redhat.com> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Subject: Re: [Qemu-devel] [RFC PATCH] libcacard: move it to a standalone project List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org On 08/30/2015 06:29 AM, marcandre.lureau@redhat.com wrote: > From: Marc-André Lureau > > As previously discussed in the thread "libcacard: use the library?", > make libcacard a standalone library and use it if --enable-smartcard. > > I made the following repo out of a qemu filter-branch and added autofoo > build-sys: https://github.com/elmarco/libcacard. Since the original qemu > git tags are there, I think the first release version should be v2.5, > but there is no need to follow the same versions as qemu later on. > > libcacard could use the same infrastructure as the qemu project (mailing > list, git server, reviews etc). I have some experience with libcacard, > and I would be willing to maintain it, however I don't expect to do > more development with it in the near future. > > Signed-off-by: Marc-André Lureau I have reviewed this patch, and the new libcacard module. Modulo a few minor nits (which Marc-André agreed to fix on irc), it all works well. I think this is a sensible change and I'm willing to help with libcacard maintenance, if we were to establish it as a project. Reviewed-by: Jeremy White Cheers, Jeremy > --- > .gitignore | 1 - > Makefile | 3 - > Makefile.objs | 15 - > configure | 51 +- > docs/libcacard.txt | 483 ---------------- > hw/usb/Makefile.objs | 3 +- > hw/usb/ccid-card-passthru.c | 2 +- > libcacard/Makefile | 45 -- > libcacard/cac.c | 414 -------------- > libcacard/cac.h | 31 -- > libcacard/card_7816.c | 757 ------------------------- > libcacard/card_7816.h | 62 --- > libcacard/card_7816t.h | 165 ------ > libcacard/event.c | 103 ---- > libcacard/eventt.h | 29 - > libcacard/libcacard.pc.in | 13 - > libcacard/libcacard.syms | 77 --- > libcacard/link_test.c | 22 - > libcacard/vcard.c | 325 ----------- > libcacard/vcard.h | 86 --- > libcacard/vcard_emul.h | 66 --- > libcacard/vcard_emul_nss.c | 1274 ------------------------------------------- > libcacard/vcard_emul_type.c | 57 -- > libcacard/vcard_emul_type.h | 32 -- > libcacard/vcardt.c | 40 -- > libcacard/vcardt.h | 59 -- > libcacard/vcardt_internal.h | 6 - > libcacard/vevent.h | 27 - > libcacard/vreader.c | 578 -------------------- > libcacard/vreader.h | 55 -- > libcacard/vreadert.h | 24 - > libcacard/vscard_common.h | 178 ------ > libcacard/vscclient.c | 785 -------------------------- > 33 files changed, 19 insertions(+), 5849 deletions(-) > delete mode 100644 docs/libcacard.txt > delete mode 100644 libcacard/Makefile > delete mode 100644 libcacard/cac.c > delete mode 100644 libcacard/cac.h > delete mode 100644 libcacard/card_7816.c > delete mode 100644 libcacard/card_7816.h > delete mode 100644 libcacard/card_7816t.h > delete mode 100644 libcacard/event.c > delete mode 100644 libcacard/eventt.h > delete mode 100644 libcacard/libcacard.pc.in > delete mode 100644 libcacard/libcacard.syms > delete mode 100644 libcacard/link_test.c > delete mode 100644 libcacard/vcard.c > delete mode 100644 libcacard/vcard.h > delete mode 100644 libcacard/vcard_emul.h > delete mode 100644 libcacard/vcard_emul_nss.c > delete mode 100644 libcacard/vcard_emul_type.c > delete mode 100644 libcacard/vcard_emul_type.h > delete mode 100644 libcacard/vcardt.c > delete mode 100644 libcacard/vcardt.h > delete mode 100644 libcacard/vcardt_internal.h > delete mode 100644 libcacard/vevent.h > delete mode 100644 libcacard/vreader.c > delete mode 100644 libcacard/vreader.h > delete mode 100644 libcacard/vreadert.h > delete mode 100644 libcacard/vscard_common.h > delete mode 100644 libcacard/vscclient.c