found 408116 0.80.1
thanks

Hi,

I'm very sorry, but due to other activities I didn't discover earlier that 
sqspell.php has a syntax error. It needs to separate the values of the array 
with a comma: ",".

I've created attached patch that fixes the issue. It also removes the 
closing "?>", which is not necessary any may cause problems if somehow 
inadvertently whitespace was added after it.


thanks!

Thijs
--- /tmp/DuaGWIs59w/dictionaries-common-0.80.1/scripts/Debian/DictionariesCommon.pm.in	2007-04-10 16:54:56.000000000 +0200
+++ /tmp/ODsPZYCuv9/dictionaries-common-0.80.2/scripts/Debian/DictionariesCommon.pm.in	2007-05-10 12:43:16.000000000 +0200
@@ -394,15 +394,17 @@
   my $class        = "ispell";
   my $dictionaries = loaddb ($class);
   my $php          = generate_comment ("### ");
+  my @dictlist;
   
   $php .= "<?php\n\$SQSPELL_APP = array (\n";
   foreach ( keys %$dictionaries ){
     next if m/.*[^a-z]tex[^a-z]/i;            # Discard tex variants
     my $spellchecker_params =
 	&dc_get_spellchecker_params($class,$dictionaries->{$_});
-    $php .= qq {  '$_' => 'ispell -a $spellchecker_params'\n};
+    push(@dictlist, qq {  '$_' => 'ispell -a $spellchecker_params'\n});
   }
-  $php .= ");\n?>\n";
+  $php .= join(",", @dictlist);
+  $php .= ");\n";
   
   open (PHP, "> $cachedir/$squirrelmailsupport")
       or die "Cannot open SquirrelMail cache file";

Attachment: pgpq97jBZ43MZ.pgp
Description: PGP signature

Reply via email to