Re: [xslt] create Directory bug in libxslt for W32



Aleksey Gurtovoy writes:
> Daniel Veillard writes:
> > On Tue, Nov 30, 2004 at 03:10:38PM -0600, Aleksey Gurtovoy wrote:
> > > Thomas Fischer writes:
> > > > Hi folks,
> > > > i found a bug in libxslt and Daniel Veillard told me, that's
> > > > a win only bug and he can't help   :o<
> > > > 
> > > > The Bug occurs, when i try to create a directory via command
> > > > line parameter --output (like "xsltproc -o does_not_exist/res ...")
> > > > or via <exslt:document href="does_not_exist/res"
> > > > 
> > > > Anybody knows a workaround or patch for this problem?
> > > 
> > > See the attachments for the latter (against the current CVS).
> > 
> >   I do not understand the first patch nor the associated
> > side effects. On windows the file path separator is '\' why is this
> > wrong and need to be removed.
> 
> I agree that the patch is a little controversial, but only due to the fact 
> that 'xmlParserGetDirectory' is used to extract directory paths from both 
> URI/URLs _and_ file system paths. For the former the running platform is 
> irrelevant; for the latter it's not (obviously). I'm not that familiar with
> the libxslt/libxml2 codebase to be able to say in how many cases when a
> function is passed a 'filename' argument it actually _is_ a native 
> filesystem path (as opposite to a URL), but I assume that at least some 
> of them are. Of course you're in much better position to answer this.
> 
> In any case, my take on the issue is that, ideally, all current calls
> to 'xmlParserGetDirectory' should become calls to either 
> 'xmlParserGetURIDirectory' or 'xmlParserGetFileDirectory' (both currently 
> non-existent), the latter differing from the former in that it would 
> respect the running platform's conventions.

Or we can simply make 'xmlParserGetDirectory' on Windows to respect both
forward and backward slashes ('/' is not a valid path symbol on Windows).

--
Aleksey Gurtovoy
MetaCommunications Engineering



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