[Bug 463266] Review Request: globalplatform - Access OpenPlatform and GlobalPlatform smart cards library

bugzilla at redhat.com bugzilla at redhat.com
Sun Oct 5 13:16:26 UTC 2008


Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=463266





--- Comment #4 from Mamoru Tasaka <mtasaka at ioa.s.u-tokyo.ac.jp>  2008-10-05 09:16:24 EDT ---
Created an attachment (id=319494)
 --> (https://bugzilla.redhat.com/attachment.cgi?id=319494)
Patch to provide globalplatform.pc

(In reply to comment #3)
> I've addressed all issues (see changelog in spec and spec itself) except the
> pkg-config one as I'm not quite sure how to proceed with this.
> 
> I was digging through the configure output and checking the config.log file to
> see what is going on there. The globalplatform configure script seems to use
> pkg-config to detect the location of pcsc-libs:

Yes, and it is not what I mentioned before. What I said is that it is better
that you create globalplatform.pc(.in) and install it under %_libdir/pkgconfig
so that other applications using globalplatform-devel package will find
the needed CFLAGS easily. How do you think?

Note: When applying this patch, your spec file should be fixed like:
-------------------------------------------------------------------
Name: globalplatform
Version: 5.0.0
......
BuildRequires: openssl-devel >= 0.9.7e
BuildRequires: zlib-devel >= 1.2.3
BuildRequires: pcsc-lite-devel
BuildRequires: automake
BuildRequires: libtool
......
%prep
%setup -q
%patch0 -p1 -b .add

aclocal
autoconf
automake
touch -r Makefile.in configure aclocal.m4

%build
........
%files devel
%defattr(-,root,root,-)
%{_libdir}/*.so
%{_includedir}/GlobalPlatform/
%{_libdir}/pkgconfig/*.pc
-------------------------------------------------

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.




More information about the package-review mailing list