X-Recipient: archive-cygwin AT delorie DOT com
X-SWARE-Spam-Status: No, hits=-2.0 required=5.0 	tests=AWL,BAYES_00,SARE_MSGID_LONG40,SPF_PASS
X-Spam-Check-By: sourceware.org
MIME-Version: 1.0
In-Reply-To: <4B115A56.2060705@alice.it>
References: <4B115A56 DOT 2060705 AT alice DOT it>
Date: Sat, 28 Nov 2009 19:12:53 -0500
Message-ID: <a13b5a590911281612r796d112bw1208a869433d2b88@mail.gmail.com>
Subject: Re: [ANNOUNCEMENT] [1.7] Updated: cygwin-1.7.0-67
From: Robert Pendell <shinji AT elite-systems DOT org>
To: cygwin AT cygwin DOT com
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
X-IsSubscribed: yes
Mailing-List: contact cygwin-help AT cygwin DOT com; run by ezmlm
Precedence: bulk
List-Id: <cygwin.cygwin.com>
List-Unsubscribe: <mailto:cygwin-unsubscribe-archive-cygwin=delorie DOT com AT cygwin DOT 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

On Sat, Nov 28, 2009 at 12:13 PM, Angelo Graziosi wrote:
>
> Corinna Vinschen wrote:
>>
>> Fixes a few last-minute bugs:
>
> Now, it seems that creating directories and files in some manner acquires=
 an extra unexpected '+' flag. For example (from Cygwin.bat or MinTTY):
>
>
> $ echo "Not " > foo1.txt
>
> $ echo "Not OK" > foo2.txt
>
> $ diff -Naur foo1.txt foo2.txt > foo.diff
>
> $ cp foo1.txt foo.txt
>
> $ patch foo.txt foo.diff
> patching file foo.txt
>
> $ mkdir foo
>
> $ ls -lrtd foo*
> -rw-r--r-- =C2=A01 ... =C2=A0 5 28 Nov 18:02 foo1.txt
> -rw-r--r-- =C2=A01 ... =C2=A0 7 28 Nov 18:02 foo2.txt
> -rw-r--r-- =C2=A01 ... 124 28 Nov 18:02 foo.diff
> -rw-r--r--+ 1 ... =C2=A0 7 28 Nov 18:03 foo.txt
> drwxr-xr-x+ 1 ... =C2=A0 0 28 Nov 18:03 foo
>
> See the extra '+' for 'foo.txt' and 'foo'!
>
> I didn't note this with previous version -66, -65...
>
> Is it to be expected with -67?
>
>
> Ciao,
> Angelo.
>

( resending as text-only... >.< )

Looks like there is a change in permissions with the file during the
diff program run.  I might run a strace on it if someone tells me the
appropriate command to do.  Anyways here is the relevant part with
icacls before and after the diff command.

shinji AT Balthasar ~
$ icacls foo.txt
foo.txt Balthasar\shinji:(R,W,D,WDAC,WO)
        Balthasar\None:(R)
        Everyone:(R)

Successfully processed 1 files; Failed processing 0 files

shinji AT Balthasar ~

$ patch foo.txt foo.diff
patching file foo.txt

shinji AT Balthasar ~
$ icacls foo.txt
foo.txt Balthasar\shinji:(R,W,D,WDAC,WO)
        Balthasar\None:(R)
        Everyone:(R)
        NT AUTHORITY\SYSTEM:(F)
        BUILTIN\Administrators:(F)

Successfully processed 1 files; Failed processing 0 files

The 2 additional permission lines on the second icacls run is what
causes the + to appear.

Robert Pendell
shinji AT elite-systems DOT org
CAcert Assurer
"A perfect world is one of chaos."

--
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