[FFmpeg-devel] [PATCH] Compute absolute paths in configure

Stefano Sabatini stefano.sabatini-lala
Fri Jul 11 15:31:29 CEST 2008


On date Friday 2008-07-11 12:02:36 +0100, M?ns Rullg?rd encoded:
> 
> Stefano Sabatini wrote:
> > Hi all,
> >
> > if you set for example a prefix like this:
> > configure --prefix=../..
> 
> Then you either have a good reason to do so, or you can blame yourself.
> 
> > then you'll find the relative path for example in the pkg-config
> > files, which doesn't work properly.
> >
> > This patch make configure compute the absolute path of every path set
> > by configure (yes maybe it needs to be splitted also for what regards
> > the PATHS_LIST part).
> 
> Are yo sure about that?  Why are you messing with cross_prefix?
> 
> > I'm not sure if this is correct though, maybe there are cases where
> > there is the need to keep relative paths (but this doesn't work for
> > sure with pkg-config paths, which need to be absolute), in those cases
> > the user is forced to specify absolute paths.
> 
> We should use whatever the user specifies.  There is nothing more
> annoying than tools trying to be too clever.

Yes, anyway relative paths seem OK everywhere but when you set them in
the various pkg-config file.

The attached patch fixes the problem in a maybe slightly less invasive
way than the previous one.

BTW, this doesn't work if the prefix as specified by the user doesn't
exist, maybe we should add a check for that.

Regards.
-- 
FFmpeg = Free and Funny Minimal Prodigious Extreme Glue
-------------- next part --------------
A non-text attachment was scrubbed...
Name: absolute-pkg-config-paths-00.patch
Type: text/x-diff
Size: 556 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080711/66e046cf/attachment.patch>



More information about the ffmpeg-devel mailing list