Hi Sidnei, Martijn,
Sidnei da Silva wrote:
> On Mon, Sep 18, 2006 at 09:30:21PM -0300, Sidnei da Silva wrote:
> | On Mon, Sep 18, 2006 at 06:34:31PM +0200, Martijn Faassen wrote:
> | | It would of course be good if we didn't do this by default and that this
> | | should be enabled explicitly, as it's a potential security concern to
> | | pull in stylesheets from URLs. I'm not sure right now what the API would
> | | be like at all though in the first place.
I think the right API would simple be a call to ElementTree.xslt() without
stylesheet. Maybe we should then raise an exception if no PI is found.
> I actually tried to go ahead and create a patch for this but lxml
> doesn't even compile here.
>
> On Win32, with Pyrex 0.9.4.1 (with the patch applied) I get this error:
>
> src\lxml\etree.c(49704) : error C2137: empty character constant
> error: command '"C:\Arquivos de programas\Microsoft Visual C++ Toolkit
> 2003\bin\cl.exe"' failed with exit status 2
>
> The line in question has:
>
> if (*t->s == '^@')
> continue; /* shortcut for erased string entries */
That's a known bug. Please use the updated Pyrex version from the lxml SVN
(see build.txt).
> On Ubuntu Edgy, with Pyrex 0.9.3 it doesn't work either.
Sure, it can't handle some of the newer lxml features (and may even have some
bugs that prevent it from compiling lxml at all, even older versions).
Stefan
|
