[Kolab-devel] Re: steffen: server/kolab-resource-handlers/kolab-resource-handlers/fbview/fbview/framework/Net_IMAP IMAP.php, NONE, 1.1 IMAPProtocol.php, NONE, 1.1 package.xml, NONE, 1.1

[Steffen Hansen]

On Tuesday 21 September 2004 17:30, Stuart K. Bingë wrote:
> Hi Steffen,
>
> On Tuesday, 21 September 2004 16:33, cvs at intevation.de wrote:
> > Author: steffen
> >
> > Update of
> > /kolabrepository/server/kolab-resource-handlers/kolab-resource-hand
> >lers/fbv iew/fbview/framework/Net_IMAP In directory
> > doto:/tmp/cvs-serv20540/Net_IMAP
> >
> > Added Files:
> >  IMAP.php IMAPProtocol.php package.xml
> > Log Message:
> > imported Net_IMAP
> >
> > [...]
>
> Is this really necessary? Wouldn't it make more sense to just pull in
> the tarball from PEAR in the kolab-resource-handlers RPM spec?

To me it looks like there are already heaps of PEAR packages in the 
framework dir in CVS, so installing one in a different way than the 
others is no good solution either.

> The problem I have with this is that the webclient also depends on
> Net_IMAP, and I can see it's going to get messy when trying to
> install the webclient through an RPM if a copy of Net_IMAP is already
> installed.
>
> I've already uncovered some bugs in Net_IMAP that need fixing for the
> webclient to function correctly (they don't affect the resmgr code)
> and now we're going to have to fix these bugs in two separate places.
> Either that, or we'll need to maintain two separate copies of
> Net_IMAP under the /kolab hierarchy; one for the resmgr and one for
> the webclient.

If a new release if Net_IMAP is made, I'll be happy to import that into 
CVS.

> To me it just seems to be a very messy solution to include it here
> :-/

I _do_ agree with you that it is not really nice. The proper solution 
would be to build an rpm from each PEAR package we use, but I don't 
have time for that. Volunteers are welcome though :-)

regards
-- 
Steffen Hansen          |       Klarälvdalens Datakonsult AB
Senior Software Engineer|       http://www.klaralvdalens-datakonsult.se
                        |
                        |       Platform-independent
                        |       software solutions