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=448567
Summary: Review Request: xorg-x11-drv-suntcx - X driver for suntcx hardware Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: dennis@ausil.us QAContact: extras-qa@fedoraproject.org CC: fedora-package-review@redhat.com,notting@redhat.com
Spec URL: http://ausil.us/packages/xorg-x11-drv-suntcx.spec SRPM URL: http://ausil.us/packages/xorg-x11-drv-suntcx-1.1.0-2.fc9.src.rpm Description: X driver for suntcx hardware
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: xorg-x11-drv-suntcx - X driver for suntcx hardware
https://bugzilla.redhat.com/show_bug.cgi?id=448567
dennis@ausil.us changed:
What |Removed |Added ---------------------------------------------------------------------------- Summary|Review Request: xorg-x11- |Review Request: xorg-x11- |drv-suntcx - X driver for |drv-suntcx - X driver for |suntcx hardware |suntcx hardware
------- Additional Comments From dennis@ausil.us 2008-06-03 19:25 EST ------- Scratch build http://sparc.koji.fedoraproject.org/koji/taskinfo?taskID=58928
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: xorg-x11-drv-suntcx - X driver for suntcx hardware
https://bugzilla.redhat.com/show_bug.cgi?id=448567
bdpepple@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|nobody@fedoraproject.org |bdpepple@gmail.com Flag| |fedora-review+
------- Additional Comments From bdpepple@gmail.com 2008-06-03 21:19 EST ------- MD5Sum: 779a31151e44d2fd2a5ee1794f836889 xf86-video-suntcx-1.1.0.tar.bz2
Good: * Source URL is canonical * Upstream source tarball verified * Package name conforms to the Fedora Naming Guidelines * Group Tag is from the official list * Buildroot has all required elements * All paths begin with macros * All necessary BuildRequires listed. * Make succeeds even when %{_smp_mflags} is defined * Files have appropriate permissions and owners
Bad: * rpmlint produces the following error: xorg-x11-drv-suntcx.sparc64: W: invalid-license MIT/X11
Looking at the license matrix, it looks like it should just be MIT. You can fix this when you import this into CVS.
Note: I don't have the hardware to verify that this driver works, but from a packaging standpoint this package looks fine.
+1 APPROVE.
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: xorg-x11-drv-suntcx - X driver for suntcx hardware
https://bugzilla.redhat.com/show_bug.cgi?id=448567
bdpepple@gmail.com 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: Review Request: xorg-x11-drv-suntcx - X driver for suntcx hardware
https://bugzilla.redhat.com/show_bug.cgi?id=448567
dennis@ausil.us changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs+
------- Additional Comments From dennis@ausil.us 2008-06-03 22:46 EST ------- New Package CVS Request ======================= Package Name: xorg-x11-drv-suntcx Short Description: X driver for suntcx Owners: ausil spot jima pjones Branches: F-9 InitialCC: Cvsextras Commits: yes
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: xorg-x11-drv-suntcx - X driver for suntcx hardware
https://bugzilla.redhat.com/show_bug.cgi?id=448567
dennis@ausil.us changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |RAWHIDE
------- Additional Comments From dennis@ausil.us 2008-06-06 17:46 EST ------- imported and built thanks for the review
package-review@lists.fedoraproject.org