Norbert Preining <[EMAIL PROTECTED]> wrote:

> On Die, 05 Jun 2007, Frank Küster wrote:
>> > ===================================================================
>> > --- tpm2deb-source.pl      (revision 2935)
>> > +++ tpm2deb-source.pl      (working copy)
>> 
>> Fine except:
>> 
>> > @@ -83,6 +83,7 @@
>> >  
>> >  our $Master;
>> >  our %TpmData;
>> > +our %Config;
>> 
>> Why is that needed?
>
> Because of this:
>
> On Die, 05 Jun 2007, Norbert Preining wrote:
>> +    foreach my $foo (@{$TeXLive{'source'}{$package}{'binary_packages'}}) {
>> +            if (ismember($foo,@{$Config{'add_packages'}})) {
>
>                                     ^^^^^^^^^^^^^^^^^^^^^^^
>
> is this bad?

I'm not sure at the moment.  I thought there was a method to access all
variables we need without adding additional "our" declarations.  Maybe
it's because all information that tpm2debcommon.pm knows about *and*
thinks should be exported is copied to %TeXLive?  If that's true, it
would be cleaner to do it that way.

Regards, Frank
-- 
Frank Küster
Single Molecule Spectroscopy, Protein Folding @ Inst. f. Biochemie, Univ. Zürich
Debian Developer (teTeX/TeXLive)

Reply via email to