libexec/zmldapinit bad sed syntax?

[langthang]

I tried to file a bug report but it keeps telling me to select a Component even I did so I report it here instead.
I try to install zcs-4.0.0_GA_303.DEBIAN3.1.tar.gz and it failed, trace it to

mail:/opt/zimbra# su - zimbra -c "libexec/zmldapinit"
sed: -e expression #1, char 61: Extra characters after command
ERROR - failed to start slapd

patch libexec/zmldapinit with

--- libexec/zmldapinit.orig     2006-09-07 15:48:49.000000000 -0700
+++ libexec/zmldapinit  2006-09-07 15:51:05.000000000 -0700
@@ -66,7 +66,8 @@
                sed -e '\|/opt/zimbra/openldap/etc/openldap/schema/zimbra.schema| a\
                include "/opt/zimbra/lib/conf/zimbra-ext.schema"' ${config_dir}/slapd.conf.in > /tmp/slapd.conf
        else
-               sed -e '\|/opt/zimbra/openldap/etc/openldap/schema/zimbra.schema| a include "/opt/zimbra/lib/conf/zimbra-ext.schema"' ${config_dir}/slapd.conf.in > /tmp/slapd.conf
+               sed -e '\|/opt/zimbra/openldap/etc/openldap/schema/zimbra.schema| a \
+include "/opt/zimbra/lib/conf/zimbra-ext.schema"' ${config_dir}/slapd.conf.in > /tmp/slapd.conf
        fi
        mv -f /tmp/slapd.conf ${config_dir}/slapd.conf.in
 fi

will let the install went through.

[marcmac]

plat=`/opt/zimbra/libexec/get_plat_tag.sh`
if [ "x$plat" = "xMACOSX" -o "x$plat" = "xMACOSXx86" ]; then

EDITED after drinking coffee:

What veersion of debian are you on? I've only seen that sed syntax required on the mac, which is why the previous sed line contains the syntax that you want.

What does sed --version return for you?

[langthang]

Originally Posted by marcmac

EDITED after drinking coffee

I need some too

Originally Posted by marcmac

What veersion of debian are you on?

$ cat /etc/debian_version
3.1

Originally Posted by marcmac

I've only seen that sed syntax required on the mac, which is why the previous sed line contains the syntax that you want.

What does sed --version return for you?

$ sed --version
GNU sed version 4.1.2

but I can't reproduce it anymore may be I did try to install on a older debian verison. Anyway a newline after a\ is safe for all sed I think.

[langthang]

it was sed installed with debian 3.0 causes problem.

$ echo -e "foo\n123" > /tmp/foobar && sed '/^foo/a \bar' /tmp/foobar
sed: -e expression #1, char 10: Extra characters after command
$ echo -e "foo\n123" > /tmp/foobar && sed '/^foo/a \
> bar' /tmp/foobar
foo
bar
123

most sed man pages I found have
a \
text Insert text, blahblah ...
I guess you can replace that block with just one sed line instead