X-Recipient: archive-cygwin AT delorie DOT com
X-SWARE-Spam-Status: No, hits=-1.3 required=5.0	tests=AWL,BAYES_20,KHOP_RCVD_UNTRUST,RCVD_IN_HOSTKARMA_W,RCVD_IN_HOSTKARMA_WL
X-Spam-Check-By: sourceware.org
X-Spam-Processed: mail1.multiplay.co.uk, Thu, 27 Sep 2012 14:40:17 +0100	(not processed: message from trusted or authenticated source)
X-MDRemoteIP: 188.220.16.49
X-Return-Path: prvs=161757989e=killing AT multiplay DOT co DOT uk
X-Envelope-From: killing AT multiplay DOT co DOT uk
X-MDaemon-Deliver-To: cygwin AT cygwin DOT com
Message-ID: <D9227F105C634BA2B30A6B1086A93C5B@multiplay.co.uk>
From: "Steven Hartland" <killing AT multiplay DOT co DOT uk>
To: "Cygwin List" <cygwin AT cygwin DOT com>
Subject: Patch to fix tar directory access time changes under cygwin
Date: Thu, 27 Sep 2012 14:40:21 +0100
MIME-Version: 1.0
Content-Type: text/plain;	format=flowed;	charset="iso-8859-1";	reply-type=original
Content-Transfer-Encoding: 7bit
X-IsSubscribed: yes
Mailing-List: contact cygwin-help AT cygwin DOT com; run by ezmlm
List-Id: <cygwin.cygwin.com>
List-Subscribe: <mailto:cygwin-subscribe AT cygwin DOT com>
List-Archive: <http://sourceware.org/ml/cygwin/>
List-Post: <mailto:cygwin AT cygwin DOT com>
List-Help: <mailto:cygwin-help AT cygwin DOT com>, <http://sourceware.org/ml/#faqs>
Sender: cygwin-owner AT cygwin DOT com
Mail-Followup-To: cygwin AT cygwin DOT com
Delivered-To: mailing list cygwin AT cygwin DOT com

The following little patch fixes tar under cygwin not ignoring
directory change times.

Without it a simple tar can easily fail.

--- src/create.c.orig   2011-01-07 10:04:33.297364800 -0800
+++ src/create.c    2011-01-07 09:51:37.804364800 -0800
@@ -1788,7 +1788,7 @@
           /* Original ctime will change if the file is a directory and
          --remove-files is given */
           && !(remove_files_option && is_dir))
-         || original_size < final_stat.st_size)
+         || (!is_dir && original_size != final_stat.st_size))
        {
          WARNOPT (WARN_FILE_CHANGED,
               (0, 0, _("%s: file changed as we read it"),


Would be good if this could be included in the offical port

    Regards
    Steve


--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple