[Bug 725909] Review Request: php53-mapi - The PHP 5.3 MAPI extension by Zarafa

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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=725909

--- Comment #2 from Robert Scheck <redhat-bugzilla@xxxxxxxxxxxx> 2011-07-27 19:36:47 EDT ---
Regarding a) + b): I fired some test builds and I will remove the additional 
CFLAGS in the Zarafa package for the next build and thus in php53-mapi, too.
Is it okay, if I apply this change after import into git? It's the removal of
the "export CFLAGS=..." line in the end. I would treat this as minor change.

Yes, there are locales in po subdirectory, but they're built to *.mo with the
build of the main zarafa package and packaged into zarafa-client subpackage.
The php53-mapi package depends on zarafa-client as follows:

 1. php53-mapi requires libmapi.so.0()(64bit)
 2. libmapi provides libmapi.so.0()(64bit)
 3. libmapi requires zarafa-client (because it dlopen()s libzarafaclient.so)
 4. zarafa-client provides *.mo (and thus the compiled *.po files)
 5. The gettext domain that is used everywhere is zarafa, not php53-mapi

That should eliminate any need to handle them twice. On the other hand, I was
unable to find a direct requirement for gettext in php-ext directory. If there
is a requirement, it's satisfied. If there is no requirement it is satisfied
anyway, because at least libzarafaclient.so requires *.mo stuff and libmapi.so
depends on libzarafaclient.so (until there is no alternative implementation).

Was I able to clarify all your questions?

-- 
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.
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]