[FFmpeg-devel] [PATCH] examples: add README file with simple compilation instructions
ubitux at gmail.com
Tue Oct 16 00:04:44 CEST 2012
On Mon, Oct 15, 2012 at 11:02:08PM +0200, Stefano Sabatini wrote:
> On date Friday 2012-09-14 02:19:44 +0200, Clément Bœsch encoded:
> > On Thu, Sep 13, 2012 at 09:50:07PM +0200, Stefano Sabatini wrote:
> > PS: thanks for writing that README, and sorry to be nitpicking so much
> > (but since it's for users I believe details are important to avoid any
> > confusion).
> Rewritten, based on a variant proposed by Clement, which looks simpler
> and possible more clear.
> FFmpeg = Fundamentalist and Funny Magic Problematic Elegant Ghost
> From 887994636e3329eb7b46c688283765a722114219 Mon Sep 17 00:00:00 2001
> From: Stefano Sabatini <stefasab at gmail.com>
> Date: Thu, 13 Sep 2012 21:12:56 +0200
> Subject: [PATCH] examples: add README file with simple compilation
> Makefile | 2 +-
> doc/examples/README | 18 ++++++++++++++++++
> 2 files changed, 19 insertions(+), 1 deletions(-)
> create mode 100644 doc/examples/README
> diff --git a/Makefile b/Makefile
> index 303f80f..0abb300 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -41,7 +41,7 @@ FFLIBS-$(CONFIG_SWSCALE) += swscale
> FFLIBS := avutil
> DATA_FILES := $(wildcard $(SRC_PATH)/presets/*.ffpreset) $(SRC_PATH)/doc/ffprobe.xsd
> -EXAMPLES_FILES := $(wildcard $(SRC_PATH)/doc/examples/*.c) $(SRC_PATH)/doc/examples/Makefile
> +EXAMPLES_FILES := $(wildcard $(SRC_PATH)/doc/examples/*.c) $(SRC_PATH)/doc/examples/Makefile $(SRC_PATH)/doc/examples/README
> SKIPHEADERS = cmdutils_common_opts.h
> diff --git a/doc/examples/README b/doc/examples/README
> new file mode 100644
> index 0000000..84362d4
> --- /dev/null
> +++ b/doc/examples/README
> @@ -0,0 +1,18 @@
> +FFmpeg examples README
> +Both following use case rely on pkg-config and make, thus make sure
> +that you have them installed and working on your system.
> +1) Build the installed examples in a generic read/write user directory
> +Copy to a read/write user directory and just use "make", it will link
> +to the libraries on your system, assuming the PKG_CONFIG_PATH is
> +correctly configured.
> +2) Build the examples in-tree
> +Assuming you're in the doc/example directory of a source FFmpeg
- you are
> +checkout, you need to build FFmpeg, no need to make install in any
"assuming you're in the doc/example directory [...] you need to build
FFmpeg" doesn't sound right. Possible alternative:
"Assuming you are in the source FFmpeg checkout directory, you need to build
FFmpeg (no need to make install in any prefix). Then you can go into the
doc/examples and run commands such as PKG_CONFIG_PATH=pc-uninstalled make"
Or something like this.
> +prefix, and set PKG_CONFIG_PATH=pc-uninstalled for make.
Anyway, LGTM, reword at your own discretion and please apply.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 490 bytes
Desc: not available
More information about the ffmpeg-devel