This service will be undergoing maintenance at 00:00 UTC, 2017-10-23 It is expected to last about 30 minutes
Bug 227109 - Review Request: pmd-3.6-1jpp - Scans Java source code and looks for potential problems
Review Request: pmd-3.6-1jpp - Scans Java source code and looks for potential...
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
All Linux
medium Severity medium
: ---
: ---
Assigned To: Matt Wringe
Fedora Package Reviews List
Depends On: 227068
  Show dependency treegraph
Reported: 2007-02-02 12:55 EST by Rafael H. Schloming
Modified: 2014-12-01 18:14 EST (History)
4 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2007-03-20 10:57:58 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
dbhole: fedora‑review+
wtogami: fedora‑cvs+

Attachments (Terms of Use)

  None (edit)
Description Rafael H. Schloming 2007-02-02 12:55:15 EST
Spec URL:
Description: PMD scans Java source code and looks for potential
problems like:
+ Unused local variables
+ Empty catch blocks
+ Unused parameters
+ Empty 'if' statements
+ Duplicate import statements
+ Unused private methods
+ Classes which could be Singletons
+ Short/long variable and method names
PMD has plugins for JDeveloper, JEdit, JBuilder,
NetBeans/Sun ONE Studio, IntelliJ IDEA, TextPad,
Maven, Ant, Eclipse, Gel, and Emacs.

Documentation for pmd.

Javadoc for pmd.
Comment 1 Jeff Johnston 2007-02-16 17:19:07 EST
All problems below taken care of except binary build which is
blocked due to jaxen requirement.

X - remove defines at top
  - fill in Name, Version, and Release fields


X - release should be of form Xjpp.Y%{?dist}


X - instructions for source tar are incomplete


X - checking out source from CVS as specified and
    tarring does not result in same md5sum

Fixed.  Source replaced.

X correct buildroot
 - should be:
   %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)


X Vendor tag should not be used


X Distribution tag should not be used


X package successfully compiles and builds on at least x86
  - waiting on jaxen

X make sure lines are <= 80 characters


X run rpmlint on the src RPMs
W: pmd non-standard-group Development/Testing
E: pmd no-cleaning-of-buildroot %install
W: pmd mixed-use-of-spaces-and-tabs (spaces: line 9, tab: line 30)

Comment 2 Andrew Overholt 2007-03-13 11:26:40 EDT
Small remaining issues:

. Get rid of the double hash marks - just use single ones
. Get rid of the cvs login - it's unnecessary

I can't duplicate the md5sum of the generated source tarball but a recursive
diff on the expanded contents shows no differences.

I've added the blocker on jaxen.  Everything else is fine.  Thanks.
Comment 4 Andrew Overholt 2007-03-13 16:37:15 EDT

Thanks, Jeff.
Comment 5 Matt Wringe 2007-03-13 17:06:53 EDT
New Package CVS Request
Package Name: pmd
Short Description: Scans Java source code and looks for potential problems
Branches: devel
Comment 6 Deepak Bhole 2007-03-14 17:37:54 EDT
This package does not build in mock:
Comment 7 Deepak Bhole 2007-03-15 10:00:28 EDT
Fixed a BR. New spec and srpm:

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