From owner-cvs-ports  Thu Apr  6 00:42:15 1995
Return-Path: cvs-ports-owner
Received: (from majordom@localhost)
          by freefall.cdrom.com (8.6.10/8.6.6) id AAA14201
          for cvs-ports-outgoing; Thu, 6 Apr 1995 00:42:15 -0700
Received: from time.cdrom.com (time.cdrom.com [192.216.223.46])
          by freefall.cdrom.com (8.6.10/8.6.6) with ESMTP id AAA14191
          ; Thu, 6 Apr 1995 00:42:12 -0700
Received: from vinkku.hut.fi (vinkku.hut.fi [130.233.245.1]) by time.cdrom.com (8.6.11/8.6.9) with ESMTP id AAA11064; Thu, 6 Apr 1995 00:42:00 -0700
Received: (from vode@localhost) by vinkku.hut.fi (8.6.11/8.6.7) id KAA00423; Thu, 6 Apr 1995 10:41:07 +0300
Date: Thu, 6 Apr 1995 10:41:07 +0300
From: Kai Vorma <vode@snakemail.hut.fi>
Message-Id: <199504060741.KAA00423@vinkku.hut.fi>
To: "Justin T. Gibbs" <gibbs@estienne.CS.Berkeley.EDU>
Cc: Kai.Vorma@hut.fi, "Jordan K. Hubbard" <jkh@freefall.cdrom.com>,
        "Rodney W. Grimes" <rgrimes@gndrsh.aac.dev.com>,
        jkh@freebsd.org (Jordan K. Hubbard), CVS-commiters@time.cdrom.com,
        cvs-ports@time.cdrom.com
Subject: Re: cvs commit: ports/net/sup/patches patch-ae 
In-Reply-To: <199504060655.XAA21894@estienne.cs.berkeley.edu>
References: <199504060644.JAA25387@vinkku.hut.fi>
	<199504060655.XAA21894@estienne.cs.berkeley.edu>
Reply-To: Kai.Vorma@hut.fi
Sender: cvs-ports-owner@freebsd.org
Precedence: bulk

Justin T. Gibbs writes:

 > It also doesn't work when the "delete" option is turned on so it gives
 > you a false sense of security.  SUP will go ahead and delete the file
 > you're working on.

No. Look at the patches. Every unlink uses rname as which is the one
with #sup suffix.

 > Please, just back this out for now.  The real way to implement this is to
 > piggy back it ontop of my "rename" option that i've had working and tested
 > for months at TCS.  I was just cleaning up the comments so I could submit
 > it when this stuff allstarted to happen.  The rename option takes care of 
 > all of the delete problems and you can simply write your feature as an 
 > extension to it (ie pretend that we are renaming the destination file to 
 > filename.#sup and the rest of the book keeping is handled automatically).

That's fine. I wanted a simple but important feature for myself and
thought someone else would like it. If you have something better I
will gladly use it. But before that happens I will use sup and my
patch because it works now.

..vode