[FFmpeg-devel] [PATCH] Include content of the news article in the website RSS
Timothy Gu
timothygu99 at gmail.com
Thu Aug 28 16:23:53 CEST 2014
On Thu, Aug 28, 2014 at 7:26 AM, Alexander Strasser <eclipse7 at gmx.net> wrote:
> Hi!
>
> On 2014-05-23 00:28 +0200, Alexander Strasser wrote:
>> On 2014-05-11 01:37 +0200, Gerion Entrup wrote:
>> > Am Donnerstag 01 Mai 2014, 23:48:29 schrieb Alexander Strasser:
>> > > alternative. What do you and/or others think of it?
>> > >
>> > > Besides validation I couldn't yet properly test what feed
>> > > readers make of it. So please forgive me if it causes obvious
>> > > problems with your reader.
>> > (As said offline already) My reader works with it.
>
> My patch still works I think. I would like to commit it,
> though it won't be flexible enough to add much other stuff
> that needs more elaborate logic. I want to apply it, because
> it is a neat feature to have now.
Good to hear you are still working on this.
> Any objections to this?
LGTM from me, assuming it is tested and works. I slightly prefer your
patch over Gerion's, but both are OK.
>
> In the not-so-short term I intend to widen security checks
> for the repo on the server to other files than just the Makefile.
> I already started this work, but need to fix one known problem
> before I can put it to work on the server.
Yes, that would be much better. The awk and sed hack is getting too
cumbersome. Or we could just change the server host.
Timothy
More information about the ffmpeg-devel
mailing list