Hi Dave,
Thanks for your input and contribution.
Yes, your changes are correct. Basically I fixed it on my working copy
long times ago when Database.pm is changed but I forgot to check my
codes in. :-(
If you don't mind, can I add your changes to trunk?
Regards,
- DongInn
Dave Love wrote:
> As far as I can tell, the configuration for the postfix setup is
> completely broken -- it bails out with an incomprehensible Perl error.
> This change does what I think was intended, but I don't know postfix,
> so I don't know if it actually produces an appropriate configuration.
>
>
> ------------------------------------------------------------------------
>
> Index: packages/mta-config/scripts/post_install
> ===================================================================
> --- packages/mta-config/scripts/post_install (revision 5780)
> +++ packages/mta-config/scripts/post_install (working copy)
> @@ -91,13 +91,17 @@
> }
> $line_num .= "q";
> system("sed $line_num $postfix_conf > main.cf.server.$$");
> - my $nodes = join ", ", OSCAR::Database::get_nodes(\%options, [EMAIL
> PROTECTED]);
> + my @nodes = (); my $nodelist;
> + OSCAR::Database::get_nodes([EMAIL PROTECTED], \%options, [EMAIL
> PROTECTED])
> + or die "Can't get node list from database";
> + for (my $i=0; $i<@nodes; $i++) {$nodes[$i] = $nodes[$i]{hostname};}
> + $nodelist = join ", ", @nodes;
> open CONF, ">> main.cf.server.$$"
> or die "Can not open the config file: $!";
>
> print CONF "\n";
> print CONF "# Allow the mail to access to only the following list of
> nodes by setting the 'mynetworks'\n";
> - print CONF "mynetworks = $nodes\n";
> + print CONF "mynetworks = $nodelist\n";
> print CONF "\n";
> print CONF "#\n";
> print CONF "# END OF ADDED CONFIGURATION OPTIONS FOR OSCAR\n";
>
> ------------------------------------------------------------------------
>
>
>
> ------------------------------------------------------------------------
>
> -------------------------------------------------------------------------
> Take Surveys. Earn Cash. Influence the Future of IT
> Join SourceForge.net's Techsay panel and you'll get the chance to share your
> opinions on IT & business topics through brief surveys-and earn cash
> http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
> ------------------------------------------------------------------------
>
> _______________________________________________
> Oscar-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/oscar-devel
>
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Oscar-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/oscar-devel
