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

[Thilo Borgmann]

Am 07.04.12 12:36, schrieb Thilo Borgmann:
> Am 07.04.12 01:36, schrieb Michael Niedermayer:
>> [...]
>>>> 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...)
> 
> Oh of course. Well it is a file because a new clone or a push at a day without
> any $(DATE)* file should use the previously defined text. A new clone will fail
> if there is no file already in the repo. Then I think...
> 
> 
>>> 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 ?
> 
> ... it's unavoidable to have such a directory which contains files not under
> version control. AFAIK CSS cannot handle the job - it can change color, size,
> style and things but not influence a hovertext. One has to choose between
> javascript or plain html for that.

New patches attached to use files in such a directory.

For 0001:
Bug fixed by not including $(LOGO_FILES) in $(TARGETS). Otherwise make clean
does delete the files in src/logik...

For 0002:
Works well and even with non-phony targets. But I don't know how to include an
empty directory into git. Alternatively also create the inter/ directory in the
makefile... or have a dummy/readme file in inter/ for git...

-Thilo
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: 0001-Let-the-Makefile-copy-the-website-logo-according-to-.patch
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120407/56e195e0/attachment.ksh>
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: 0002-Generate-a-hovertext-for-the-logo-based-on-src-logik.patch
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120407/56e195e0/attachment-0001.ksh>