X-MimeOLE: Produced By Microsoft Exchange V6.5
Received: by onstor-exch02.onstor.net 
	id <01C86440.26880493@onstor-exch02.onstor.net>; Thu, 31 Jan 2008 12:33:26 -0700
MIME-Version: 1.0
Content-Type: text/plain;
	charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
Content-class: urn:content-classes:message
Subject: RE: PERFORCE change 27546 for review
Date: Thu, 31 Jan 2008 12:33:26 -0700
Message-ID: <BB375AF679D4A34E9CA8DFA650E2B04E080589AB@onstor-exch02.onstor.net>
In-Reply-To: <WEBMAILKDG6UBWXV9R500004c8c@mail.onstor.com>
X-MS-Has-Attach: 
X-MS-TNEF-Correlator: 
Thread-Topic: PERFORCE change 27546 for review
Thread-Index: AchkNazm19V/EILCRUSXuMYJ1ltzLQACnPRw
From: "Tim Gardner" <tim.gardner@onstor.com>
To: "Andy Sharp" <andy.sharp@onstor.com>

What's a PL?

> -----Original Message-----
> From: Andy Sharp
> Sent: Thursday, January 31, 2008 10:18 AM
> To: Amit Bothra; Andy Sharp; Brian DeForest; Charissa Willard; Chris
> Vandever; Danqing Jin; Deepak Veliath; Eric Barrett; Henry Lau; Ian
Brown;
> Jobi Ariyamannil; Jonathan Goldick; Ken Renshaw; Larry Scheer; Maxim
> Kozlovsky; Mike Lee; Sandrine Boulanger; Svati Chandra; Tim Gardner;
> Yuvarani Cothandaraman
> Subject: PERFORCE change 27546 for review
>=20
> Change 27546 by andys@ripper on 2008/01/31 05:15:59
>=20
> 	Move GNU licensed code out of genlib.  Because PL
> 	Max is afraid of the big bad GPL.  Also it's a bit
> 	better split of the code, although it is a bit strange
> 	to have some mount functions in lib zebra.
>=20
> 	Oh yeah, fix several bugs too, including: get_device_names()
> 	not switching the device names like it is supposed to;
> 	mount_secondary() switched to using modify_readonly_file()
instead
> 	of it's own code, but the logic afterwards wasn't right;
mkpath()
> 	was modifying the path argument but it shouldn't; too many
others
> 	to keep listing them.
>=20
> 	reviewed by larrys
>=20
> Affected files ...
>=20
> ... //depot/dev/nfx-tree/code/sm-sct/Makefile#3 edit
> ... //depot/dev/nfx-tree/code/ssc-genlib/Makefile#6 edit
> ... //depot/dev/nfx-tree/code/ssc-genlib/config.linux.h#2 delete
> ... //depot/dev/nfx-tree/code/ssc-genlib/config.openbsd.h#2 delete
> ... //depot/dev/nfx-tree/code/ssc-genlib/fs-api.h#4 edit
> ... //depot/dev/nfx-tree/code/ssc-genlib/fs.c#5 edit
> ... //depot/dev/nfx-tree/code/ssc-genlib/log.c#2 delete
> ... //depot/dev/nfx-tree/code/ssc-genlib/log.h#2 delete
> ... //depot/dev/nfx-tree/code/ssc-genlib/memory.c#2 delete
> ... //depot/dev/nfx-tree/code/ssc-genlib/memory.h#2 delete
> ... //depot/dev/nfx-tree/code/ssc-genlib/redo-hosts.c#3 delete
> ... //depot/dev/nfx-tree/code/ssc-genlib/vector.c#2 delete
> ... //depot/dev/nfx-tree/code/ssc-genlib/vector.h#2 delete
> ... //depot/dev/nfx-tree/code/ssc-genlib/zebra.h#2 delete
> ... //depot/dev/nfx-tree/code/ssc-initial-config/initial-config.c#4
edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/Makefile#11 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/buffer.c#4 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_admin.c#7 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_arp.c#4 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_audit.c#9 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_autosupport.c#11 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_cifs.c#5 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_cifssrvr.c#5 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_cluster.c#22 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_cshare.c#8 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_disk.c#5 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_domain.c#10 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_elog.c#10 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_filesystem.c#10 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_flash-linux.c#13 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_flash.c#6 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_gns.c#5 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_help.c#18 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_idmap.c#5 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_interface.c#14 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_localmap.c#7 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_localuseraccounts.c#6 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_lvol.c#18 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_nameservice.c#4 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_nfs.c#14 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_port.c#9 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_privileges.c#6 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_promupgrade.c#11 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_quota.c#13 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_raid.c#4 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_rcon.c#6 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_remoteaccess.c#5 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_route.c#5 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_sanm.c#12 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_scsi.c#7 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_snapshot.c#13 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_snmp.c#7 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_spool.c#18 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_stats.c#13 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_symlinkmap.c#6 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_sysHealth.c#5 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_sysdvt.c#6 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_system-linux.c#8 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_system-openbsd.c#8 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_system.c#40 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_tape.c#5 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_upgrade.c#21 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_vif.c#8 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_vol.c#40 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_vscan.c#6 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_vsvr.c#18 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/command.c#7 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/command.h#3 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/linklist.c#4 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/menu.c#10 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/vty.c#9 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/vtysh-linux.c#3 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/vtysh-openbsd.c#12 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/vtysh.c#10 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/vtysh_config.c#9 edit
> ... //depot/dev/nfx-tree/code/ssc-nfxsh/vtysh_main.c#12 edit
> ... //depot/dev/nfx-tree/code/ssc-zebra/Makefile#1 add
> ... //depot/dev/nfx-tree/code/ssc-zebra/config.linux.h#1 branch
> ... //depot/dev/nfx-tree/code/ssc-zebra/config.openbsd.h#1 branch
> ... //depot/dev/nfx-tree/code/ssc-zebra/fs-vector.h#1 add
> ... //depot/dev/nfx-tree/code/ssc-zebra/log.c#1 branch
> ... //depot/dev/nfx-tree/code/ssc-zebra/log.h#1 branch
> ... //depot/dev/nfx-tree/code/ssc-zebra/memory.c#1 branch
> ... //depot/dev/nfx-tree/code/ssc-zebra/memory.h#1 branch
> ... //depot/dev/nfx-tree/code/ssc-zebra/vector.c#1 branch
> ... //depot/dev/nfx-tree/code/ssc-zebra/vector.h#1 branch
> ... //depot/dev/nfx-tree/code/ssc-zebra/zebra.h#1 branch
>=20
>=20
> http://liszt:1818/@md=3Dd&cd=3D//depot/$c=3DG35@/27546?ac=3D10
