[FFmpeg-devel] [PATCH 1/3] web: move RSS generation to a seperate shell script

[Stefano Sabatini]

On date Monday 2013-11-25 14:25:05 -0800, Timothy Gu encoded:
> Easier to read.
> 
> Signed-off-by: Timothy Gu <timothygu99 at gmail.com>
> ---
>  Makefile   | 20 +++-----------------
>  rss-gen.sh | 59 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
>  2 files changed, 62 insertions(+), 17 deletions(-)
>  create mode 100755 rss-gen.sh
> 
> diff --git a/Makefile b/Makefile
> index db2ac28..36e9818 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -18,22 +18,8 @@ htdocs/%.html: src/% src/%_title $(PAGE_DEPS)
>  	src/template_footer > $@
>  
>  htdocs/main.rss: htdocs/index.html
> -	echo '<?xml version="1.0" encoding="UTF-8" ?>' > $@
> -	echo '<rss version="2.0" xmlns:atom="http://www.w3.org/2005/Atom">' >> $@
> -	echo '<channel>' >> $@
> -	echo '    <title>FFmpeg RSS</title>' >> $@
> -	echo '    <link>http://ffmpeg.org</link>' >> $@
> -	echo '    <description>FFmpeg RSS</description>' >> $@
> -	echo '    <atom:link href="http://ffmpeg.org/main.rss" rel="self" type="application/rss+xml" />' >> $@
> -	grep '<a *id=".*" *></a><h3>.*20..,.*</h3>' $< | sed 'sX<a *id="\(.*\)" *> *</a> *<h3>\(.*20..\), *\(.*\)</h3>X\
> -    <item>\
> -        <title>\2, \3</title>\
> -        <link>http://ffmpeg.org/index.html#\1</link>\
> -        <guid>http://ffmpeg.org/index.html#\1</guid>\
> -    </item>\
> -X' >> $@
> -	echo '</channel>' >> $@
> -	echo '</rss>' >> $@
> -
> +	./rss-gen.sh start $@
> +	$(foreach html, $^, ./rss-gen.sh middle $@ $(html);)
> +	./rss-gen.sh end   $@
>  
>  .PHONY: all clean
> diff --git a/rss-gen.sh b/rss-gen.sh
> new file mode 100755
> index 0000000..a2edb07
> --- /dev/null
> +++ b/rss-gen.sh
> @@ -0,0 +1,59 @@
> +#!/bin/sh
> +
> +die() {
> +    cat <<EOT >&2
> +Usage: $0 <mode> <htdocs/output.rss> [<input.html>]
> +
> +This utility parses & converts generated FFmpeg news HTML page into RSS format.
> +"mode" can be begin, middle, or end. If mode is middle, then input HTML is
> +required.
> +EOT
> +    exit 1
> +}
> +
> +# Sanity checks and parsing command line
> +
> +MODE=$1
> +RSS=$2
> +HTML=$3
> +
> +HTML_NODIR=${HTML##*/}
> +
> +if test $# -lt 2; then
> +    echo 'Too few arguments' >&2
> +    die
> +elif test $# -gt 3; then
> +    echo 'Too many arguments' >&2
> +    die
> +elif test $MODE = "middle" && test $# -ne 3; then
> +    echo 'No HTML specified' >&2
> +    die
> +fi
> +
> +if test $MODE = "start"; then
> +    cat << EOT > $RSS
> +<?xml version="1.0" encoding="UTF-8" ?>
> +<rss version="2.0" xmlns:atom="http://www.w3.org/2005/Atom">
> +<channel>
> +    <title>FFmpeg RSS</title>
> +    <link>http://ffmpeg.org</link>
> +    <description>FFmpeg RSS</description>
> +    <atom:link href="http://ffmpeg.org/main.rss" rel="self" type="application/rss+xml" />
> +EOT
> +elif test $MODE = "middle"; then

> +    grep '<a *id=".*" *></a><h3>.*20..,.*</h3>' $HTML | sed 'sX<a *id="\(.*\)" *> *</a> *<h3>\(.*20..\), *\(.*\)</h3>X\

Note: "X" is a weird choice and will fail in case we add an "X" in the
quoted text.

> +    <item>\
> +        <title>\2, \3</title>\
> +        <link>http://ffmpeg.org/'$HTML_NODIR'#\1</link>\
> +        <guid>http://ffmpeg.org/'$HTML_NODIR'#\1</guid>\
> +    </item>\
> +X' >> $RSS
> +elif test $MODE = "end"; then
> +    cat << EOT >> $RSS
> +</channel>
> +</rss>
> +EOT
> +else
> +    echo 'Unknown mode' >&2
> +    die
> +fi

LGTM.
-- 
FFmpeg = Fanciful and Freak Minimal Patchable Esoteric Glue