[no subject]

Also,  I would appreciate it if the other code changes could be incorporated
for the next release so that I can use imported/included namespaces in a
complex schema I have for another project...  the more complex schema you
can refer to is at: http://github.com/oblivian/geda-xml/tree/master.

Thanks,
Jason

On Wed, Mar 25, 2009 at 5:36 AM, Daniel Veillard <veillard redhat com>wrote:

> On Wed, Mar 04, 2009 at 06:55:42PM -0600, Luke Robison wrote:
> > Thank you Jason!  I can also confirm validate_me.xml validates with this
> > patch applied, although patching 2.7.3 doesn't work, I had to grab the
> > svn version (currently r3819).  Thanks for the help.
>
>   I don't know that part of the code (was done by Kasimier some years
> ago), I'm a bit surprized that just removing the check leads to a
> correct solution. With or without the patch make check and make tests
> in xstc lead to the same results, so I have no way to assert if the
> patch introduce regressions.
>  If you could give it a bit more testing on different cases and report
> I would apply it based on your feedback.
>
>  thanks !
>
> Daniel
>
> --
> Daniel Veillard      | libxml Gnome XML XSLT toolkit  http://xmlsoft.org/
> daniel veillard com  | Rpmfind RPM search engine http://rpmfind.net/
> http://veillard.com/ | virtualization library  http://libvirt.org/
>

--000e0cd247e4df90930466063add
Content-Type: text/html; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable

Daniel,<br><br>From what I&#39;ve researched on other mailing lists.=A0 The=
 parsing of includes/imports and redefines of namespaces after they&#39;ve =
been included/imported or redefined already is allowed by the spec (or at l=
east it doesn&#39;t explicitly disallowed it).=A0 Most other parsers allow =
it, I think libxml2 is about the only one that doesn&#39;t.=A0 From the com=
ments in the code I removed it looks like Kasimier was just being careful, =
but it does remove functionality that is implicitly allowed by the specific=
ation.=A0 After further consideration, I don&#39;t think that it should imp=
act normal test results and that the patch should be good.=A0 The test case=
 Luke provided should probably be implemented in xstc which I don&#39;t hav=
e the time to do right now.=A0 If Luke could do it that would be great.<br>
<br>Also,=A0 I would appreciate it if the other code changes could be incor=
porated for the next release so that I can use imported/included namespaces=
 in a complex schema I have for another project...=A0 the more complex sche=
ma you can refer to is at: <a href=3D"http://github.com/oblivian/geda-xml/t=
ree/master">http://github.com/oblivian/geda-xml/tree/master</a>.<br>
<br>Thanks,<br>Jason<br><br><div class=3D"gmail_quote">On Wed, Mar 25, 2009=
 at 5:36 AM, Daniel Veillard <span dir=3D"ltr">&lt;<a href=3D"mailto:veilla=
rd redhat com">veillard redhat com</a>&gt;</span> wrote:<br><blockquote cla=
ss=3D"gmail_quote" style=3D"border-left: 1px solid rgb(204, 204, 204); marg=
in: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class=3D"im">On Wed, Mar 04, 2009 at 06:55:42PM -0600, Luke Robison wr=
ote:<br>
&gt; Thank you Jason! =A0I can also confirm validate_me.xml validates with =
this<br>
&gt; patch applied, although patching 2.7.3 doesn&#39;t work, I had to grab=
 the<br>
&gt; svn version (currently r3819). =A0Thanks for the help.<br>
<br>
</div> =A0I don&#39;t know that part of the code (was done by Kasimier some=
 years<br>
ago), I&#39;m a bit surprized that just removing the check leads to a<br>
correct solution. With or without the patch make check and make tests<br>
in xstc lead to the same results, so I have no way to assert if the<br>
patch introduce regressions.<br>
 =A0If you could give it a bit more testing on different cases and report<b=
r>
I would apply it based on your feedback.<br>
<br>
 =A0thanks !<br>
<br>
Daniel<br>
<font color=3D"#888888"><br>
--<br>
Daniel Veillard =A0 =A0 =A0| libxml Gnome XML XSLT toolkit =A0<a href=3D"ht=
tp://xmlsoft.org/" target=3D"_blank">http://xmlsoft.org/</a><br>
<a href=3D"mailto:daniel veillard com">daniel veillard com</a> =A0| Rpmfind=
 RPM search engine <a href=3D"http://rpmfind.net/"; target=3D"_blank">http:/=
/rpmfind.net/</a><br>
<a href=3D"http://veillard.com/"; target=3D"_blank">http://veillard.com/</a>=
 | virtualization library =A0<a href=3D"http://libvirt.org/"; target=3D"_bla=
nk">http://libvirt.org/</a><br>
</font></blockquote></div><br>

--000e0cd247e4df90930466063add--