Bug 445915
| Summary: | Review Request: flute - Java CSS parser using SAC | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Caolan McNamara <caolanm> |
| Component: | Package Review | Assignee: | Lubomir Rintel <lkundrak> |
| Status: | CLOSED RAWHIDE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
| Severity: | medium | Docs Contact: | |
| Priority: | medium | ||
| Version: | rawhide | CC: | fedora-package-review, notting |
| Target Milestone: | --- | Flags: | lkundrak:
fedora-review+
kevin: fedora-cvs+ |
| Target Release: | --- | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Fixed In Version: | Doc Type: | Bug Fix | |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2008-07-23 17:58:54 UTC | Type: | --- |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Embargoed: | |||
|
Description
Caolan McNamara
2008-05-09 20:03:40 UTC
Crap, I happened to not notice the jpp thing in the release before I took this package, and I was trying to avoid reviewing any packages that used it. So I'll return this to the queue. Thanks for the package!
Please remove the jpp tag. Revision: 4%{?dist} will be just fine I guess.
Also, the %description sucks, you can probably turn it into something more
descriptive.
The package seems fine in other respects, rpmlint is happy, so is the QA script
and so am I. I'll approve it once the mock build finishes for me.
Okay, built in mock. rpmlint passes for resulting rpms, nothing strange in build.log. Dependencies seem sane. APPROVED And please address the issue with Release: and %description mentioned in the comment above this before you commit. Thanks! New Package CVS Request ======================= Package Name: flute Short Description: Java CSS parser Owners: caolanm Branches: InitialCC: Cvsextras Commits: yes cvs done. |