Bug 282261 - Review Request: isync - Tool to synchronize IMAP4 and Maildir mailboxes
Summary: Review Request: isync - Tool to synchronize IMAP4 and Maildir mailboxes
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Marek Mahut
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2007-09-07 12:56 UTC by Lubomir Kundrak
Modified: 2016-08-14 16:22 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2007-09-10 18:33:49 UTC
Type: ---
Embargoed:
mmahut: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Lubomir Kundrak 2007-09-07 12:56:03 UTC
Spec URL: http://people.redhat.com/lkundrak/SPECS/isync.spec
SRPM URL: http://people.redhat.com/lkundrak/mock-results/isync-1.0.3-1.fc7.i386/isync-1.0.3-1.fc7.src.rpm
Description: Tool to synchronize IMAP4 and Maildir mailboxes

isync is a command line application which synchronizes mailboxes; currently Maildir and IMAP4 mailboxes are supported. New messages, message deletions and flag changes can be propagated both ways. isync is suitable for use in IMAP-disconnected mode.

Comment 1 Lubomir Kundrak 2007-09-07 12:58:19 UTC
rpmlint is silent on all generated packages.
The build logs from mock can help you review the package:
http://people.redhat.com/lkundrak/mock-results/isync-1.0.3-1.fc7.i386/
Thanks for the review.

Comment 2 Lubomir Kundrak 2007-09-07 13:16:32 UTC
I added an openssl-devel dependency to enable SSL support and bumped the
Revision by one. And thus, here are the actual files:
Spec URL: http://people.redhat.com/lkundrak/SPECS/isync.spec
SRPM URL:
http://people.redhat.com/lkundrak/mock-results/isync-1.0.3-2.fc7.i386/isync-1.0.3-2.fc7.src.rpm

Mock output is
http://people.redhat.com/lkundrak/mock-results/isync-1.0.3-2.fc7.i386/

Comment 4 Lubomir Kundrak 2007-09-09 19:42:56 UTC
Till: thanks for your suggestions, the respective changes have been included
together with a fix for compilation on Rawhide (new open() semantics).

The new packages are here:
Spec URL: http://people.redhat.com/lkundrak/SPECS/isync.spec
SRPM URL:
http://people.redhat.com/lkundrak/mock-results/isync-1.0.3-3.fc8.i386/isync-1.0.3-3.fc8.src.rpm

Mock output is:
http://people.redhat.com/lkundrak/mock-results/isync-1.0.3-3.fc8.i386/

Comment 5 Marek Mahut 2007-09-09 20:18:18 UTC
Package appears correct and builds correctly as well.

APPROVED

BTW, sync isn't Apple(TM) trademark as per
http://www.apple.com/legal/trademark/appletmlist.html

Comment 6 Lubomir Kundrak 2007-09-09 20:25:49 UTC
Marek: thanks for looking up that apple isync thing, I was a bit worried about it.
CVS admins, here's the request:

New Package CVS Request
=======================
Package Name: isync
Short Description: Tool to synchronize IMAP4 and Maildir mailboxes
Owners: lkundrak
Branches: FC-6 F-7 EL-5 EL-4
Cvsextras Commits: yes

Comment 7 Kevin Fenzi 2007-09-09 22:38:33 UTC
cvs done. 

Comment 8 Lubomir Kundrak 2007-09-10 18:33:49 UTC
The package has been imported into CVS and built in all branches. Thanks Kevin,
Marek and Tiil.


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