[Bug 282261] New: Review Request: isync - Tool to synchronize IMAP4 and Maildir mailboxes

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

https://bugzilla.redhat.com/show_bug.cgi?id=282261

Summary: Review Request: isync - Tool to synchronize IMAP4 and
                    Maildir mailboxes
           Product: Fedora
           Version: devel
          Platform: All
        OS/Version: Linux
            Status: NEW
          Severity: medium
          Priority: medium
         Component: Package Review
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]
         QAContact: [e-mail]
                CC: [e-mail],[e-mail]

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

[Bug 282261] Review Request: isync - Tool to synchronize IMAP4 and Maildir mailboxes

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: isync - Tool to synchronize IMAP4 and Maildir mailboxes
Alias: isync-review

https://bugzilla.redhat.com/show_bug.cgi?id=282261

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
              Alias|                            |isync-review

------- Additional Comments From [e-mail]  2007-09-07 08:58 EST -------
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.

[Bug 282261] Review Request: isync - Tool to synchronize IMAP4 and Maildir mailboxes

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: isync - Tool to synchronize IMAP4 and Maildir mailboxes
Alias: isync-review

https://bugzilla.redhat.com/show_bug.cgi?id=282261

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|[e-mail]    |[e-mail]
             Status|NEW                         |ASSIGNED
               Flag|                            |fedora-review?

[Bug 282261] Review Request: isync - Tool to synchronize IMAP4 and Maildir mailboxes

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: isync - Tool to synchronize IMAP4 and Maildir mailboxes
Alias: isync-review

https://bugzilla.redhat.com/show_bug.cgi?id=282261

------- Additional Comments From [e-mail]  2007-09-07 09:16 EST -------
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-...

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

[Bug 282261] Review Request: isync - Tool to synchronize IMAP4 and Maildir mailboxes

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: isync - Tool to synchronize IMAP4 and Maildir mailboxes
Alias: isync-review

https://bugzilla.redhat.com/show_bug.cgi?id=282261

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

------- Additional Comments From [e-mail]  2007-09-08 19:37 EST -------
Please use:
Source0: http://downloads.sourceforge.net/%%7Bname%7D/%%7Bname%7D-%%7Bversion%7D.tar....
See:
http://fedoraproject.org/wiki/Packaging/SourceURL#head-e27982f18a3bfd26b5b6e...

Unuseful comments should be removed:
#Requires:

[Bug 282261] Review Request: isync - Tool to synchronize IMAP4 and Maildir mailboxes

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: isync - Tool to synchronize IMAP4 and Maildir mailboxes
Alias: isync-review

https://bugzilla.redhat.com/show_bug.cgi?id=282261

------- Additional Comments From [e-mail]  2007-09-09 15:42 EST -------
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-...

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

[Bug 282261] Review Request: isync - Tool to synchronize IMAP4 and Maildir mailboxes

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: isync - Tool to synchronize IMAP4 and Maildir mailboxes
Alias: isync-review

https://bugzilla.redhat.com/show_bug.cgi?id=282261

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-review?              |fedora-review+

------- Additional Comments From [e-mail]  2007-09-09 16:18 EST -------
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

[Bug 282261] Review Request: isync - Tool to synchronize IMAP4 and Maildir mailboxes

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: isync - Tool to synchronize IMAP4 and Maildir mailboxes
Alias: isync-review

https://bugzilla.redhat.com/show_bug.cgi?id=282261

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|                            |fedora-cvs?

------- Additional Comments From [e-mail]  2007-09-09 16:25 EST -------
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

[Bug 282261] Review Request: isync - Tool to synchronize IMAP4 and Maildir mailboxes

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: isync - Tool to synchronize IMAP4 and Maildir mailboxes
Alias: isync-review

https://bugzilla.redhat.com/show_bug.cgi?id=282261

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-cvs?                 |fedora-cvs+

------- Additional Comments From [e-mail]  2007-09-09 18:38 EST -------
cvs done.

[Bug 282261] Review Request: isync - Tool to synchronize IMAP4 and Maildir mailboxes

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: isync - Tool to synchronize IMAP4 and Maildir mailboxes
Alias: isync-review

https://bugzilla.redhat.com/show_bug.cgi?id=282261

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |CLOSED
         Resolution|                            |NEXTRELEASE

------- Additional Comments From [e-mail]  2007-09-10 14:33 EST -------
The package has been imported into CVS and built in all branches. Thanks Kevin,
Marek and Tiil.