Re: [sab-php] bughunting

At 15:11 8-10-2002, you wrote:

Melvyn Sopacua wrote:

Problem is: which binary is the reference for CWD?

The cwd is set according to an executable that loads and executes the sablib code. To work with modules running scripts, a convenient default base should be set to override the cwd. This is what you do now, aren't you?

Nope. This is what we do, when you tell which binary is the reference :-)

I think the cwd implementation is the correct one, as it's PHP's task to set the current working directory to the top-level script being executed. Is this change in CVS?

Yes, a couple of releases already. Just my "theory" has been out-of-date.

Good. Judging from David's message, the base should be a file, not a path and you call dirname on the base. Is this subject to change/discussion? I think it's more intuitive to set a base dir, rather than a base file, even though it's more work to do so in userland. We can make this a doc issue though, but that would also imply that we would stat the argument of xslt_set_base in php and issue a warning when it's a directory as a curtosy to the user now that we have the fancy php_error_docref :-).

4.3 release is aimed at 1st week november.

Looking at current developments, this estimation of mine, may be a little optimistic, but time will tell. It's unlikely to be sooner IMO.

OK. If we have a new release candidate, we will make it available for tests before the release.

Thanx! I'll keep sablot cvs in sync with any tests/modifications we'll produce and keep you posted (on this and other issues of course :-).

-- Petr Cimprich Ginger Alliance www.gingerall.com ------------------------------------------------------------------------ Archives and info: http://www.gingerall.org/charlie/ga/xml/m_ml.xml Mailing list maintained by Ginger Alliance

Best regards, Melvyn Sopacua WebMaster IDG.nl _/_/_/_/_/_/_/_/_/_/_/_/_/_/_/_/_/_/_/_/_/_/_/_/_/_/_/_/_/ If it applies, where it applies - this email is a personal contribution and does not reflect the views of my employer IDG.nl. \_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\_\