You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@shindig.apache.org by Paul Lindner <li...@inuus.com> on 2010/04/28 19:54:12 UTC
Re: Shindig Feed Processor: Parse MediaRSS items into optional Media
element (issue1019041)
Looks good to me.
I can fix up the tabs vs spaces.
Anyone opposed to adding rome-modules as a dependency?
On Tue, Apr 27, 2010 at 6:19 AM, <ma...@gmail.com> wrote:
> Apologies, uploaded wrong patch set
>
>
> http://codereview.appspot.com/1019041/show
>
Fwd: Shindig Feed Processor: Parse MediaRSS items into optional Media
element (issue1019041)
Posted by Paul Lindner <li...@inuus.com>.
---------- Forwarded message ----------
From: Paul Lindner <li...@inuus.com>
Date: Wed, Apr 28, 2010 at 10:54 AM
Subject: Re: Shindig Feed Processor: Parse MediaRSS items into optional
Media element (issue1019041)
To: mat.mannion@gmail.com, shindig.remailer@gmail.com, lindner@inuus.com,
reply@codereview.appspotmail.com
Looks good to me.
I can fix up the tabs vs spaces.
Anyone opposed to adding rome-modules as a dependency?
On Tue, Apr 27, 2010 at 6:19 AM, <ma...@gmail.com> wrote:
> Apologies, uploaded wrong patch set
>
>
> http://codereview.appspot.com/1019041/show
>
Re: Shindig Feed Processor: Parse MediaRSS items into optional Media
element (issue1019041)
Posted by Paul Lindner <li...@inuus.com>.
+dev@shindig.apache.org
Getting one more round of feedback, if no objections I'll commit this
tomorrow via lazy consensus.
On Wed, Apr 28, 2010 at 10:54 AM, Paul Lindner <li...@inuus.com> wrote:
> Looks good to me.
>
> I can fix up the tabs vs spaces.
>
> Anyone opposed to adding rome-modules as a dependency?
>
>
> On Tue, Apr 27, 2010 at 6:19 AM, <ma...@gmail.com> wrote:
>
>> Apologies, uploaded wrong patch set
>>
>>
>> http://codereview.appspot.com/1019041/show
>>
>
>