Bug 612241 - Review Request: telepathy-logger - Telepathy framework logging daemon
Summary: Review Request: telepathy-logger - Telepathy framework logging daemon
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Matthias Clasen
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2010-07-07 16:09 UTC by Brian Pepple
Modified: 2010-07-09 18:41 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2010-07-09 18:41:32 UTC
Type: ---
Embargoed:
mclasen: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Brian Pepple 2010-07-07 16:09:05 UTC
Spec URL: http://bpepple.fedorapeople.org/rpms/telepathy-logger.spec
SRPM URL: http://bpepple.fedorapeople.org/rpms/telepathy-logger-0.1.3-1.fc14.src.rpm
Description: telepathy-logger is a headless Observer client that logs information
received by the Telepathy framework. It features pluggable backends to log different sorts of messages, in different formats.

Note: This is needed for the latest version of Empathy.

Comment 1 Matthias Clasen 2010-07-08 01:00:59 UTC
Builds fine in mock. rpmlint output:

$ rpmlint /var/lib/mock/fedora-rawhide-x86_64/result/*.rpm
telepathy-logger.src: W: spelling-error %description -l en_US pluggable -> plug gable, plug-gable, plugged
telepathy-logger.src: W: spelling-error %description -l en_US backends -> backbends, back ends, back-ends
telepathy-logger.src:19: W: mixed-use-of-spaces-and-tabs (spaces: line 1, tab: line 19)
telepathy-logger.x86_64: W: spelling-error %description -l en_US pluggable -> plug gable, plug-gable, plugged
telepathy-logger.x86_64: W: spelling-error %description -l en_US backends -> backbends, back ends, back-ends
telepathy-logger.x86_64: E: binary-or-shlib-defines-rpath /usr/libexec/telepathy-logger ['/usr/lib64']
telepathy-logger.x86_64: W: non-conffile-in-etc /etc/gconf/schemas/telepathy-logger.schemas
telepathy-logger.x86_64: W: dangerous-command-in-%pre rm
telepathy-logger.x86_64: W: dangerous-command-in-%post rm
4 packages and 0 specfiles checked; 1 errors, 8 warnings.

Comment 2 Matthias Clasen 2010-07-08 01:07:03 UTC
Looks like all the rpmlint warnings are ignorable, except for the rpath.

Some minor cleanups possible:

BuildRoot is not longer needed

rm -rf $RPM_BUILD_ROOT in %install is no longer needed

%clean is no longer needed

Comment 3 Brian Pepple 2010-07-08 01:20:48 UTC
Ah, forgot to run rpmlint on an x86_64 binary. I'll go ahead and clean up the changes you mentioned and upload a new version tomorrow. Thanks for taking the time to look at this, Matthias!

Comment 4 Brian Pepple 2010-07-08 12:44:43 UTC
Spec URL: http://bpepple.fedorapeople.org/rpms/telepathy-logger.spec
SRPM URL: http://bpepple.fedorapeople.org/rpms/telepathy-logger-0.1.3-2.fc13.src.rpm

* Wed Jul  7 2010 Brian Pepple <bpepple> - 0.1.3-2
- Remove unnecessary buildroot info.
- Remove rpath.

Comment 5 Matthias Clasen 2010-07-08 14:14:38 UTC
package name: ok
spec file name: ok
packaging guidelines: ok
license: ok
license field: ok
license file: ok
spec file language: ok
spec file readable: yes
upstream sources: ok
buildable: yes
excludearch: ok
buildrequires: ok
locale handling: ok
shared libs: ok
system libs: ok
relocatable: ok
directory ownership: ok
duplicate files: ok
file permissions: ok
macro use: ok
permissible content: ok
large docs: ok
%doc content: ok
header files: ok
static libs: ok
shared libs: ok
devel deps: ok
libtool: ok
gui apps: ok
file ownership: ok
utf8 filenames: ok

Approved

Comment 6 Brian Pepple 2010-07-08 14:22:09 UTC
New Package CVS Request
=======================
Package Name: telepathy-logger
Short Description: Telepathy framework logging daemon
Owners: bpepple
Branches: 
InitialCC:

Comment 7 Kevin Fenzi 2010-07-09 18:23:53 UTC
CVS done (by process-cvs-requests.py).

Comment 8 Brian Pepple 2010-07-09 18:41:32 UTC
Imported & built for devel. Matthias, thanks again for the review.


Note You need to log in before you can comment on or make changes to this bug.