For Programmers: Free Programming Magazines  


Home > Archive > PHP PEAR Questions and Answers > March 2007 > Smb. with CVS access please apply patch for File_SearchReplace









You are viewing an archived Text-only version of the thread. To view this thread in it's original format and/or if you want to reply to this thread please [click here]

 

Author Smb. with CVS access please apply patch for File_SearchReplace
techtonik

2007-03-19, 7:04 pm

Hello.

There is a patch for File_SearchReplace in
http://pear.php.net/bugs/bug.php?id=10395
I've reviewed it and it is ok, but at the moment I do not have CVS access.

Could somebody commit the patch and release version 1.1.2 of the
package? Seems like the format of the package.xml is changed and I do
not have local copy of PEAR to port and repack it accordingly.

Thanks in advance.

--
--anatoly t.
Greg Beaver

2007-03-20, 7:02 pm

techtonik wrote:
> Hello.
>
> There is a patch for File_SearchReplace in
> http://pear.php.net/bugs/bug.php?id=10395
> I've reviewed it and it is ok, but at the moment I do not have CVS access.
>
> Could somebody commit the patch and release version 1.1.2 of the
> package? Seems like the format of the package.xml is changed and I do
> not have local copy of PEAR to port and repack it accordingly.
>
> Thanks in advance.


Why don't you have CVS access? Perhaps Martin can grant it since you're
a lead.

In addition, you must install a local copy of PEAR, otherwise you can't
be certain that the package will actually install, this is a basic
pre-requisite of developing PEAR packages. You can't, for instance,
test installation through the PEAR installer without having it installed.

Greg
techtonik

2007-03-20, 7:02 pm

On 3/20/07, Greg Beaver <cellog@php.net> wrote:

> Why don't you have CVS access? Perhaps Martin can grant it since you're
> a lead.


Thanks, but PHP account is not an issue. I am firewalled and can't
make CVS work through a proxy.

> In addition, you must install a local copy of PEAR, otherwise you can't
> be certain that the package will actually install, this is a basic
> pre-requisite of developing PEAR packages. You can't, for instance,
> test installation through the PEAR installer without having it installed.


I am pretty sure the patch is trivial and the file will work. I just
do not have time to study differences between package formats and port
it myself.

--
--anatoly t.
techtonik

2007-03-25, 6:59 pm

Finally I've got a shell access to make a release. Buggy PEAR
interface took me a lot of time to apply simple patch and CVS messed
up linefeeds in the commit. Moreover PEAR version 1.4.9 refused to
convert the package description files and I am unable to release it.

$ pear convert
Logger must be compatible with PEAR_Common::log

I attach it to this letter in hope somebody will release it, because I
do not have time anymore.


On 3/20/07, Greg Beaver <cellog@php.net> wrote:
> techtonik wrote:
>
> Why don't you have CVS access? Perhaps Martin can grant it since you're
> a lead.
>
> In addition, you must install a local copy of PEAR, otherwise you can't
> be certain that the package will actually install, this is a basic
> pre-requisite of developing PEAR packages. You can't, for instance,
> test installation through the PEAR installer without having it installed.
>
> Greg
>



--
--anatoly t.

Sponsored Links







Also available: Server administration forum archive | Web Design forum archive | Software forum archive | Hardware reviews archive

Copyright 2008 codecomments.com