Hi,

On 7 Feb 2006, Bart Martens told this:

> The attached patch fixes the bug.
>
> Regards,
>
> Bart Martens diff -ruN orig/mailagent-3.73/debian/postinst
> mailagent-3.73/debian/postinst
> --- orig/mailagent-3.73/debian/postinst       2006-02-07 15:13:58.000000000 
> +0100
> +++ mailagent-3.73/debian/postinst    2006-02-07 15:16:26.000000000 +0100
> @@ -20,6 +20,12 @@
> # Ignore all invocations uxcept when called on to configure.
> exit 0 unless $ARGV[0] =~ /configure/;
>
> +eval {require Debconf::Client::ConfModule;};
> +if ( ! $@ )
> +{
> +  import Debconf::Client::ConfModule ':all';
> +}
> +
> sub handle_org (){
> my $ret;


        Umm, we already do this -- if you look around line 105 that
 you mention, you'll see this:

  eval {require Debconf::Client::ConfModule;};
  if ( $@ ) {
    # No debconf
    print STDERR <<"EOH";
 ...
EOH
  }
  else {
    import Debconf::Client::ConfModule ':all';
    ...
  }

        So the import is only done if the require does indeed go
 through. Did your tests succeed with that patch? If so, there is
 something I truly do not understand.

        manoj
-- 
Humpty Dumpty was pushed.
Manoj Srivastava   <[EMAIL PROTECTED]>    <[EMAIL PROTECTED]> 
1024D/BF24424C print 4966 F272 D093 B493 410B  924B 21BA DABB BF24 424C


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

Reply via email to