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

[Diego Biurrun]

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.

> --- 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.

Diego