Skip to content
This repository has been archived by the owner on Sep 14, 2018. It is now read-only.

Implement pyexpat module #393

Open
ironpythonbot opened this issue Dec 9, 2014 · 12 comments
Open

Implement pyexpat module #393

ironpythonbot opened this issue Dec 9, 2014 · 12 comments
Assignees
Labels

Comments

@ironpythonbot
Copy link

When trying to use the Parse function from xml.dom.minidom I got the next
message:
can't import expatbuilder from xml.dom

Work Item Details

Original CodePlex Issue: Issue 20023
Status: Active
Reason Closed: Unassigned
Assigned to: slide_o_mix
Reported on: Dec 1, 2008 at 8:32 PM
Reported by: zacnoam
Updated on: Nov 7 at 10:52 AM
Updated by: tcalmant

Plaintext Attachments

CodePlex Issue #20023 Plain Text Attachments

@ironpythonbot
Copy link
Author

On 2009-01-27 03:41:34 UTC, harrypierson commented:

Currently, there is no IPy implementation of pyexpat module, which expatbuilder depends on. Changing the name of this issue to reflect the need to implement pyexpat

@ironpythonbot
Copy link
Author

On 2009-01-28 08:57:02 UTC, jdhardy commented:

FePY includes a mostly-complete implementation that has worked for my purposes.

@ironpythonbot
Copy link
Author

On 2009-02-02 13:37:31 UTC, j2uiles commented:

Need to be able to parse strings to DOM.

@ironpythonbot
Copy link
Author

On 2009-05-01 03:57:45 UTC, dhgoldman commented:

I'm trying to migrate my CPython scripts to IronPython, but any script that uses xml.dom.minidom explodes at the "parseString" method. Any chance of these getting fixed in the near future?

@ironpythonbot
Copy link
Author

On 2010-03-13 06:18:14 UTC, judell commented:

This is working for me in the standard IronPython 2.6's included Lib

Put the FePy versions of expatbuilder.py and pyexpat.py into Lib/xml/dom

Note: The version of expatbuilder.py I found had stuff like:

class FilterVisibilityController(NewStyle):

I had to change these to:

class FilterVisibilityController()

Put the FePy version of expat.py into Lib/xml/parsers

@ironpythonbot
Copy link
Author

On 2010-07-09 07:10:49 UTC, judell commented:

I just did this in another environment. CPython is in c:/python26, and IronPython is in :c:/program files/ironpython 2.6"

Take
https://fepy.svn.sourceforge.net/svnroot/fepy/trunk/lib/pyexpat.py, rename to expat.py, place in c:/program files/ironpython 2.6/lib/xml/parsers
Copy c:/python26/lib/xml/dom/expatbuilder.py to c:/program files/ironpython 2.6/lib/xml/dom

@ironpythonbot
Copy link
Author

On 2011-08-31 23:44:49 UTC, jdhardy commented:

I've attached the partial implementation from FePy for anyone who wants to use it.

@ironpythonbot
Copy link
Author

On 2012-05-22 23:55:58 UTC, slide_o_mix commented:

Initial implementatiton in 8446d8b

@tarunlalwani
Copy link

@ironpythonbot , any chances this will be added to the main release?

@jdhardy
Copy link
Member

jdhardy commented Aug 7, 2015

It's not a complete implementation, unfortunately, which is probably worse
than not including one at all. Doing it exactly right probably means
wrapping the C Expat library.

On Thu, Jul 9, 2015 at 2:02 AM, Tarun Lalwani [email protected]
wrote:

@ironpythonbot https://github.com/ironpythonbot , any chances this will
be added to the main release?


Reply to this email directly or view it on GitHub
#393 (comment).

@slide
Copy link
Contributor

slide commented Aug 7, 2015

Wrapping the C library doesn't sound very cross platform friendly...

On Thu, Aug 6, 2015 at 9:46 PM Jeff Hardy [email protected] wrote:

It's not a complete implementation, unfortunately, which is probably worse
than not including one at all. Doing it exactly right probably means
wrapping the C Expat library.

On Thu, Jul 9, 2015 at 2:02 AM, Tarun Lalwani [email protected]
wrote:

@ironpythonbot https://github.com/ironpythonbot , any chances this
will
be added to the main release?


Reply to this email directly or view it on GitHub
<#393 (comment)
.


Reply to this email directly or view it on GitHub
#393 (comment).

@jdhardy
Copy link
Member

jdhardy commented Aug 7, 2015

Well, we could just load the system one on Linux/MacOS and use and embedded
one on Windows (store as a resource, load 32/64 bit as appropriate). The
mobile platforms get a bit weirder but should be supportable as well (not
that those are well supported anyway). I have thoughts on how to do this
but just not the time to actually implement them. :(

On Thu, Aug 6, 2015 at 10:02 PM, Alex Earl [email protected] wrote:

Wrapping the C library doesn't sound very cross platform friendly...

On Thu, Aug 6, 2015 at 9:46 PM Jeff Hardy [email protected]
wrote:

It's not a complete implementation, unfortunately, which is probably
worse
than not including one at all. Doing it exactly right probably means
wrapping the C Expat library.

On Thu, Jul 9, 2015 at 2:02 AM, Tarun Lalwani [email protected]
wrote:

@ironpythonbot https://github.com/ironpythonbot , any chances this
will
be added to the main release?


Reply to this email directly or view it on GitHub
<
#393 (comment)
.


Reply to this email directly or view it on GitHub
<#393 (comment)
.


Reply to this email directly or view it on GitHub
#393 (comment).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

4 participants