Hi Sylvain,

I've applied the patch to the 5_2 branch ; it is legitimate : http://sourcesup.cru.fr/cgi/viewcvs.cgi/sympa/wwsympa/wwsympa.fcgi?only_with_tag=sympa-5_2-branch&r2=1.652.2.13&r1=1.652.2.12

We might release another 5.2 release if we have major fixes applied to this branch.

Sylvain Amrani wrote:
css_path is $Conf{'static_content_path'}.'/css' if not defined

It's been fixed this way in the development branch :

@@ -723,7 +771,8 @@
      $param->{'css_url'} = &Conf::get_robot_conf($robot, 'css_url');
      ## If CSS file not found, let Sympa do the job...
      unless (-f $param->{'css_path'}.'/style.css') {
-        &wwslog('err','Could not find CSS file %s, using default CSS', 
$param->{'css_path'}.'/style.css');
+ &wwslog('err','Could not find CSS file %s, using default CSS', $param->{'css_path'}.'/style.css') + if ($param->{'css_path'}); ## Notice only if path was defined
         $param->{'css_url'} = $param->{'base_url'}.$param->{'path_cgi'}.'/css';
      }

If  css_url dir does not exist, the Upgrade process will create it and create 
.css files in it.

So, please :
Stefan Hornburg : could you please double-check that the default 
static_content_pass/css directory is writable for user sympa ?
Sympa-author : will you release a new 5.2 release ? If so, would you apply this 
patch in the 5_2 branch ?



--
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to