Spec Name or Url: http://fedora.lowlatency.de/review/wmCalClock.spec SRPM Name or Url: http://fedora.lowlatency.de/review/wmCalClock-1.25-1.src.rpm Description: A Calendar clock with antialiased text
Important: - the man page is set executable Minor: - please use the "-p" switch of install to preserve timestamps
Here you go: http://fedora.lowlatency.de/review/wmCalClock.spec http://fedora.lowlatency.de/review/wmCalClock-1.25-2.src.rpm
Review for release 2: * RPM name is OK * Source wmCalClock-1.25.tar.gz is the same as upstream * Builds fine in mock * rpmlint of wmCalClock looks OK * File list of wmCalClock looks OK * Works fine Minor nitpick: In the scriptlets, "%{_builddir}/%{name}-%{version}" is the current working directory, and can be replaced by "."
Thanks for the review... imported and build :)
Normalize summary field for easy parsing