[FFmpeg-devel] Sed broken on solaris 10 sparc

[Robinson, Greg]

Hi all,

/bin/sed is broken on solaris 10 sparc, and so the configure script
produces empty encoders/decoders as such:

Enabled decoders:
pr:  -- empty file

Enabled encoders:
pr:  -- empty file

Enabled parsers:
pr:  -- empty file

Enabled demuxers:
pr:  -- empty file

...

/bin/sed does work for the other instances of sed throughout the
configure script though.

I have made a patch to use /usr/xpg4/bin/sed for the find_things()
function.
I am hoping it could be applied to the latest snapshot.  I have tested
this against
ffmpeg-export-2009-01-20 and it works well.

I do not have /usr/xpg4/bin in my path at all, and prefer this quick
work around for this instance.

I hope my mailer does not wrap lines too badly...

Thankx,

Greg.

--- configure.orig      Mon Jan 19 05:01:54 2009
+++ configure   Thu Jan 22 12:57:27 2009
@@ -1184,10 +1184,19 @@
 FFMPEG_CONFIGURATION="$@"
 
 find_things(){
+# determine a good sed
+    case `uname -s` in
+    'SunOS')
+       good_sed=/usr/xpg4/bin/sed
+       ;;
+    *)
+       good_sed=sed
+       ;;
+    esac
     thing=$1
     pattern=$2
     file=$source_path/$3
-    sed -n "s/^[^#]*$pattern.*([^,]*,
*\([^,]*\)\(,.*\)*).*/\1_$thing/p" "$file"
+    $good_sed -n "s/^[^#]*$pattern.*([^,]*,
*\([^,]*\)\(,.*\)*).*/\1_$thing/p" "$file"
 }
 
 ENCODER_LIST=$(find_things  encoder  ENC      libavcodec/allcodecs.c)

IMPORTANT: This email remains the property of the Australian Defence Organisation and is subject to the jurisdiction of section 70 of the CRIMES ACT 1914.  If you have received this email in error, you are requested to contact the sender and delete the email.