[Bug 226357] Merge Review: rdate

[bugzilla at 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: rdate


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


bugzilla at redhat.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|normal                      |medium
           Priority|normal                      |medium

pknirsch at redhat.com changed:

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




------- Additional Comments From pknirsch at redhat.com  2007-05-29 09:26 EST -------
* Remove the dot at the end of Summary line.

Fixed

* It seems that there is no upstream. No URL is given, and the Source address
doesn't work either. So I can't check that this is the same as the upstream
source. (BLOCKER)

There is no real upstream anymore except the tarball from sopwith (Elliot Lee)

* Change BuildRoot to %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)

Fixed

* Use the option %{?_smp_mflags} for make.

Fixed

* Don't use %makeinstall, if possible. See
http://fedoraproject.org/wiki/Packaging/Guidelines#MakeInstall

Fixed

* Use %defattr(-,root,root,-) instead of %defattr(-,root,root)

Fixed

* Consider using %{?dist} in Release tag.

Fixed

* The binary file 'rdate' is installed with permission 555 in the Makefile. It
should be 755. This can be fixed either by patching the Makefile or by
explicitly changing the permission in the %install or %files section. (BLOCKER)

Fixed


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