junhu619 / pydelicious

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

Allow for importing of pydelicious without necessarily having feedparser lib #20

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
A large chunk of the pydelicious library - especially the API portion has 
no dependency on feedparser  (from what I can see the feedparser module is 
only used in dlcs_rss_request). However, because  the 'import feedparser' 
statement is within the module scope - the feedparser module is always 
required.

I was wondering if it be possible that we move the import feedparser 
statement to within the dlcs_rss_request function? I do think allowing for 
segments of pydelicious to be used with the minimal of dependencies, will 
make it an even more attractive library. What do you think?

Thanks for this excellent library.

Mohan

Original issue reported on code.google.com by mohangk@gmail.com on 26 Oct 2008 at 3:27

GoogleCodeExporter commented 9 years ago
I see your point and think the same way about the code (but as I'm not using 
the RSS
code either I'mnot sure if the data is really that interesting)

But i've made the feedparser optional (this WILL let a unittest or two fail for 
now..)

Original comment by berend.v...@gmail.com on 28 Nov 2008 at 4:10