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

Bug 165910

Summary: Review Request: sylpheed-claws-plugins - Additional plugins for sylpheed-claws
Product: [Fedora] Fedora Reporter: Andreas Bierfert <andreas.bierfert>
Component: Package ReviewAssignee: Michael Schwendt <bugs.michael>
Status: CLOSED NEXTRELEASE QA Contact: David Lawrence <dkl>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: fedora-package-review
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2005-08-20 23:04:58 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Bug Depends On:    
Bug Blocks: 163779    

Description Andreas Bierfert 2005-08-14 00:05:49 UTC
Spec Name or Url:
SRPM Name or Url:
Additional plugins for sylpheed-claws

Comment 1 Michael Schwendt 2005-08-16 22:07:44 UTC
> BuildRequires: synce-devel

I believe this is a dead end on x86_64, where it doesn't build: bug 148003

You can use the %setup macro for simpler tarball extraction into
rpm build dir. This would also clean up the %doc statements:

--- sylpheed-claws-plugins.spec.orig    2005-08-14 01:55:44.000000000 +0200
+++ sylpheed-claws-plugins.spec 2005-08-17 00:03:08.000000000 +0200
@@ -104,13 +104,9 @@
 Evolution or Outlook.
-%{__mkdir_p} %{_builddir}/%{name}-%{version}/
-cd %{_builddir}/%{name}-%{version}/
-%{__tar} xzvf %{SOURCE0}
+%setup -q -c
-cd %{_builddir}/%{name}-%{version}
 # cachesaver
 cd cachesaver-0.5
 %configure --disable-static --disable-dependency-tracking
@@ -154,8 +150,6 @@
-cd %{_builddir}/%{name}-%{version}
 # cachesaver
 cd cachesaver-0.5
 %{__make} install DESTDIR=$RPM_BUILD_ROOT
@@ -195,70 +189,70 @@
-%doc %{name}-%{version}/README
+%doc README
 %files cachesaver
-%doc %{name}-%{version}/cachesaver-0.5/AUTHORS
-%doc %{name}-%{version}/cachesaver-0.5/ChangeLog
-%doc %{name}-%{version}/cachesaver-0.5/COPYING
+%doc cachesaver-0.5/AUTHORS
+%doc cachesaver-0.5/ChangeLog
+%doc cachesaver-0.5/COPYING
 %files etpan-privacy
-%doc %{name}-%{version}/etpan-privacy-0.9/AUTHORS
-%doc %{name}-%{version}/etpan-privacy-0.9/ChangeLog
-%doc %{name}-%{version}/etpan-privacy-0.9/COPYING
-%doc %{name}-%{version}/etpan-privacy-0.9/README
+%doc etpan-privacy-0.9/AUTHORS
+%doc etpan-privacy-0.9/ChangeLog
+%doc etpan-privacy-0.9/COPYING
+%doc etpan-privacy-0.9/README
 %files fetchinfo
-%doc %{name}-%{version}/fetchinfo-plugin-0.4.6/ChangeLog
-%doc %{name}-%{version}/fetchinfo-plugin-0.4.6/COPYING
-%doc %{name}-%{version}/fetchinfo-plugin-0.4.6/README
+%doc fetchinfo-plugin-0.4.6/ChangeLog
+%doc fetchinfo-plugin-0.4.6/COPYING
+%doc fetchinfo-plugin-0.4.6/README
 %files maildir
-%doc %{name}-%{version}/maildir-0.8/AUTHORS
-%doc %{name}-%{version}/maildir-0.8/ChangeLog
-%doc %{name}-%{version}/maildir-0.8/COPYING
-%doc %{name}-%{version}/maildir-0.8/README
-%doc %{name}-%{version}/maildir-0.8/doc/README.xml
+%doc maildir-0.8/AUTHORS
+%doc maildir-0.8/ChangeLog
+%doc maildir-0.8/COPYING
+%doc maildir-0.8/README
+%doc maildir-0.8/doc/README.xml
 %files mailmbox
-%doc %{name}-%{version}/mailmbox-1.2/AUTHORS
-%doc %{name}-%{version}/mailmbox-1.2/ChangeLog
-%doc %{name}-%{version}/mailmbox-1.2/COPYING
-%doc %{name}-%{version}/mailmbox-1.2/README
+%doc mailmbox-1.2/AUTHORS
+%doc mailmbox-1.2/ChangeLog
+%doc mailmbox-1.2/COPYING
+%doc mailmbox-1.2/README
 %files perl
-%doc %{name}-%{version}/perl_plugin-0.3/AUTHORS
-%doc %{name}-%{version}/perl_plugin-0.3/ChangeLog
-%doc %{name}-%{version}/perl_plugin-0.3/COPYING
-%doc %{name}-%{version}/perl_plugin-0.3/README
-%doc %{name}-%{version}/perl_plugin-0.3/sc_perl.pod
+%doc perl_plugin-0.3/AUTHORS
+%doc perl_plugin-0.3/ChangeLog
+%doc perl_plugin-0.3/COPYING
+%doc perl_plugin-0.3/README
+%doc perl_plugin-0.3/sc_perl.pod
 %files synce
-%doc %{name}-%{version}/synce_plugin-0.2/AUTHORS
-%doc %{name}-%{version}/synce_plugin-0.2/ChangeLog
-%doc %{name}-%{version}/synce_plugin-0.2/COPYING
-%doc %{name}-%{version}/synce_plugin-0.2/README
+%doc synce_plugin-0.2/AUTHORS
+%doc synce_plugin-0.2/ChangeLog
+%doc synce_plugin-0.2/COPYING
+%doc synce_plugin-0.2/README
 %files vcalendar
-%doc %{name}-%{version}/vcalendar-1.6/AUTHORS
-%doc %{name}-%{version}/vcalendar-1.6/ChangeLog
-%doc %{name}-%{version}/vcalendar-1.6/COPYING
-%doc %{name}-%{version}/vcalendar-1.6/README
+%doc vcalendar-1.6/AUTHORS
+%doc vcalendar-1.6/ChangeLog
+%doc vcalendar-1.6/COPYING
+%doc vcalendar-1.6/README

Comment 2 Andreas Bierfert 2005-08-18 05:43:48 UTC
Thanks for the review. Here is the new version with disabled synce plugin on x86_64.

Comment 3 Michael Schwendt 2005-08-19 21:54:50 UTC
First a few somewhat related findings:

$ strings $(which sylpheed-claws) | grep firefox
mozilla-firefox -remote 'openURL(%s,new-window)'

Since "mozilla-firefox" does not exist in PATH, a better default for 
Sylpheed-claws would be:

  gnome-open '%s'


I cannot seem to get Sylpheed-claws to verify GPG MIME signatures
automatically in a convenient way. Plugin is loaded, sending signed
mails works. But it only lists the signatures as
"[application/pgp-signature (189 bytes)]" and in the message body
parts list view above/right of the message text box. Only if I select
the text/plain MIME part there, I get the signature verification status
message at the bottom of the window.


For any attachments (like signatures above or the text/plain part), if I
double-click on them in the MIME Type list, I get:

  execvp: No such file or directory


Now back to this package:

/usr/include/ical.h in package "sylpheed-claws-plugins-vcalendar" is
an API for libical, which is used in this plugin. Installing it looks
like an upstream mistake and only bears the risk of creating a conflict
with any other package which would provide /usr/include/ical.h

Rest looks good.

Comment 4 Michael Schwendt 2005-08-19 22:04:00 UTC
synce is now orphaned, see /cvs/extras/owners/owners.list revision 1.103

Comment 5 Andreas Bierfert 2005-08-19 22:45:54 UTC
I will look at the sylpheed-claws issues and maybe report the gpg stuff
upstream. As for ical.h: excluded from current release. 

I will take ownership of synce as I am working on multisync packages atm anyway...

Thx for the review.

Comment 6 Michael Schwendt 2005-08-20 11:56:31 UTC
With 'devel', sylpheed-claws needs a rebuild against new GTK+/Cairo ABI change
in Rawhide before this could be built.