OK... I suspected right...

Looks like the patch I had provided for fix of 411983 was only partially
applied.

in sympa.config, the following snippet is wrong :
        db_get sympa/use_soap
        use_soap="$RET"

        if [ "$use_soap" = "true" ]; then
            soap_url="http://${host}/sympasoap";
                if [ -f /etc/sympa/sympa.conf ]; then
                # Get the soap url from sympa.conf                              
                                                                                
                                     
                        orig_soap_url=`perl -nle 'if (/^\s*soap_url\s+(\d)$/) 
{print $1;}' /etc/sympa/sympa.conf`
                        if [ -z "$orig_soap_url" ]; then
                                db_get sympa/soap_url
                                orig_soap_url="$RET"
                        else
                                soap_url="$orig_soap_url";
                        fi

                        db_set sympa/soap_url "$soap_url"

                        db_input medium sympa/soap_url || [ $? -eq 30 ]
                        db_go
                fi
        fi

Actually, at first install's config time, there's no /etc/sympa/sympa.conf
so the 
          db_set sympa/soap_url "$soap_url"

          db_input medium sympa/soap_url || [ $? -eq 30 ]
          db_go
is not done at all :( ... and later in the postinst, it won't do.

So the "fi" should be moved up a little bit :
        db_get sympa/use_soap
        use_soap="$RET"

        if [ "$use_soap" = "true" ]; then
            soap_url="http://${host}/sympasoap";
                if [ -f /etc/sympa/sympa.conf ]; then
                # Get the soap url from sympa.conf                              
                                                                                
                                     
                        orig_soap_url=`perl -nle 'if (/^\s*soap_url\s+(\d)$/) 
{print $1;}' /etc/sympa/sympa.conf`
                        if [ -z "$orig_soap_url" ]; then
                                db_get sympa/soap_url
                                orig_soap_url="$RET"
                        else
                                soap_url="$orig_soap_url";
                        fi
                fi

                db_set sympa/soap_url "$soap_url"

                db_input medium sympa/soap_url || [ $? -eq 30 ]
                db_go
        fi

Best regards.

P.S.: next time, try and applying the patches provided instead of manual cherry 
picking ;)

Le jeudi 20 septembre 2007 à 15:51 +0200, Olivier Berger a écrit :
> Le jeudi 20 septembre 2007 à 15:19 +0200, Olivier Berger a écrit :
> 
> > Error at line 166 : /etc/sympa/sympa.confLanguage::SetLang() 
> > Language::SetLang(), missing locale parameter
> > mail::smtpto() Missing Return-Path in mail::smtpto()
> > Configuration file /etc/sympa/sympa.conf has errors.
> 
> Looks like soap_url was not configured the right way :
> ## SOAP service URL
> soap_url
> 
> So... I suspect that bug 411983 was not fixed the right way, maybe.
> 
> I'll try and investigate further.
> 
> Best regards,
-- 
Olivier BERGER <[EMAIL PROTECTED]> (ATTENTION : new address)
Ingénieur Recherche - Dept INF
GET/INT at Evry (http://www.int-edu.eu/)
OpenPGP-Id: 1024D/6B829EEC




Reply via email to