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=226400
Summary: Merge Review: screen Product: Fedora Extras Version: devel Platform: All OS/Version: Linux Status: NEW Severity: normal Priority: normal Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: nobody@fedoraproject.org QAContact: fedora-package-review@redhat.com CC: mmaslano@redhat.com
Fedora Merge Review: screen
http://cvs.fedora.redhat.com/viewcvs/devel/screen/ Initial Owner: mmaslano@redhat.com
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Merge Review: screen
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226400
ed@eh3.com changed:
What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|nobody@fedoraproject.org |mmaslano@redhat.com Flag|fedora-review? |fedora-review+
------- Additional Comments From ed@eh3.com 2007-02-03 17:18 EST ------- Hi folks, here's a quick review:
good: + source matches upstream (where "upstream" apparently == Juergen, not savannah.gnu.org) + license is GPL v2 -- OK + BuildRoot OK + scriptlets OK + dir ownership looks OK
needswork: - since upstream includes a copy of the license, please include it in %doc
- please list the license as "GPL", not "GPL2"
rpmlint: - rpmlint whines about a few things (the permissions appear to be OK to ignore since screen needs them): http://linux.dell.com/files/fedora/FixBuildRequires/mock-results-core/i386/s...
Please make the two tiny changes above and this package is APPROVED.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Merge Review: screen
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226400
tibbs@math.uh.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Merge Review: screen
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226400
mmaslano@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |RAWHIDE
------- Additional Comments From mmaslano@redhat.com 2007-02-06 07:16 EST ------- Thank you for your review.
package-review@lists.fedoraproject.org