[FFmpeg-devel] [PATCH] Include content of the news article in the website RSS
gerion.entrup at t-online.de
Thu Apr 24 02:41:34 CEST 2014
Am Dienstag, 22. April 2014, 17:42:57 schrieb Timothy Gu:
> On Apr 22, 2014 5:37 PM, "Gerion Entrup" <gerion.entrup at t-online.de> wrote:
> > Am Montag 21 April 2014, 18:07:04 schrieb Timothy Gu:
> > > On Apr 11, 2014 3:29 PM, "Gerion Entrup" <gerion.entrup at t-online.de>
> > > > Fixes Trac ticket #844
> > > > ---
> > > >
> > > > Makefile | 26 +++++++++++++++++++++-----
> > > > 1 file changed, 21 insertions(+), 5 deletions(-)
> > >
> > > Sorry for the delay.
> > No problem.
> > > The patch looks good, but a bonus point will be if you
> > > remove the trailing whitespace on some lines.
> > It seems that this comes from kmail. I attached the original git
> No, not the patch itself. In the output feed there are some trailing spaces.
Send a new patch. (It checks for empty lines and don't print them in the
> > > Just curious, did you test the feed with the W3C validator?
> > Not before you asked:
> > Test with :
> > This document was successfully checked as well-formed XML!
> > One Warning: No DOCTYPE found! Checking XML syntax only.
> > In the list of common doctypes  RSS do not appear.
> See http://validator.w3.org/feed/
Ah, ok. The feed is valid, although the validator recommend to set a
description before the content and don't use indirect links. Both are not
trivial to implement, I would say.
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
More information about the ffmpeg-devel