[Bug 445410] New: Review Request: sac - Java standard interface for CSS parser

[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/show_bug.cgi?id=445410

Summary: Review Request: sac - Java standard interface for CSS
                    parser
           Product: Fedora
           Version: rawhide
          Platform: All
        OS/Version: Linux
            Status: NEW
          Severity: medium
          Priority: medium
         Component: Package Review
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]
         QAContact: [e-mail]
                CC: [e-mail],[e-mail]

Spec URL: http://people.redhat.com/caolanm/jfreereport/sac.spec
SRPM URL: http://people.redhat.com/caolanm/jfreereport/sac-1.3-3jpp.fc9.src.rpm
Description: Java standard interface for CSS parser

and is a jfreereport dependency (which itself will be an openoffice.org) dependency

[Bug 445410] Review Request: sac - Java standard interface for CSS parser

[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.

Summary: Review Request: sac - Java standard interface for CSS parser

https://bugzilla.redhat.com/show_bug.cgi?id=445410

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

------- Additional Comments From [e-mail]  2008-05-06 15:04 EST -------
It's not exactly clear to me if I want e.g. sac for rawhide/F-10 and it's
already packaged by jpackage if I should use a n-v-r which is >= the equivalent
jpackage n-v-r or not.

[Bug 445410] Review Request: sac - Java standard interface for CSS parser

[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.

Summary: Review Request: sac - Java standard interface for CSS parser

https://bugzilla.redhat.com/show_bug.cgi?id=445410

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|[e-mail]    |[e-mail]

------- Additional Comments From [e-mail]  2008-05-07 09:03 EST -------
See this for release numbering:

http://fedoraproject.org/wiki/Packaging/JPackagePolicy

I'll take this one.

[Bug 445410] Review Request: sac - Java standard interface for CSS parser

[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.

Summary: Review Request: sac - Java standard interface for CSS parser

https://bugzilla.redhat.com/show_bug.cgi?id=445410

------- Additional Comments From [e-mail]  2008-05-09 12:11 EST -------
The specfile looks fine except for:

- fix the release as per the URL in comment #2
- description a bit poorly worded
- don't drop the JPackage changelog (if there is one)

It sucks that they don't have a build.xml but the simple one written by the
JPackage packager is fine.

Don't worry about the rpmlint warning about the group.

[Bug 445410] Review Request: sac - Java standard interface for CSS parser

[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.

Summary: Review Request: sac - Java standard interface for CSS parser

https://bugzilla.redhat.com/show_bug.cgi?id=445410

------- Additional Comments From [e-mail]  2008-05-09 12:25 EST -------
So, how about...

http://people.redhat.com/caolanm/jfreereport/sac.spec
http://people.redhat.com/caolanm/jfreereport/sac-1.3-3jpp.1.fc9.src.rpm

[Bug 445410] Review Request: sac - Java standard interface for CSS parser

[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.

Summary: Review Request: sac - Java standard interface for CSS parser

https://bugzilla.redhat.com/show_bug.cgi?id=445410

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
               Flag|                            |fedora-review+

------- Additional Comments From [e-mail]  2008-05-09 12:39 EST -------
Looks good to me.  Approved.

[Bug 445410] Review Request: sac - Java standard interface for CSS parser

[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.

Summary: Review Request: sac - Java standard interface for CSS parser

https://bugzilla.redhat.com/show_bug.cgi?id=445410

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|                            |fedora-cvs?

------- Additional Comments From [e-mail]  2008-05-09 14:03 EST -------
New Package CVS Request
=======================
Package Name: sac
Short Description: Java standard interface for CSS parser
Owners: caolanm
Branches: 
InitialCC:
Cvsextras Commits: yes

[Bug 445410] Review Request: sac - Java standard interface for CSS parser

[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.

Summary: Review Request: sac - Java standard interface for CSS parser

https://bugzilla.redhat.com/show_bug.cgi?id=445410

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-cvs?                 |fedora-cvs+

------- Additional Comments From [e-mail]  2008-05-09 15:26 EST -------
cvs done.

[Bug 445410] Review Request: sac - Java standard interface for CSS parser

[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.

Summary: Review Request: sac - Java standard interface for CSS parser

https://bugzilla.redhat.com/show_bug.cgi?id=445410

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |CLOSED
         Resolution|                            |RAWHIDE