the new controlchan (+gpgverify)

[Russ Allbery]

greg andruk <supersede at india.com> writes:
> In nnfolder+Mail:inn-workers, Marco d'Itri <md at Linux.IT> wrote:

>> I'd like to use -T, but since innshellvars.pl uses eval I can't.  Does
>> anybody know a workaround?

The *best* solution is really to make an XS module that calls the same
parsing code that the rest of INN does and writes the results directly
into Perl's stashes.  But that requires getting infrastructure set up to
build Perl modules as part of installing INN, and I'm still a bit worried
about that working; hope it won't generate more difficulties compiling and
installing INN.  (I'd really like to get that to work, though, since it
would make it much, much easier to write parts of INN in Perl, which is
already fairly popular to do.)

> I used to do something like this to innshellvars.pl (kind of a bogus
> check, but we need to trust innconfval anyway)...

> 12c12,20
> < eval `/news/exp/bin/innconfval -p`;
> ---
> > undef $/;
> > open F, "/news/exp/bin/innconfval -p |" 
> > 	|| die "innshellvars.pl cant spawn innconfval";
> > $tempconf = <F>;
> > $tempconf =~ m/(.*)/s;
> > eval $1;
> > undef $tempconf;
> > close F;
> > $/ = "\n";

> There are surely prettier ways to do that!

Not a lot, but yeah, that would work.

> ...and the layout of controlchan changes like so to shut up the path
> dependency complaint (diff against the released one because it was
> there):

> *** controlchan	Fri Feb  2 22:34:35 2001
> --- cch2	Wed Feb  7 20:35:04 2001
> ***************
> *** 1,3 ****
>   #! /usr/local/bin/perl
> ! require '/news/exp/lib/innshellvars.pl';
  
> --- 1,3 ----
>   #! /usr/local/bin/perl
> ! my $innshellvars = '/news/exp/lib/innshellt.pl';
  
> ***************
> *** 35,36 ****
> --- 35,37 ----
>   delete @ENV{'IFS', 'CDPATH', 'ENV', 'BASH_ENV'};
> + require $innshellvars;

Oh, ick, that breaks fixscript.  What does taint complain about that this
silences?

-- 
Russ Allbery (rra at stanford.edu)             <http://www.eyrie.org/~eagle/>