[Slackbuilds-users] ffmpeg

[Robby Workman]

On Fri, 26 Dec 2008 10:41:09 +0100
Heinz Wiesinger <pprkut at liwjatan.at> wrote:

> Robby Workman wrote:
> > Please don't hijack unrelated threads :)
> >
> > On Thu, 25 Dec 2008 22:40:59 -0800 (PST)
> >
> > Darrell Anderson <humanreadable at yahoo.com> wrote:
> > > Hi Folks,
> > >
> > > I built ffmpeg. Initially the build script failed because I
> > > have /tmp mounted noexec. I'm not changing that. :-)
> >
> > Understandable.
> >
> > > The error message informed me that configure was looking for
> > > TMPDIR. I then ran the build script as follows:
> > >
> > > TMPDIR=/temporary/tmp/location ./ffmpeg.Slackbuild.
> > >
> > > That worked. I next modified the build script with:
> > >
> > > export TMPDIR=/temporary/tmp/location
> > >
> > > That worked too.
> > >
> > > Unlike all of the files I have built from SBo, the ffmpeg
> > > configure file is hard-coded to look for a TMPDIR environment
> > > variable, and if none is found defaults TMPDIR to /tmp.
> >
> > Wow.  I'd expect that they would use TMP, and furthermore, I think
> > that should probably be mentioned to them.
> 
> That's the relevant part I found in ffmpeg svn's configure script:
> 
> # set temporary file name
> if test ! -z "$TMPDIR" ; then
>     TMPDIR1="${TMPDIR}"
> elif test ! -z "$TEMPDIR" ; then
>     TMPDIR1="${TEMPDIR}"
> else
>     TMPDIR1="/tmp"
> fi
> 
> As one can see it looks for $TMPDIR and $TEMPDIR, so basically I
> think adding a $TMP check here should be fine as I haven't found $TMP
> anywhere else in configure.


You mean patching their configure?  If so, then sure, that's fine too,
but be sure and forward it upstream.


> > > Perhaps a caution on the ffmpeg page might help? I realize most
> > > people mount /tmp executable, but a short caution would help those
> > > who don't.
> >
> > Probably better to add something like this to the build script
> > before starting the compile process:
> >   TMPDIR=${TMPDIR:-$TMP}
> >   export TMPDIR
> > and then "unset TMPDIR" after the compile process is finished.
> 
> I guess with the above mentioned change we don't need this anymore,
> right?


If my understanding of your intention is correct, then yeah, that's
right.

-RW
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://lists.slackbuilds.org/pipermail/slackbuilds-users/attachments/20081227/7c665299/attachment.asc>