From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Petazzoni Date: Tue, 12 Jul 2011 09:17:30 +0200 Subject: [Buildroot] [PATCH] pcsc-lite and ccid support In-Reply-To: References: <1310452089-23322-1-git-send-email-ruckuus@gmail.com> <20110712084323.74ff5c0c@skate> Message-ID: <20110712091730.6372c872@skate> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: buildroot@busybox.net Hello Dwi, Le Tue, 12 Jul 2011 14:03:01 +0700, Dwi Sasongko Supriyadi a ?crit : > > The latest ccid release is 1.4.4, dated May 2011, which doesn't > > seem to be that old. In general, we prefer using stable releases > > rather than SVN versions when possible. Do you have a good reason > > for choosing a specific SVN version instead of the latest project > > release ? > > > > It should have been using the latest version. > > The reason is somehow fuzzy for me, and I need your advise. The idea > is to keep the changes in .mk file as less as possible when there is > new release, so in above case we only need to change the CCID_VERSION > based on SVN version. > > I agree to use the stable release which is in this case will look as > follow: > > CCID_VERSION = 1.4.4 > CCID_SOURCE = ccid-$(CCID_VERSION).tar.bz2 > CCID_SITE = https://alioth.debian.org/frs/download.php/3579 > > Comparing to the previous release: 1.4.3 > CCID_VERSION = 1.4.3 > CCID_SOURCE = ccid-$(CCID_VERSION).tar.bz2 > CCID_SITE = https://alioth.debian.org/frs/download.php/3535 > > There will be changes in CCID_VERSION and CCID_SITE when upgrading > from 1.4.3 to 1.4.4 > > Please advise, should this is not reasonable Well, I think updating two lines (CCID_VERSION and CCID_SITE) is reasonable when the package version needs to be changed. It would of course have been better if Alioth did not use this strange (and stupid?) URL scheme, but that's the way it is. > >> +CCID_AUTORECONF = YES Presumably, if you use the tarball versions, the ./configure script and Makefile.in files will already be available, so this line could be removed. > >> +########################################################## > >> +# > >> +# PCSC-Lite > >> +# > >> +# ######################################################## > >> +PCSC_LITE_VERSION = 5853 > >> +PCSC_LITE_SITE = svn://anonscm.debian.org/pcsclite/trunk/PCSC > > > > Same question, SVN version really needed ? > > > > Actually no. When checking out from SVN it will always get the latest > version. No, with what you have written, it will always get revision 5853, and nothing else. It'd be better to use the tarball version here as well. > >> +PCSC_LITE_AUTORECONF = YES And remove this line. > Many thanks for your comments. This is my first attempt on buildroot, > I will try my best to accomplish it. And your first attempt is very good! Thanks for contributing! Regards, Thomas -- Thomas Petazzoni, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com