>> I haven't seen reply to my question:
>>
>>> First of all, it surprises me a bit that usr/src/cmd/perl/contrib is
>>> not ported. I can not see note about it in the arc case. So perl
>>> 5.10  won't support Sun::Solaris modules? So making
>>> /usr/perl5/5.10.0/bin/perl  the default /usr/bin/perl is going to
>>> break customers scripts?
>
> I talked to Alan about this and you're probably correct that it should  
> be delivered together, so I'll get that done right away.

Ah, I'm sorry to be messenger of band news then. Pity I haven't seen
the ARC case to speak up earlier. I hope that we won't find any
dependency between those modules and ONNV gate ...



>>> usr/src/cmd/perl510/atinc.patch
>>
>> I might be missing something, but could you please explain the reason
>> behind changing this? (it was not present in previous version) It  
>> seems
>> to set some paths for tainted mode, and the paths are
>> /usr/local/lib/perl5:/usr/local/lib/perl, which might not be what we
>> want?
>
> it places site_perl and vendor_perl at the beginning of @INC, the patch 
> was  suggested by Mark Trettin and it makes sense to me

I see. To me it would make sense, if customer could override perl
modules by placing them into site_perl. Hopefully it won't break
anything. I just wonder that the patch says '/usr/local/lib/...'

  35 +#ifndef PRIVLIB_EXP
  36 +#  define PRIVLIB_EXP "/usr/local/lib/perl5:/usr/local/lib/perl"
  37 +#endif

We don't have anything in /usr/local/lib, but maybe PRIVLIB_EXP is
defined earlier, so it wont' pick those paths.

However, please, once you'll be testing new perl, try that the @INC is
sane for normal and tainted mode. Something like:

$ perl -T -e 'print join "\n", @INC'
/usr/perl5/5.8.4/lib/i86pc-solaris-64int
/usr/perl5/5.8.4/lib
/usr/perl5/site_perl/5.8.4/i86pc-solaris-64int
/usr/perl5/site_perl/5.8.4
/usr/perl5/site_perl
/usr/perl5/vendor_perl/5.8.4/i86pc-solaris-64int
/usr/perl5/vendor_perl/5.8.4

$ env - perl -e 'print join "\n", @INC'
/usr/perl5/5.8.4/lib/i86pc-solaris-64int
/usr/perl5/5.8.4/lib
/usr/perl5/site_perl/5.8.4/i86pc-solaris-64int
/usr/perl5/site_perl/5.8.4
/usr/perl5/site_perl
/usr/perl5/vendor_perl/5.8.4/i86pc-solaris-64int
/usr/perl5/vendor_perl/5.8.4
/usr/perl5/vendor_perl



I'm quite sure you already hate me, but have you seen my comments about
config.sh? I believe that few configuration options are wrong there...


Good luck!
-- 
        Vlad

Reply via email to