[Bug 188369] Review Request: ctapi-cyberjack
bugzilla at redhat.com
bugzilla at redhat.com
Wed Apr 26 15:08:16 UTC 2006
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: ctapi-cyberjack
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=188369
bugzilla at redhat.com changed:
What |Removed |Added
----------------------------------------------------------------------------
QAContact|fedora-extras- |fedora-package-
|list at redhat.com |review at redhat.com
------- Additional Comments From j.w.r.degoede at hhs.nl 2006-04-26 11:08 EST -------
Frank,
Please:
-use "pkg-config libpcsclite --variable=usbdropdir" to get the location for
dropping the PCSC-lite driver
-add executable permission to all .so files .so files should always be 755
More even more improtant:
-don't install the lib under the to generic name libctapi.so, this is asking for
conflicts with other packages. Instead name it something like
libctapi-cyberjack.so.0 (watch the versioning!) and put the ctapi lib part
in plain %{_libdir} not the PSCS-lite readers dir, since its generic.
-don't install ctapi.h (where is ctbcs.h?) in /usr/include, but put it in
/usr/include/ctapi-cyberjack.
-create a pkgconfig-file which can be used to get the correct CFLAGS and
LDFLAGS to link against the cyberjack ctapi.
Alternativly come up with some other scheme to allow multiple libctapi's to
co-exist, I think you / we should be able to come up with something better, just
taking the libctapi.so and ctapi.h names in the global namespace is unacceptable
since for example the towitoko drivers provide the same.
Once this is done I would mind sponsering you and doing a full review of this
package, but this problem has to be tackled first.
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.
More information about the Fedora-package-review
mailing list