Hi Mark, Here's the new webrev. http://cr.opensolaris.org/~spoorthy/ws_cryptcbc3/ This is for perl 5.8.4. and BUGTRAQ field is not filled in [because they are waiting for ARC approval] Note that this crypt-cbc has the dependency on a package called crypt-des. I have sent code review request.
Thanks Spoorthy Mark Phalan wrote: > On Fri, 2009-09-04 at 10:02 +0530, Spoorthy H.S wrote: > >> Hi Mark, >> >> Here's the confusion...I need to use perl 5.10.0. In that case, I need >> to keep my package under /usr/perl5/vendor_perl/5.10.0/Crypt. But, I >> dont see this in Targetdirs or the directory for 5.10.0 under >> /usr/perl5/vendor_perl. I can include them in Targetdirs for now. But, >> to include crypt-cbc man page under /usr/perl5/5.10.0/man/man3, I dont >> see them in defaults to use. So I used perl 5.8.4. Or give me an >> alternative if any is available.. >> You can check the below result for the folder listing... >> >> :"/builds/ss224839/my_ws/proto/root_sparc/usr/perl5" 9 $ ls -l >> total 9 >> drwxr-xr-x 5 ss224839 staff 5 Aug 31 21:59 5.8.4 >> drwxr-xr-x 3 ss224839 staff 3 Aug 6 04:18 site_perl >> drwxr-xr-x 3 ss224839 staff 3 Aug 6 00:33 vendor_perl >> :"/builds/ss224839/my_ws/proto/root_sparc/usr/perl5" 10 $ cd vendor_perl/ >> :"/builds/ss224839/my_ws/proto/root_sparc/usr/perl5/vendor_perl" 11 $ ls -l >> total 3 >> drwxr-xr-x 7 ss224839 staff 7 Sep 3 04:26 5.8.4 >> >> Hence, in this case, I need to deliver the package for perl 5.8.4 as an >> interim and for perl 5.10 later.... This decision needs to be done. >> > > I think you should deliver for 5.10 (and also maybe 5.8) but I don't > care too much which it is. Your webrev should be consistent. Currently > it isn't. > > -M > >
