Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

python3 and python2 version parsed a feed differently #62

Closed
entron opened this Issue Mar 30, 2016 · 4 comments

Comments

Projects
None yet
3 participants

entron commented Mar 30, 2016

This feed:

http://eatcodeplay.com/feed.xml

contains errors. However python2 version can find several entries while python3 version does not find any entries.

mmcdole commented Apr 19, 2016

@entron I can't speak to why python2 is able to parse it, but there is something seriously wrong with the feed:

<?xml version="1.0" encoding="UTF-8"?>
<feed/>
<feed xmlns="http://www.w3.org/2005/Atom">
  <title></title>
  <subtitle></subtitle>
  <id>http://eatcodeplay.com</id>
  <link href="http://eatcodeplay.com"/>
  <link href="http://eatcodeplay.com/feed.xml" rel="self"/>
  <updated>2016-03-29T05:52:00-07:00</updated>
  <author>
    <name>Real Scout Engineering Team</name>
  </author>
  <entry>
    <title>Decoding Webpack Config</title>
    <link rel="alternate" href="http://eatcodeplay.com/decoding-webpack-config/"/>
    <id>http://eatcodeplay.com/decoding-webpack-config/</id>
    <published>2016-03-29T05:52:00-07:00</published>
    <updated>2016-03-30T11:27:07-07:00</updated>
    <author>
      <name>Real Scout Engineering Team</name>
    </author>
    <summary type="html">This is Part 2 of our Transition to React post.  In the first part we talked about the big picture of setting up our stack to take a piecemeal approach to transition to React.  One of the sections that could take a further look is how we setup our webpack</summary>
  </entry>
</feed>

It has a self-closing feed tag right at the top of the feed. Yuck.

Owner

kurtmckee commented Apr 24, 2016

@entron You need sgmllib3k

pip3 install sgmllib3k

I'll add some documentation about this. Thanks for reporting this!

Owner

kurtmckee commented Apr 24, 2016

@entron I double-checked and sgmllib3k is already a dependency in the develop branch. It should be installed automatically when the next version is released. In the meantime you might consider using the develop branch. =)

@kurtmckee kurtmckee closed this Apr 24, 2016

entron commented Apr 24, 2016

Thanks for the solution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment