[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: RFS: acsccid (New Upstream Release)

Here is a review of your package:

I filed #657110 on lintian to check for commented out Vcs-* fields
pointing at collab-maint (which is created by dh_make). Please
consider implementing a patch for it so that others don't make the
same mistake.

You might want to run wrap-and-sort -s so diffs on debian/control are
more readable in future.

Why do you duplicate src/92_pcscd_acsccid.rules as
debian/libacsccid1.udev? I would suggest deleting
debian/libacsccid1.udev and just using the upstream file.

Upstream is hard-coding the path to libpcsclite.pc in configure.ac,
please ask them to stop doing that. AC_PREFIX_DEFAULT should not be
needed either.

debian/watch doesn't need the blank line. I would also suggest using
[\d\.]+ instead of .*

debian/control misses a build-dep on perl. If perl leaves
build-essential then your package will FTBFS.

Please use --parallel when calling dh, otherwise you don't respect
part of Debian Policy 4.9.1.

The upstream README file contains installation info and
authors/copyright/license info, which is not useful for Debian users.
You might want to get upstream to split those out into README.install,
AUTHORS or similar.

The src/openct directory is an embedded code copy. You should ask
upstream to remove it and build-depend on openct. If they are not
willing to do so, then you should do that for Debian. If that isn't
possible for whatever reason, please contact the security team and get
it added to the embedded code copies file:


Should ccid be removed from Debian? acsccid seems like a fork of it.
If it shouldn't be removed, please also get this documented by the
security team, they track forks too.

One warning from dpkg-shlibdeps:

dpkg-shlibdeps: warning:
contains an unresolvable reference to symbol log_xxd: it's probably a
dpkg-shlibdeps: warning: 1 similar warning has been skipped (use -v to see it).

lintian complaints:

X: libacsccid1: shlib-calls-exit



Reply to: