libo26 / feedparser

Automatically exported from code.google.com/p/feedparser
Other
0 stars 0 forks source link

Some microformat-related tests fail with current BeautifulSoup versions #234

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Current versions of BeautifulSoup cause four unit tests to fail. I've found and 
fixed the source of the problem, and have tested it across every combination of 
Python 2.4, 2.5, 2.6, and 2.7 with BeautifulSoup 3.0.8, 3.0.8.1, and 3.2.0. (I 
skipped the 3.1 series entirely because the author considers it a failed 
experiment.)

If it's helpful, I've additionally pushed a git branch to github:
https://github.com/kurtmckee/feedparser/tree/beautifulsoup

If there are additional considerations or changes that would make this patch 
more acceptable, please don't hesitate to let me know. It applies cleanly to 
svn trunk (currently r316).

Original issue reported on code.google.com by kurtmckee on 30 Nov 2010 at 10:27

Attachments:

GoogleCodeExporter commented 9 years ago
I applied the patch in revision 317.

Original comment by adewale on 1 Dec 2010 at 1:22

GoogleCodeExporter commented 9 years ago
I just ran into this bug again; the two elmAgent lines shown in the patch need 
to be removed or the bug persists.

Original comment by kurtmckee on 2 Dec 2010 at 6:39

GoogleCodeExporter commented 9 years ago
Looks like I did this manually rather than applying the patch. I've fixed it 
now in revision 328

Original comment by adewale on 4 Dec 2010 at 11:26