On Thu, May 28, 2015 at 10:18:06AM +0000, Emil Nordling wrote:
> Hi Dave,
> I couldn't find an issue tracker for generateDS, so I'm writing you like this
> i
> nstead. I apologize in advance.
>
> I got this:
> NameError: global name 'XMLParser_import_library' is not defined
> when I parsed an erroneous xml-document with code generated with generateDS
> 2.1
> 5b.Â
>
>
> It seems to come from line 5233 in generateDS.py:
> 5232 def raise_parse_error(node, msg):
> 5233 if XMLParser_import_library == XMLParser_import_lxml:
Emil,
Thanks for catching this. It looks like that is code left over from
when generateDS.py was able to use ElementTree, whereas currently
Lxml is required. So, I've fixed it by removing that test.
I've attached a patch file, in case you have not already fixed your
copy of generateDS.py. This fix will be in the next version of
generateDS.py.
Thanks again.
>
> [...],
> where both names are used without being defined.
>
> In all else: Thank you for a very nice library!
Great. I appreciate the comment. Hope it's helpful.
Dave
--
Dave Kuhlman
http://www.davekuhlman.org
diff -r 3ef0afb0b755 generateDS.py
--- a/generateDS.py Wed May 20 19:20:19 2015 -0700
+++ b/generateDS.py Thu May 28 08:47:36 2015 -0700
@@ -5228,11 +5228,7 @@
def raise_parse_error(node, msg):
- if XMLParser_import_library == XMLParser_import_lxml:
- msg = '%%s (element %%s/line %%d)' %% (
- msg, node.tag, node.sourceline, )
- else:
- msg = '%%s (element %%s)' %% (msg, node.tag, )
+ msg = '%%s (element %%s/line %%d)' %% (msg, node.tag, node.sourceline, )
raise GDSParseError(msg)
------------------------------------------------------------------------------
_______________________________________________
generateds-users mailing list
generateds-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/generateds-users
