[Bug 547988] Review Request: kpilot - Utility to synchronize PIM (Personal Information Management) data with PalmOS devices

[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=547988


Rex Dieter <rdieter@xxxxxxxxxxxx> changed:

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




--- Comment #3 from Rex Dieter <rdieter@xxxxxxxxxxxx>  2009-12-16 11:17:23 EDT ---
rpmlint:
kpilot.src: E: summary-too-long Utility to synchronize PIM (Personal
Information Management) data with PalmOS devices
kpilot.src: E: description-line-too-long Utility to synchronize PIM (Personal
Information Management) data with PalmOS devices.
$ rpmlint *src.rpm x86_64/*.rpm
kpilot.src: E: summary-too-long Utility to synchronize PIM (Personal
Information Management) data with PalmOS devices
kpilot.src: E: description-line-too-long Utility to synchronize PIM (Personal
Information Management) data with PalmOS devices.
error checking signature of kpilot-5.3.0-1.fc13.src.rpm
kpilot.x86_64: E: summary-too-long Utility to synchronize PIM (Personal
Information Management) data with PalmOS devices
kpilot.x86_64: E: description-line-too-long Utility to synchronize PIM
(Personal Information Management) data with PalmOS devices.
kpilot.x86_64: E: invalid-soname /usr/lib64/libkpilot_akonadibase.so
libkpilot_akonadibase.so
kpilot.x86_64: E: invalid-soname /usr/lib64/libkpilot_conduit_base.so
libkpilot_conduit_base.so
3 packages and 0 specfiles checked; 6 errors, 0 warnings.

Summary is longish, perhaps omit the "Utility to" part at the beginning?

sources: ok
a4f32a268802be8eb852d1de00cbd2f6  kdepim-4.3.4.tar.bz2

license: ok

scriptlets: ok

deps: ok


Suggestions (consider these SHOULD items, can be addressed post-review):

1.  make -libs package to be more multilib friendly

2.  Add (something like):
%if 0%{?fedora} < 13
Conflicts: kdepim < 6:4.3.80
%endif


Otherwise, looks good, APPROVED.

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

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

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