[Bug 227109] New: Review Request: pmd-3.6-1jpp - Scans Java source code and looks for potential problems
by Red Hat Bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=227109
Summary: Review Request: pmd-3.6-1jpp - Scans Java source code
and looks for potential problems
Product: Fedora Extras
Version: devel
Platform: All
OS/Version: Linux
Status: NEW
Severity: normal
Priority: normal
Component: Package Review
AssignedTo: nobody(a)fedoraproject.org
ReportedBy: rafaels(a)redhat.com
QAContact: fedora-package-review(a)redhat.com
Spec URL: http://people.redhat.com/rafaels/specs/pmd-3.6-1jpp.spec
SRPM URL: ftp://jpackage.hmdc.harvard.edu/JPackage/1.7/generic/SRPMS.free/pmd-3.6-1...
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.
--
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.
15 years, 7 months
[Bug 251071] New: Review Request: StatelessServer - Server for Stateless Clients
by Red Hat Bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=251071
Summary: Review Request: StatelessServer - Server for Stateless
Clients
Product: Fedora
Version: devel
Platform: All
OS/Version: Linux
Status: NEW
Severity: medium
Priority: medium
Component: Package Review
AssignedTo: nobody(a)fedoraproject.org
ReportedBy: law(a)redhat.com
QAContact: extras-qa(a)fedoraproject.org
CC: fedora-package-review@redhat.com,notting(a)redhat.com
Spec URL: http://people.redhat.com/~law/StatelessServer.spec
SRPM URL: http://people.redhat.com/~law/StatelessServer-0.1-1.src.rpm
Description: Minimal setup and configuration of a server for stateless clients. The stateless server is responsible for initial imaging, updates and configuration management of stateless clients.
rpmlint issues errors due to non-standard uid/gids as several files need to be owned by the puppet uid/gid. I'm not sure how to prevent this error from rpmlint (StatelessServer does list the puppet packages as dependencies).
rpmlint also complains about % in the %post section. One of the puppet configuration files updated by the %post section has a %H and %h in it which trips rpmlint.
Finally, the %post actually builds another RPM and installs it into the www directory on the server via cp, this causes rpmlint to balk with a dangerous command in %post error.
I'm open to any suggestions for how to resolve those errors and/or cleaning up the %post section which (IMHO) has to do far too much tweaking of existing configuration files to set up the stateless server.
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
15 years, 7 months