[FFmpeg-devel] [PATCH] web: Copy FFmpeg Logo by Makefile instead of cronjob

Michael Niedermayer michaelni at gmx.at
Sat Apr 7 01:36:37 CEST 2012

On Fri, Apr 06, 2012 at 11:53:16PM +0200, Thilo Borgmann wrote:
> Am 06.04.12 17:43, schrieb Michael Niedermayer:
> >> +LOGO_FILES := $(wildcard src/logik/$(DATE).*)
> > 
> > what will happen if theres a
> > src/logik/0605.`rm -rf .`$(SHELL rm -rf .)
> > ?
> for `` files should be no target present (which I could also test...)
> for $(shell...) things is that I don't think they get expanded any time after
> being assigned.
> I'm not familiar with usual hacks but I doubt someone who can write such a file
> is not able to manually rm -rf anything in there already.

well, you can add files with a git account, you cant directly do a
rm -rf with a git account
and git definitly allows you to put funny chars in the names.

> Does not matter anyway, we can change that line to something like:
> LOGO_FILES := $(wildcard src/logik/$(DATE).png src/logik/$(DATE).hover)
> According to the targets. Save enough?

looks ok to me but iam no makefile security expert ...

> >>  src/logik/$(DATE)-standard:
> >>  	cmp htdocs/FFmpeg_standard.png htdocs/ffmpeg-logo.png || \
> >>  	cp htdocs/FFmpeg_standard.png htdocs/ffmpeg-logo.png
> >> +	echo "FFmpeg" > src/template_head2_hovertext
> > 
> > files in /src should not be written to, its a nightmare to maintain
> > and in this case will not even work as the git hook will break on the
> > next checkout
> Hm this is less easy to solve... What exactly does that git hook do?

it updates the checkout :)
and if theres a changed file in there it will fail. (your patch
contained the file too...)

> Maybe it's sufficient and more suitable then to have an intermediate directory
> for dynamically generated parts of the website?

cant the hover text be changed by CSS and the whole CSS file be changed
like the logo ?


Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

When you are offended at any man's fault, turn to yourself and study your
own failings. Then you will forget your anger. -- Epictetus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120407/357184fd/attachment.asc>

More information about the ffmpeg-devel mailing list