Bug 750854

Summary: Review Request: evolution-ews - Evolution extension for Exchange Web Services
Product: [Fedora] Fedora Reporter: Matthew Barnes <mbarnes>
Component: Package ReviewAssignee: David Woodhouse <dwmw2>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: dwmw2, notting, package-review, rkhadgar
Target Milestone: ---Flags: dwmw2: fedora-review+
gwync: 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: 2011-11-07 18:02:10 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 Matthew Barnes 2011-11-02 15:01:10 UTC
Spec URL: http://mbarnes.fedorapeople.org/package-review/evolution-ews.spec
SRPM URL: http://mbarnes.fedorapeople.org/package-review/evolution-ews-3.2.1-1.fc16.src.rpm

This package allows Evolution to interact with Microsoft Exchange servers,
versions 2007 and later, through its Exchange Web Services (EWS) interface.

Comment 1 David Woodhouse 2011-11-02 16:14:41 UTC
evolution-ews.x86_64: E: binary-or-shlib-defines-rpath /usr/lib64/evolution-data-server/addressbook-backends/libebookbackendews.so ['/usr/lib64/evolution-data-server-3.2', '/usr/lib64']
evolution-ews.x86_64: E: zero-length /usr/share/doc/evolution-ews-3.2.1/AUTHORS
evolution-ews.x86_64: W: install-file-in-docs /usr/share/doc/evolution-ews-3.2.1/INSTALL

Comment 3 Matthew Barnes 2011-11-03 13:34:59 UTC
>   As a workaround, removing AUTHOR and INSTALL from doc section will help.

Consider them removed.  I'll do a new upload if something more substantial needs changed.

Comment 4 David Woodhouse 2011-11-07 17:14:00 UTC
Passes review guidelines at http://fedoraproject.org/wiki/Packaging:ReviewGuidelines

Consider removing the BuildRoot tag since it's apparently not needed since Fedora 10 (http://fedoraproject.org/wiki/Packaging/Guidelines#BuildRoot_tag) but it's harmless.

Comment 5 Matthew Barnes 2011-11-07 17:33:33 UTC
Thanks! I updated the linked-to spec file (not the SRPM), removing BuildRoot, AUTHORS and INSTALL as suggested.

Comment 6 Matthew Barnes 2011-11-07 17:46:56 UTC
New Package SCM Request
=======================
Package Name: evolution-ews
Short Description: Evolution extension for Exchange Web Services
Owners: mbarnes mcrha
Branches: f16
InitialCC:

Comment 7 Gwyn Ciesla 2011-11-07 17:49:19 UTC
Git done (by process-git-requests).

Comment 8 Matthew Barnes 2011-11-07 18:02:10 UTC
Closing.