[Bug 1183924] Review Request: perl-Time-TAI64 - Perl extension for converting TAI64 strings into standard unix timestamps

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

Paul Howarth <paul@xxxxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
              Flags|fedora-review?              |fedora-review+



--- Comment #2 from Paul Howarth <paul@xxxxxxxxxxxx> ---
rpmlint output
==============
perl-Time-TAI64.noarch: W: spelling-error Summary(en_US) unix -> UNIX, Unix,
uni
perl-Time-TAI64.noarch: W: spelling-error Summary(en_US) timestamps -> time
stamps, time-stamps, times tamps
perl-Time-TAI64.noarch: W: spelling-error %description -l en_US timestamps ->
time stamps, time-stamps, times tamps
perl-Time-TAI64.noarch: W: spelling-error %description -l en_US multilog ->
multilevel
perl-Time-TAI64.noarch: W: spelling-error %description -l en_US timestamp ->
time stamp, time-stamp, times tamp
perl-Time-TAI64.src: W: spelling-error Summary(en_US) unix -> UNIX, Unix, uni
perl-Time-TAI64.src: W: spelling-error Summary(en_US) timestamps -> time
stamps, time-stamps, times tamps
perl-Time-TAI64.src: W: spelling-error %description -l en_US timestamps -> time
stamps, time-stamps, times tamps
perl-Time-TAI64.src: W: spelling-error %description -l en_US multilog ->
multilevel
perl-Time-TAI64.src: W: spelling-error %description -l en_US timestamp -> time
stamp, time-stamp, times tamp
2 packages and 0 specfiles checked; 0 errors, 10 warnings.

checklist
=========
- rpmlint OK (just spelling complaints about technical terms)
- package and spec file naming OK
- package meets guidelines
- license is same as perl, OK for Fedora, matches upstream
- no license text to include
- spec file written in English and is legible
- source file matches upstream, including timestamp
- package builds OK in mock for Rawhide x86_64, and is not arch-specific
- buildreqs OK
- no locales, libraries etc. to bother with
- no intent to make the package relocatable
- directory ownership and permissions OK
- no duplicate files
- macro usage is consistent
- code, not content
- no large docs, docs don't affect runtime
- not a GUI app, no desktop file needed
- filenames are all ASCII


notes
=====
%description needs re-wording to improve English, e.g. s/provides/providing/

Should BR: perl itself, needed to run Makefile.PL


Package APPROVED.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review





[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]