[FFmpeg-devel] [PATCH] Include content of the news article in the website RSS

[Gerion Entrup]

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>
> 
> wrote:
> > > > 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
> 
> mailoutput.
> 
> 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 
main.rss.

> > > Just curious, did you test the feed with the W3C validator?
> > 
> > Not before you asked:
> > 
> > Test with [1]:
> > This document was successfully checked as well-formed XML!
> > One Warning: No DOCTYPE found! Checking XML syntax only.
> > 
> > In the list of common doctypes [2] 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.
> 
> [...]
> 
> Timothy
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel