AF:
NF:0
PS:10
SRH:1
SFN:
DSR:
MID:
CFG:
PT:0
S:andy.sharp@onstor.com
RQ:
SSV:onstor-exch02.onstor.net
NSV:
SSH:
R:<larry.scheer@onstor.com>
MAID:1
X-Sylpheed-Privacy-System:
X-Sylpheed-Sign:0
SCF:#mh/Mailbox/sent
RMID:#imap/andys@onstor.net@onstor-exch02.onstor.net/INBOX	0	BB375AF679D4A34E9CA8DFA650E2B04E02215767@onstor-exch02.onstor.net
X-Sylpheed-End-Special-Headers: 1
Date: Wed, 25 Apr 2007 16:51:10 -0700
From: Andrew Sharp <andy.sharp@onstor.com>
To: "Larry Scheer" <larry.scheer@onstor.com>
Subject: Re: Review request (changes to remove time zone hack put in for
 Lamborghini)
Message-ID: <20070425165110.78a43b67@ripper.onstor.net>
In-Reply-To: <BB375AF679D4A34E9CA8DFA650E2B04E02215767@onstor-exch02.onstor.net>
References: <BB375AF679D4A34E9CA8DFA650E2B04E02215767@onstor-exch02.onstor.net>
Organization: Onstor
X-Mailer: Sylpheed-Claws 2.6.0 (GTK+ 2.8.20; x86_64-pc-linux-gnu)
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit

Looks good.

a

On Wed, 25 Apr 2007 16:48:30 -0700 "Larry Scheer"
<larry.scheer@onstor.com> wrote:

> Change 23719 by larrys@larrys-r14-dmip on 2007/04/25 15:35:06
> *pending*
> 
>         Remove the time zone hack placed in the distribution for the
> broken
>         upgrade command that is found in all releases prior to
> 2.2.0.0. This change assumes the broken upgrade will never install the
>         software for this release. (3.0.0.0 aka Delorean.)
> 
>         Also added ;true after a rm -f to prevent make from saying
>         Error (ignored) when make release is run.
> 
> Affected files ...
> 
> ... //depot/FB-DELOREAN/nfx-tree/Makefile#8 edit
> ... //depot/FB-DELOREAN/nfx-tree/code/ssc-pm/rc.agile#2 edit
> 
> 
> ==== //depot/FB-DELOREAN/nfx-tree/Makefile#8 -
> /home/larrys/perforce/trees/FB-DELOREAN/nfx-tree/Makefile ====
> @@ -528,9 +528,9 @@
>         cd ${PATH_TO_MAIN}
>         echo "cwd is `pwd`"
>         @- [ -h ${PATH_TO_RELEASE}/usr/local/agile/conf/current.bom ]
> && \
> -       $(SUDO) rm -f
> ${PATH_TO_RELEASE}/usr/local/agile/conf/current.bom
> +       $(SUDO) rm -f
> ${PATH_TO_RELEASE}/usr/local/agile/conf/current.bom; true
>         @- [ -f
> ${PATH_TO_RELEASE}/usr/local/agile/conf/$(RELNAME).bom ] && \
> -       $(SUDO) rm -f
> ${PATH_TO_RELEASE}/usr/local/agile/conf/$(RELNAME).bom
> +       $(SUDO) rm -f
> ${PATH_TO_RELEASE}/usr/local/agile/conf/$(RELNAME).bom; true
>         cd ${PATH_TO_RELEASE} && \
>         $(SUDO) find . -type f -o -type l | \
>         (while read fname ; do \
> @@ -547,11 +547,6 @@
>           ${PATH_TO_RELEASE}/usr/local/agile/conf/current.bom 
>         $(SUDO) chown 0:0
> ${PATH_TO_RELEASE}/usr/local/agile/conf/$(RELNAME).bom
>  
> -# Upgrade is mis-handling zoneinfo files here is part of the
> work-around until
> -# upgrade is fixed
> -       cd ${PATH_TO_RELEASE} && \
> -       $(SUDO) tar zcvf var/tmp/zoneinfo.tgz ./usr/share/zoneinfo
> -
>  # Need to correct permission on some files
>         cd ${PATH_TO_RELEASE} && \
>         $(SUDO) chown 0:0 usr/local/agile/lib/*
> 
> 
> ==== //depot/FB-DELOREAN/nfx-tree/code/ssc-pm/rc.agile#2 -
> /home/larrys/perforce/trees/FB-DELOREAN/nfx-tree/code/ssc-pm/rc.agile
> ====
> @@ -25,18 +25,6 @@
>  
>  echo -n ' mgmtbus0'
>  
> -# New Zoneinfo file for DST upgrades installed once at boot time
> -if [ -f /var/tmp/zoneinfo.tgz ] ; then
> -   mount -u -w `df /usr | tail -1 | awk '{print $NF}'`
> -   tar zxpf /var/tmp/zoneinfo.tgz -C /
> -   if [ -f /usr/share/zoneinfo/America/Kentucky/Louisville ] ; then
> -      rm -f /var/tmp/zoneinfo.tgz
> -         echo ""
> -      echo "New Time Zone Files for Daylight Savings Time (DST)
> Installed"
> -   fi
> -   mount -u -o ro `df /usr | tail -1 | awk '{print $NF}'`
> -fi
> -
>  # Debuggage. In the operational system, we won't give them a choice
