Bug 282261
| Summary: | Review Request: isync - Tool to synchronize IMAP4 and Maildir mailboxes | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Lubomir Kundrak <lkundrak> |
| Component: | Package Review | Assignee: | Marek Mahut <mmahut> |
| Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
| Severity: | medium | Docs Contact: | |
| Priority: | medium | ||
| Version: | rawhide | CC: | fedora-package-review, notting, opensource |
| Target Milestone: | --- | Flags: | mmahut:
fedora-review+
kevin: fedora-cvs+ |
| Target Release: | --- | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Fixed In Version: | Doc Type: | Bug Fix | |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2007-09-10 18:33:49 UTC | Type: | --- |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Embargoed: | |||
|
Description
Lubomir Kundrak
2007-09-07 12:56:03 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. 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/ Please use: Source0: http://downloads.sourceforge.net/%{name}/%{name}-%{version}.tar.gz See: http://fedoraproject.org/wiki/Packaging/SourceURL#head-e27982f18a3bfd26b5b6ecbee113d2d8f3f006f2 Unuseful comments should be removed: #Requires: 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/ 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 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 cvs done. The package has been imported into CVS and built in all branches. Thanks Kevin, Marek and Tiil. |