[gpodder-devel] Woodchuck patches: Add more hooks
- From: neal at walfield.org (Neal H. Walfield)
- Date: Mon, 01 Aug 2011 17:06:23 +0200
Hi, Thomas,
At Mon, 1 Aug 2011 11:39:24 +0200,
Thomas Perl wrote:
> On Fri, Jul 29, 2011 at 05:48:03PM +0200, Neal H. Walfield wrote:
> > At Thomas' request on IRC: my repository with the Woodchuck changes is
> > available at:
> >
> > http://hssl.cs.jhu.edu/~neal/woodchuck/ports/gpodder.git/
> >
> > The changes for Woodchuck are on the 'woodchuck' branch. (I haven't
> > yet rebased the woodchuck integration patch, but the required
> > infrastructure changes are there and tested.)
>
> Here is some feedback on your commits:
>
> * 97165b78eaca440fc17264ddcadd549303f18eba (Add more hooks)
>
> This one looks good to me already; about that "XXX" - yes, you
> probably want to call the hook on every downloaded episode (i.e.
> get_downloaded_episodes()). After that, it's ready for a merge.
I've taken care of this and tested it. You can see the patch here:
http://hssl.cs.jhu.edu/~neal/woodchuck/ports/gpodder/commits/fba5aa5044e259b32c4f639ba38cb3279375dfb8/
Or, pull the woodchuck branch of:
http://hssl.cs.jhu.edu/~neal/woodchuck/ports/gpodder.git
Thanks,
Neal
Other related posts:
- » [gpodder-devel] Woodchuck patches: Add more hooks - Neal H. Walfield
