Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 174495 - Review Request: libopensync-plugin-kdepim
Summary: Review Request: libopensync-plugin-kdepim
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Aurelien Bompard
QA Contact: David Lawrence
URL: http://www.opensync.org/
Whiteboard:
Depends On: 174486
Blocks: FE-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2005-11-29 15:42 UTC by Andreas Bierfert
Modified: 2007-11-30 22:11 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-03-02 22:34:38 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Andreas Bierfert 2005-11-29 15:42:39 UTC
Spec Name or Url: http://fedora.lowlatency.de/review/libopensync-plugin-kdepim.spec
SRPM Name or Url:
http://fedora.lowlatency.de/review/libopensync-plugin-kdepim-0.18-1.src.rpm
Description:
KDE plugin for libopensync

Comment 1 Rex Dieter 2005-11-29 16:26:10 UTC
1 small comment (which probably can be applied to all/most of the
libopensync-plugin-* pkgs you just submitted):
1.  about naming.  For pkgs that provide "plugin" type services, pkg names are
sometimes/often(*) rooted in what they're a plugin for, so in this case, another
name for this pkg to consider is:
kdepim-opensync

On the other hand, I'm also a strong proponent of following upstream pkg names
too (unless there is good reason to do otherwise, of course).  (-:

(*) precendents include addons/plugins for (off the top of my head) xmms, valgrind


Comment 2 Andreas Bierfert 2005-11-29 16:27:57 UTC
Maybe as provide?

Comment 3 Aurelien Bompard 2005-12-12 21:34:54 UTC
In my opinion, the foo-bar name means that bar enhances foo. In this case, the
kdepim plugin enhances opensync more than enhances kdepim, so I's stick with the
original name. (on the contrary, 
I didn't find any packaging errors with this plugin, but at the end of the
synchronization, it pops up a kcrash dialog. I've tried adding back the *.la
files and rebuilding it on my system (not in mock), but it still crashes.
Any idea ?

Comment 4 Aurelien Bompard 2005-12-12 21:39:28 UTC
Missing end of sentence:
...(on the contrary, for the xmms-arts plugin for example, xmms is enhanced by
the plugin, not arts.)

Comment 5 Andreas Bierfert 2005-12-13 14:29:01 UTC
(In reply to comment #3)
> I didn't find any packaging errors with this plugin, but at the end of the
> synchronization, it pops up a kcrash dialog. I've tried adding back the *.la
> files and rebuilding it on my system (not in mock), but it still crashes.
> Any idea ?

What are you trying to sync? I tried evo2 <-> kdepim and it worked a bit... It
has trouble with some of the entries but it did not crash...


Comment 6 Aurelien Bompard 2005-12-13 15:39:37 UTC
I'm syncing kdepim with a directory (file-sync plugin). Sometimes it pops up the
kcrash dialog :
The sync was successful
KCrash: Application 'libopensync-kdepim-plugin' crashing...

Sometimes it ouputs:
The sync was successful
*** glibc detected *** msynctool: free(): invalid pointer: 0x08eff0c0 ***
And stops there.

It does not seem to be related to packaging, so it should not block this package
(opensync, and particularly the kde plugin, is under heavy development at the
moment). But you'll be the one assigned to bug reports, so be prepared... :)

Comment 7 Andreas Bierfert 2005-12-15 09:19:01 UTC
sure... but then I will bug upstream with it ;)

Comment 8 Andreas Bierfert 2006-03-01 06:52:20 UTC
So what shall we do with this?  =) Fine by me if we push this and take reported
matters upstream...

Comment 9 Aurelien Bompard 2006-03-02 16:36:18 UTC
Alright then.

Review for release 1:
* RPM name is OK
* Builds fine in mock
* rpmlint of libopensync-plugin-kdepim looks OK
* File list of libopensync-plugin-kdepim looks OK
* Tarball has the same MD5SUM as upstream
* Works (most of the time :) )
APPROVED

Comment 10 Andreas Bierfert 2006-03-02 22:34:38 UTC
Imported, fixed ;), pushed.

Thanks for the review...


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