Mailing-List:	contact cygwin-apps-help AT cygwin DOT com; run by ezmlm
Sender:	cygwin-apps-owner AT cygwin DOT com
List-Subscribe:	<mailto:cygwin-apps-subscribe AT cygwin DOT com>
List-Archive:	<http://sources.redhat.com/ml/cygwin-apps/>
List-Post:	<mailto:cygwin-apps AT cygwin DOT com>
List-Help:	<mailto:cygwin-apps-help AT cygwin DOT com>, <http://sources.redhat.com/lists.html#faqs>
Delivered-To:	mailing list cygwin-apps AT cygwin DOT com
Date:	Fri, 15 Mar 2002 15:10:49 +0100
From:	Pavel Tsekov <ptsekov AT syntrex DOT com>
X-Mailer:	The Bat! (v1.53d) UNREG / CD5BF9353B3B7091
Reply-To:	Pavel Tsekov <ptsekov AT syntrex DOT com>
Organization:	Syntrex, Inc.
X-Priority:	3 (Normal)
Message-ID:	<14121753109.20020315151049@syntrex.com>
To:	"Robert Collins" <robert DOT collins AT itdomain DOT com DOT au>
CC:	"CygWin-Apps" <cygwin-apps AT cygwin DOT com>
Subject:	Re: Setup ready #2
In-Reply-To:	<FC169E059D1A0442A04C40F86D9BA760014BC7@itdomain003.itdomain.net.au>
References:	<FC169E059D1A0442A04C40F86D9BA760014BC7 AT itdomain003 DOT itdomain DOT net DOT au>
MIME-Version:	1.0

Hello Robert,

Friday, March 15, 2002, 2:01:22 PM, you wrote:

Well, let me report another issue... The new setup.exe (codename
setup-200202) seems to break /etc/setup/package_name.lst.gz files
by adding an extra null terminator character after the newline
character.

For more info see attached files. The one with the extension .NEW is
produced from setup-20020315.exe just some minutes ago.

I wild guess is that this is caused by the new String class.

This "bug" breaks the uninstall functionality. Packed is removed from
installed.db but package contents are not removed at all.

void
packagemeta::uninstall ()
{
  if (installed)
    {
      /* this will need to be pushed down to the version, or even the source level
       * to allow differences between formats to be seamlessly managed
       * but for now: here is ok
       */
      hash dirs;
      String line = installed->getfirstfile ();

      try_run_script ("/etc/preremove/", name);
      while (line.size())
        {
        
I think we got here for the first time only and then when the zero char is
read we are f****d :(

I'm sorry for being such a pain in the ass :(

RC> Ok, Setup200202 now

- Raw text -