X-Recipient: archive-cygwin AT delorie DOT com
DomainKey-Signature: a=rsa-sha1; c=nofws; d=sourceware.org; h=list-id
	:list-unsubscribe:list-subscribe:list-archive:list-post
	:list-help:sender:message-id:date:from:mime-version:to:subject
	:references:in-reply-to:content-type:content-transfer-encoding;
	 q=dns; s=default; b=XOFsqQMAIk6/lxhup1LvHslHi91AmkU7i7PuEEbSqRx
	hR0bU9+K0rrPHCglwJCO4nw5FT8LJangdXEA6NfJJazByWZEtQ16vFQ2oOP2GUTq
	1qOjyXcCK9YDlh8Mu/W40JAdUXDHM1Ioh90tpg9kMMzU/ivjur1EiLbGJ8yuTfH0
	=
DKIM-Signature: v=1; a=rsa-sha1; c=relaxed; d=sourceware.org; h=list-id
	:list-unsubscribe:list-subscribe:list-archive:list-post
	:list-help:sender:message-id:date:from:mime-version:to:subject
	:references:in-reply-to:content-type:content-transfer-encoding;
	 s=default; bh=lvDQV5WviSsH/WjiBxsgzNQ1S4A=; b=qQkX0cPNZNIeFeUyA
	Rsk56F0yVjcioNvJRIjFgZmtQiZI8pdzukVxvo6Wv3RZ1XjtYyEr2A98TA8NIOiG
	dVMsCBYbjv3QRSj24p+GIJ/UoUz8w3GqybhiPn4uVfUBsjsDb3Oxutecp7YHFkxp
	aHexDKSSUw+sq9l4kImEm4yN2g=
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
Authentication-Results: sourceware.org; auth=none
X-Virus-Found: No
X-Spam-SWARE-Status: No, score=-1.4 required=5.0 tests=AWL,BAYES_00,RCVD_IN_BL_SPAMCOP_NET,RCVD_IN_DNSWL_NONE,RP_MATCHES_RCVD autolearn=no version=3.3.2
X-HELO: mailout04.t-online.de
Message-ID: <5450001C.2030609@t-online.de>
Date: Tue, 28 Oct 2014 21:44:12 +0100
From: Christian Franke <Christian DOT Franke AT t-online DOT de>
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:29.0) Gecko/20100101 Firefox/29.0 SeaMonkey/2.26.1
MIME-Version: 1.0
To: cygwin AT cygwin DOT com
Subject: Re: [ANNOUNCEMENT] TEST RELEASE: Cygwin 1.7.33-0.3
References: <announce DOT 20141027214634 DOT GA20607 AT calimero DOT vinschen DOT de>
In-Reply-To: <announce.20141027214634.GA20607@calimero.vinschen.de>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
X-IsSubscribed: yes

Corinna Vinschen wrote:
> Hi Cygwin friends and users,
>
>
> I just released a 3rd TEST version of the next upcoming Cygwin release,
> 1.7.33-0.3.
>
> Changes compared to the former test version 1.7.33-0.2:
>
> - Add -b/--remove-all option to setfacl to reduce the ACL to only the
>    entries representing POSIX permission bits.

Thanks, I missed that functionality several times.

But unfortunaltely this change introduced a regression.
This testcase is from the syslog-ng-config script and worked with 
1.7.33-0.2:

$ setfacl -m u:system:rwx FILE
setfacl: Invalid argument

Hmm...
$ git diff cygwin-1.7.32-0.2
...
-  else if ((lcnt = acl (path, GETACL, MAX_ACL_ENTRIES, lacl)) < 0
-      || (lcnt = modacl (lacl, lcnt, acls, cnt)) < 0
-      || (action != Delete && (lcnt = addmissing (lacl, lcnt)) < 0)
-      || (lcnt = acl (path, SETACL, lcnt, lacl)) < 0)
...
+    default:
+      if ((lcnt = acl (path, GETACL, MAX_ACL_ENTRIES, lacl)) < 0
+         || (lcnt = modacl (lacl, lcnt, acls, cnt)) < 0
+         || (lcnt = addmissing (lacl, lcnt) < 0)   // <==== Hmm.... :-)
+         || (lcnt = acl (path, SETACL, lcnt, lacl)) < 0)


> - Drop code removing current working directory from the default DLL
>    search path.  Instead:
>    When exec'ing applications, check if $PATH exists and is non-empty.  If not,
>    add PATH variable with Cygwin installation directory as content to Windows
>    environment to allow loading of Cygwin system DLLs.

This works as expected.

Interesting:
The windows PATH is set to "\\?\X:\cygwin_dir\bin" instead of 
"X:\cygwin_dir\bin". This apparently works to find DLLs by 
CreateProcess() but not to find commands by cmd.exe:

$ export PATH=/bin

$ /cygdrive/c/Windows/System32/cmd /c PATH
PATH=C:\cygwin\bin

$ /cygdrive/c/Windows/System32/cmd /c uname
CYGWIN_NT-6.1-WOW64

$ unset PATH

$ /cygdrive/c/Windows/System32/cmd /c PATH
PATH=\\?\C:\cygwin\bin

$ /cygdrive/c/Windows/System32/cmd /c uname
... some localized 'uname not found' message


Thanks,
Christian


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