[Bug 476315] Review Request: evolution-mapi - Exchange 2007 support for Evolution

2009-02-27 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Bug 476315 depends on bug 453083, which changed state.

Bug 453083 Summary: Review Request: Samba4 - Samba4 CIFS and AD server and 
client
https://bugzilla.redhat.com/show_bug.cgi?id=453083

   What|Old Value   |New Value

 Status|NEW |CLOSED
 Resolution||RAWHIDE



-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 476315] Review Request: evolution-mapi - Exchange 2007 support for Evolution

2009-02-27 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Matthew Barnes mbar...@redhat.com changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||RAWHIDE




--- Comment #21 from Matthew Barnes mbar...@redhat.com  2009-02-26 23:25:18 
EDT ---
Package built.  Closing review.

http://kojipkgs.fedoraproject.org/packages/evolution-mapi/

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 476315] Review Request: evolution-mapi - Exchange 2007 support for Evolution

2009-02-27 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Bug 476315 depends on bug 453395, which changed state.

Bug 453395 Summary: Review Request: OpenChange - Microsoft Exchange access with 
native protocols
https://bugzilla.redhat.com/show_bug.cgi?id=453395

   What|Old Value   |New Value

 Status|NEW |CLOSED
 Resolution||RAWHIDE



-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 476315] Review Request: evolution-mapi - Exchange 2007 support for Evolution

2009-02-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #16 from Matthew Barnes mbar...@redhat.com  2009-02-26 11:01:45 
EDT ---
Got clarification from upstream that the correct license is LGPLv2+ and the
COPYING file is apparently wrong (and so is my spec file).  But there's also a
COPYING.LGPL2 and COPYING.LGPL3 in the SVN repo which isn't included in the
tarball.  *confusion*

Novell promised to clarify the license in the next release.  So can we mark it
as the -intended- license, and I'll make sure it gets fixed?

The grep thing I must have copy-n-pasted from another package.  It's gone now.

http://mbarnes.fedorapeople.org/mapi/SPECS/evolution-mapi.spec
http://mbarnes.fedorapeople.org/mapi/SRPMS/evolution-mapi-0.25.91-3.fc10.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 476315] Review Request: evolution-mapi - Exchange 2007 support for Evolution

2009-02-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Matthias Clasen mcla...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-review+




--- Comment #17 from Matthias Clasen mcla...@redhat.com  2009-02-26 12:25:29 
EDT ---
 Novell promised to clarify the license in the next release.  So can we mark it
 as the -intended- license, and I'll make sure it gets fixed?

Sounds good enough to me. 
And ship no license file then, instead of the wrong license file, I guess.
Approved.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 476315] Review Request: evolution-mapi - Exchange 2007 support for Evolution

2009-02-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #18 from Matthew Barnes mbar...@redhat.com  2009-02-26 12:59:36 
EDT ---
Thanks again for the reviews.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 476315] Review Request: evolution-mapi - Exchange 2007 support for Evolution

2009-02-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Matthew Barnes mbar...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #19 from Matthew Barnes mbar...@redhat.com  2009-02-26 13:00:49 
EDT ---
New Package CVS Request
===
Package Name: evolution-mapi
Short Description: Exchange 2007 support for Evolution
Owners: mbarnes
Branches: 
InitialCC:

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 476315] Review Request: evolution-mapi - Exchange 2007 support for Evolution

2009-02-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Kevin Fenzi ke...@tummy.com changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #20 from Kevin Fenzi ke...@tummy.com  2009-02-26 19:29:02 EDT ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 476315] Review Request: evolution-mapi - Exchange 2007 support for Evolution

2009-02-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #12 from Matthew Barnes mbar...@redhat.com  2009-02-25 13:20:33 
EDT ---
Oded: http://bugzilla.gnome.org/enter_bug.cgi?product=evolution-mapi

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 476315] Review Request: evolution-mapi - Exchange 2007 support for Evolution

2009-02-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Matthias Clasen mcla...@redhat.com changed:

   What|Removed |Added

 CC||mcla...@redhat.com
 AssignedTo|nob...@fedoraproject.org|mcla...@redhat.com




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 476315] Review Request: evolution-mapi - Exchange 2007 support for Evolution

2009-02-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #13 from Matthias Clasen mcla...@redhat.com  2009-02-25 22:42:06 
EDT ---
Builds ok in mock

rpmlint output:

evolution-mapi-devel.x86_64: W: no-documentation
4 packages and 0 specfiles checked; 0 errors, 1 warnings.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 476315] Review Request: evolution-mapi - Exchange 2007 support for Evolution

2009-02-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #14 from Matthias Clasen mcla...@redhat.com  2009-02-25 22:55:16 
EDT ---
Loooking around the provides and requires a bit, I think it would be good to
have explicit requires against evolution and e-d-s (since they own 
/usr/lib/evolution/2.26/plugins and
/usr/lib/evolution-data-server-1.2/extension). They _are_ currently pulled in
by library dependencies, but thats not very obvious (evo seems to get pulled in
via libeutil ?)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 476315] Review Request: evolution-mapi - Exchange 2007 support for Evolution

2009-02-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #15 from Matthias Clasen mcla...@redhat.com  2009-02-25 23:36:52 
EDT ---
formal review

package name: ok
spec file name: ok
packaging guidelines: ok
license: ok
package name: ok
spec file name: ok
packaging guidelines: ok
license: ok
license field/file: the license field says GPLv2+, but COPYING is GPLv3. What
gives ?
spec language: ok
spec legible: ok
upstream sources: ok
buildable: ok
excludearch: ok
build deps: ok
locale handling: ok, but I don't know what that extra grep business is about.
Shouldn't be necessary ?
ldconfig: ok
relocatable: ok
directory ownership: ok, but see earlier comment about deps
duplicate files: ok
permissions: ok
%clean: ok
macro use: ok
permissible content: ok
large docs: ok
%doc content: ok
header files: ok
static libs: ok
pkgconfig files: ok
shared libs: ok
devel dep: ok
libtool archives: ok
gui apps: ok
file ownership: ok
%install: ok
utf8 filenames: ok

summary:
 - clarify license
 - consider getting rid of grep business around %find_lang
 - consider adding explicit deps for evo/eds

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 476315] Review Request: evolution-mapi - Exchange 2007 support for Evolution

2009-02-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #11 from Oded Arbel o...@geek.co.il  2009-02-23 05:45:07 EDT ---
Matthew - where can I report problems with the evolution-mapi implementation
(not the package)?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 476315] Review Request: evolution-mapi - Exchange 2007 support for Evolution

2009-02-19 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #10 from Matthew Barnes mbar...@redhat.com  2009-02-19 12:29:06 
EDT ---
Update adds some missing BuildRequires.

http://mbarnes.fedorapeople.org/mapi/SPECS/evolution-mapi.spec
http://mbarnes.fedorapeople.org/mapi/SRPMS/evolution-mapi-0.25.91-2.fc10.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 476315] Review Request: evolution-mapi - Exchange 2007 support for Evolution

2009-02-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #9 from Matthew Barnes mbar...@redhat.com  2009-02-16 08:04:06 
EDT ---
Update for 0.25.91.  No changes to packaging.

http://mbarnes.fedorapeople.org/mapi/SPECS/evolution-mapi.spec
http://mbarnes.fedorapeople.org/mapi/SRPMS/evolution-mapi-0.25.91-1.fc10.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 476315] Review Request: evolution-mapi - Exchange 2007 support for Evolution

2009-02-05 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #8 from Matthew Barnes mbar...@redhat.com  2009-02-06 01:02:13 
EDT ---
Update for 0.25.90.  No changes to packaging.

http://mbarnes.fedorapeople.org/mapi/SPECS/evolution-mapi.spec
http://mbarnes.fedorapeople.org/mapi/SRPMS/evolution-mapi-0.25.90-1.fc10.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 476315] Review Request: evolution-mapi - Exchange 2007 support for Evolution

2009-01-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #7 from Matthew Barnes mbar...@redhat.com  2009-01-20 07:01:17 
EDT ---
Update for 0.25.5.  No changes to packaging.

http://mbarnes.fedorapeople.org/mapi/SPECS/evolution-mapi.spec
http://mbarnes.fedorapeople.org/mapi/SRPMS/evolution-mapi-0.25.5-1.fc10.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 476315] Review Request: evolution-mapi - Exchange 2007 support for Evolution

2009-01-06 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #6 from Matthew Barnes mbar...@redhat.com  2009-01-06 11:02:44 
EDT ---
Update for 0.25.4, which introduces translations:

http://mbarnes.fedorapeople.org/mapi/SPECS/evolution-mapi.spec
http://mbarnes.fedorapeople.org/mapi/SRPMS/evolution-mapi-0.25.4-1.fc10.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 476315] Review Request: evolution-mapi - Exchange 2007 support for Evolution

2008-12-15 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #3 from Matthew Barnes mbar...@redhat.com  2008-12-15 13:18:16 
EDT ---
(In reply to comment #1)
 This fails to build for me on rawhide; it needs openchange-devel which is not
 available.

Jason, see http://fedoraproject.org/wiki/Features/OpenChange.

OpenChange and Samba4 are also under review to support this.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 476315] Review Request: evolution-mapi - Exchange 2007 support for Evolution

2008-12-15 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #4 from Jason Tibbitts ti...@math.uh.edu  2008-12-15 13:24:35 EDT 
---
Then could you perhaps set the ticket dependencies properly so that something
indicates that this isn't actually reviewable now?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 476315] Review Request: evolution-mapi - Exchange 2007 support for Evolution

2008-12-15 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Matthew Barnes mbar...@redhat.com changed:

   What|Removed |Added

 Depends on||453083, 453395




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 476315] Review Request: evolution-mapi - Exchange 2007 support for Evolution

2008-12-15 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #5 from Matthew Barnes mbar...@redhat.com  2008-12-15 13:37:42 
EDT ---
evolution-mapi saw its first formal release today: 0.25.3.

Think I got the rpmlint issues from comment #2 worked out as well.

http://mbarnes.fedorapeople.org/mapi/SPECS/evolution-mapi.spec
http://mbarnes.fedorapeople.org/mapi/SRPMS/evolution-mapi-0.25.3-1.fc10.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 476315] Review Request: evolution-mapi - Exchange 2007 support for Evolution

2008-12-14 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Sandro Mathys r...@fedoraproject.org changed:

   What|Removed |Added

 CC||r...@fedoraproject.org




--- Comment #2 from Sandro Mathys r...@fedoraproject.org  2008-12-14 21:41:22 
EDT ---
rpmlint output when run on the resulting RPMs:

evolution-mapi.i386: E: library-without-ldconfig-postin
/usr/lib/libexchangemapi-1.0.so.0.0.0
evolution-mapi.i386: E: library-without-ldconfig-postun
/usr/lib/libexchangemapi-1.0.so.0.0.0
evolution-mapi.i386: E: zero-length /usr/share/doc/evolution-mapi-0.1/NEWS
evolution-mapi.i386: E: no-changelogname-tag
evolution-mapi-debuginfo.i386: E: no-changelogname-tag
evolution-mapi-devel.i386: W: no-documentation
evolution-mapi-devel.i386: E: no-changelogname-tag
3 packages and 0 specfiles checked; 6 errors, 1 warnings.

the no-changelogname-tag errors seem to be fixed in the spec file linked above,
but not in the spec file that is included in the SRPM.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 476315] Review Request: evolution-mapi - Exchange 2007 support for Evolution

2008-12-12 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #1 from Jason Tibbitts ti...@math.uh.edu  2008-12-12 20:31:33 EDT 
---
This fails to build for me on rawhide; it needs openchange-devel which is not
available.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review