Note: This is a beta release of Red Hat Bugzilla 5.0. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Also email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback here.
Bug 531363 - Review request: WiKIDToken - Token for the WiKID Strong Authentication System
Summary: Review request: WiKIDToken - Token for the WiKID Strong Authentication System
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
Depends On:
TreeView+ depends on / blocked
Reported: 2009-10-27 21:12 UTC by Nick Owen
Modified: 2010-12-17 15:24 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2010-12-17 15:24:49 UTC

Attachments (Terms of Use)

Description Nick Owen 2009-10-27 21:12:38 UTC
Spec file:



The PC token client for the dual-source WiKID Strong Authentication System. Please visit to test, to get the server and for more information.  The Token client takes a PIN, encrypts with a private key and sends it to the server. If the PIN is correct, account active & encryption ok, an OTP is generated and returned to the token client.

It is our intention to also submit an rpm for the server.

This is my first package. I am looking for a sponsor.

Comment 1 Nick Owen 2009-12-04 20:57:30 UTC
Still looking for a sponsor... :)

Comment 2 Jason Tibbitts 2010-11-03 17:30:08 UTC
The SRPM link above is invalid, though the spec link still works.

Comment 4 Jason Tibbitts 2010-11-04 16:54:13 UTC
Java isn't my strong suit; I'll make some comments and hopefully the Java SIG folks can help out.

In %prep, you need to delete any jar and class files present in the zip.  It looks like there's a pile of bundled stuff in there, and we need to make sure that none of it gets into the final package.  Unfortunately I did that and the package fails to build, so it looks like you're relying on that bundled stuff.

Bundling libraries in that manner is not permissible in Fedora.  See and and for more information.

This is a strong blocker, but there are a few other things I see on a first glance.

Please remove the suse stuff from the spec.

Don't reference $RPM_BUILD_DIR at all.

Don't redefine %_prefix.

You probably want %_datadir instead of defining %_sharedir to what %_datadir 
normally contains.

Be careful of using macros in comments.  '#' isn't a comment character in a spec file, and macros are still expanded.  This means that multiline macros will cause problems.

Do you really find it simpler to use "%__chmod" instead of just "chmod"?  If so, you need to be consistent, so use %__mkdir, %__mkdir_p and such as well.  Or just drop the needless complexity and call the commands directly.

Don't mix usage of %buildroot and $RPM_BUILD_ROOT.  Pick one style and use it consistently.

Fedora does not need BuildRoot:, the first line of %install or the %clean section.  (EPEL 4 and 5 still do, though, so it's not mandatory that you remove them.)

%fedora_version, %rhel_version and %centos_version are not defined anywhere in your spec, and they aren't defined in the buildsystem.  You should remove them.  If you want to conditionalize things based on RHEL or Fedora versions, see

Note You need to log in before you can comment on or make changes to this bug.