X-MimeOLE: Produced By Microsoft Exchange V6.5
Received: by onstor-exch02.onstor.net 
	id <01C874DC.DEC481EC@onstor-exch02.onstor.net>; Thu, 21 Feb 2008 15:55:35 -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: please review 27961
Date: Thu, 21 Feb 2008 15:55:35 -0700
Message-ID: <BB375AF679D4A34E9CA8DFA650E2B04E056C93E4@onstor-exch02.onstor.net>
In-Reply-To: <20080220230631.35a90fd0@ripper.onstor.net>
X-MS-Has-Attach: 
X-MS-TNEF-Correlator: 
Thread-Topic: please review 27961
Thread-Index: Ach0WEmSYQIXRgS8RNep70hGm3wYBgAhEwdg
From: "Larry Scheer" <larry.scheer@onstor.com>
To: "Andy Sharp" <andy.sharp@onstor.com>

linux/Pkgs/binary/Deb/exim4-config_4.63-17_all.deb

     looks good

linux/Pkgs/source/exim4/Makefile

     Lines 40 and 41. Maybe I am missing something here; but it
     appears you are copying a file that is already inr BIN_PKG_DIR
     to BIN_PKG_DIR?

linux/Pkgs/source/exim4/debian.patch

     looks good

linux/rootfs/etc/exim4/exim4.conf.template

     looks good


-----Original Message-----
From: Andy Sharp=20
Sent: Wednesday, February 20, 2008 11:07 PM
To: Larry Scheer
Subject: please review 27961

Change 27961 by andys@ripper on 2008/02/20 10:33:21 *pending*

	Fix several small problems with exim4 package:

	Fix typo in exim4 configuration template file that was
	causing configuration error.

	Fix a few bugs in update-exim4defaults that was making
	it work not so well.

	Remove change to exim logrotate config for new log file.
	Log file was put in onstor directory instead and onstor
	logrotate config file started in cl#27953

	Change makefile to do the right thing if packages are up to
	date but not in perforce directory.

	Add exim4-config to list of changed packages in makefile.

	reviewed by

Affected files ...

... //depot/dev/linux/Pkgs/binary/Deb/exim4-config_4.63-17_all.deb#1
edit ... //depot/dev/linux/Pkgs/source/exim4/Makefile#4 edit
... //depot/dev/linux/Pkgs/source/exim4/debian.patch#5 edit
... //depot/dev/linux/rootfs/etc/exim4/exim4.conf.template#3 edit

