search

nolenfelten / xbmc-addons

Automatically exported from code.google.com/p/xbmc-addons
0 stars 0 forks source link

Patch to hulu plugin to consolidate subscriptions, queue support; adds clips support to subscriptions #11

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
1) cleaned up some regular expressions in subscriptions
2) added support for clips into subscriptions (note: only tested on Mac OSX)
3) other general clean up in subscriptions

Attached is a diffs file to go from r655 to incorporating the above. I've also 
attached a ZIP of the 
whole thing.

Original issue reported on code.google.com by jonm42@gmail.com on 24 Feb 2009 at 1:22

Attachments:

GoogleCodeExporter commented 9 years ago 
Actually from r656,

Original comment by jonm42@gmail.com on 24 Feb 2009 at 1:45

GoogleCodeExporter commented 9 years ago 
TortoiseSVN puked on the diff, when I applied it manually the plugin crashed.

Once we get hulu running decently again I'll look into this.

Original comment by rwparris2 on 26 Feb 2009 at 1:42

GoogleCodeExporter commented 9 years ago 
[deleted comment]
GoogleCodeExporter commented 9 years ago 
Please use the attached ZIP vs. the one above; I completely refactored the 
_subscriptions and _rss modules. The base version is the hulu-air.zip attached 
to 
issue 15.

(Note -- I will be attaching this once I get home. Thanks.)

Original comment by jonm42@gmail.com on 6 Mar 2009 at 9:03

GoogleCodeExporter commented 9 years ago 
This is the one to try for the subscriptions, queue, and reworked RSS. Some of 
the RSS are acting a bit wonky, 
but work fine when I paste them directly, so you might want to skip that part.

Original comment by jonm42@gmail.com on 7 Mar 2009 at 3:10

Attachments:

GoogleCodeExporter commented 9 years ago 
Turns out the wonky is general in the rss somewhere, not just my rework. will 
figure it out and replace rss.

Original comment by jonm42@gmail.com on 7 Mar 2009 at 1:07

GoogleCodeExporter commented 9 years ago 
What exactly is 'wonky' about the RSS stuff?  I've never noticed anything, but 
that
certainly doesn't mean much because I rarely use them.

Post a diff of only working stuff and I'll commit it.

Original comment by rwparris2 on 8 Mar 2009 at 9:00

GoogleCodeExporter commented 9 years ago 
Wonky means sometimes it grabs and parses the feed and other times it just 
doesn't. More details are here: 
(http://xbmc.org/forum/showpost.php?p=294252&postcount=40). The baseline for 
this is r655 + the AIR 
changes as posted. I've also included just a ZIP of the whole Hulu directory in 
case diff chokes on you again.

Original comment by jonm42@gmail.com on 8 Mar 2009 at 6:42

Attachments:

GoogleCodeExporter commented 9 years ago 
I've saved the diffs but I'm going to mark this as "won't fix" until we get 
Hulu up
and running again.

Original comment by rwparris2 on 26 Mar 2009 at 9:23

GoogleCodeExporter commented 9 years ago 
Unfortunate, but makes sense. Thanks for the update.

Original comment by jonm42@gmail.com on 26 Mar 2009 at 10:11