[FFmpeg-devel] [PATCH] Install preset files from ffpresets/ to ${prefix}/share/ffmpeg

[Robert Swain]

2009/1/24 Diego Biurrun <diego at biurrun.de>:
> On Sat, Jan 24, 2009 at 04:54:32PM +0000, Robert Swain wrote:
>> 2009/1/24 Diego Biurrun <diego at biurrun.de>:
>> > On Sat, Jan 24, 2009 at 04:04:36PM +0000, Robert Swain wrote:
>> >> 2009/1/24 Robert Swain <robert.swain at gmail.com>:
>> >> > Please find attached a patch to install the preset files, that reside
>> >> > in the ffpresets subdirectory of trunk, to ${prefix}/share/ffmpeg.
>> >>
>> >> Oops, I meant to attach this patch...
>> >>
>> >> --- Makefile  (revision 16713)
>> >> +++ Makefile  (working copy)
>> >> @@ -133,6 +136,10 @@
>> >>
>> >> +install-data: $(DATA_FILES)
>> >> +     install -d "$(DATADIR)"
>> >> +     install -m 644 $(DATA_FILES) "$(DATADIR)"
>> >
>> > Place this above install-man.
>>
>> Why? They aren't in alphabetical order or anything. Done anyway.
>
> It's IMO more logical to have all the program installation targets
> grouped together separate from the documentation targets.

OK.

>> --- Makefile  (revision 16752)
>> +++ Makefile  (working copy)
>> @@ -145,6 +151,9 @@
>>  uninstall-man:
>>       rm -f $(addprefix "$(MANDIR)/man1/",$(ALLMANPAGES))
>>
>> +uninstall-data:
>> +     rm -rf "$(DATADIR)"
>
> Above uninstall-man please, similar to the install target.

Done.

See attached.

Regards,
Rob
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 20090124-1721-install_presets.diff
Type: text/x-diff
Size: 2679 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090124/96c791b7/attachment.diff>