[Bug 593280] Review Request: servicelog - Servicelog Tools

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

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #4 from Kamil Dudka <kdudka@xxxxxxxxxx> 2010-05-19 07:16:06 EDT ---
+ silent rpmlint
+ valid source URL and project URL
+ dist tag OK
+ BuildRoot tag OK
+ license tag OK
+ no downstream patches for now
+ configure OK
+ make OK
+ %install OK
+ %clean OK
+ %files OK
- I wonder why the upstream tarball is not already bootstrapped
- also the CVS directories look there redundant to me
- some warnings during build on ppc64 as follows:

    warning: format '%llu' expects type 'long long unsigned int', but argument
2 has type 'uint64_t'

They seem to be false alarms since sizeof(unsigned long long) ==
sizeof(unit64_t) == 8.

Overall the package looks sane.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
_______________________________________________
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]