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

[Michael Niedermayer]

On Fri, Apr 06, 2012 at 02:11:42PM +0200, Reimar Döffinger wrote:
> On Fri, Apr 06, 2012 at 01:23:13PM +0200, Thilo Borgmann wrote:
> > +DATE := $(shell date +%m%d)
> > +
> > +LOGO_FILES := $(wildcard src/logik/$(DATE).*)
> > +
> > +ifeq ($(LOGO_FILES),)
> > +LOGO_FILES := $(wildcard src/logik/$(DATE)-standard)
> > +endif
> 
> Why not just
> LOGO_FILES := $(wildcard src/logik/$(DATE).* src/logik/$(DATE)-standard)
> Though honestly do we have to make it that messy? Why not just
> making a symlink to htdocs/FFmpeg_standard.png instead of needing
> special-casing for -standard?
> Otherwise another option is to use
> > +ifneq ($(wildcard src/logik/$(DATE)-standard),)
> > +LOGO_FILES := htdocs/FFmpeg_standard.png
> > +endif
> 
> 
> > +src/logik/$(DATE)-standard:
> > +	cmp cp htdocs/FFmpeg_standard.png htdocs/ffmpeg-logo.png || \
> > +	cp htdocs/FFmpeg_standard.png htdocs/ffmpeg-logo.png
> > +
> > +src/logik/$(DATE).png:
> > +	cmp $@ htdocs/ffmpeg-logo.png || cp $@ htdocs/ffmpeg-logo.png
> 
> There is a "cp" too much in the first one.

> Also this looks like blatant misuse of the make syntax.

i agree

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The worst form of inequality is to try to make unequal things equal.
-- Aristotle
-------------- 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/d23e9a84/attachment.asc>