DomainKey-Signature: a=rsa-sha1; c=nofws; d=sourceware.org; h=list-id
	:list-unsubscribe:list-subscribe:list-archive:list-post
	:list-help:sender:mime-version:in-reply-to:references:date
	:message-id:subject:from:to:content-type; q=dns; s=default; b=SI
	NpKpkzgbftRNzFju4AnIzyIW+aEDgQomGGmUfuIenW5tksMozwJg7OD93jxBhDwX
	KEPvF7ZtPc/z25sp3aEP8T3XZnRDxDdLiLax2yaRP9PhJ73489f1pceen53xHQDQ
	8VQvdicRqAfEd+JrkSIA9kncgNbLO6G8cdVl8z5HI=
Mailing-List: contact cygwin-help@cygwin.com; run by ezmlm
Sender: cygwin-owner@cygwin.com
Mail-Followup-To: cygwin@cygwin.com
MIME-Version: 1.0
In-Reply-To: <20150430105644.GM3657@calimero.vinschen.de>
References: <CAOC2fq8dCzjjZU3eHWWh_h5s7O0M+z482+prWz9HsQWVqAjVng@mail.gmail.com>	<397346277.275101.1430383452664.JavaMail.zimbra@ceos.com.au>	<20150430105644.GM3657@calimero.vinschen.de>
Date: Thu, 30 Apr 2015 04:11:08 -0700
Message-ID: <CAOC2fq8iArHbVJzBcdrCFUNf4uTwt9b-2=WN82uyryFox6vVEw@mail.gmail.com>
Subject: Re: Trouble with Git 2.1.x pushing to repos over Samba
From: Michael Enright <mike@kmcardiff.com>
To: cygwin@cygwin.com
Content-Type: text/plain; charset=UTF-8

Corinna,

Do you think the snapshot would change the outcome in my case?

I haven't used a snapshot before. Is there a tutorial on how to get
onto and off of a snapshot? Or should I test by using a VM?

I myself am going to be on a short vacation and compressing too much
into tomorrow to do anything with a snapshot very soon.


On Thu, Apr 30, 2015 at 3:56 AM, Corinna Vinschen
<corinna-cygwin@cygwin.com> wrote:
> Hi John,
>
> On Apr 30 18:44, John Orr wrote:
>> > From: "Michael Enright"
>> > $ git push origin master
>> > fatal: '//host/path/to/repo.git/' does not appear to be a git repository
>> > fatal: Could not read from remote repository.
>> [...]
>> #: john@johndesktop:/cygdrive/l ; ls -ld .git/objects/
>> drwxr-xr-x 1 john Unix_Group+1000 0 Nov 13 14:13 .git/objects/
>>
>> (albeit, Corinna, with my group issue still not yet resolved)
>
> You tried the /etc/group tweak as I suggested in my latest mail in that
> thread, I take it?
>
>> access(/cygdrive/l/.git, R_OK) returned 0
>> access(/cygdrive/l/.git, W_OK) returned 0
>> access(/cygdrive/l/.git, X_OK) returned -1
>>
>> The last test is the one run by git, that makes it reject my /cygdrive/l/.git directory.
>>
>> Not sure if that's relevant, but just in case.
>
> Thanks for the info.  I found a really dumb bug in my code.  The
> access() function is using a Windows function for access checking under
> the hood.  To account for the Samba account mapping in Cygwin, there's
> a function converting the S-1-22-x-y SIDs in the file's ACL to Windows
> SIDs if there *is* a mapping.  But I made a small mistake which has
> a big result: The ACL is not completly copied over, thus the Windows
> function has to deal with an incomplete ACL.
>
> I fixed that in the git repo and uploaded new snapshots to
> https://cygwin.com/snapshots/  Please give them a try.  Don't use the
> snapshots for anything else for the time being!
>
>   PLEASE TEST ASAP AND REPORT BACK!
>
>   I'll be unavailable for a few weeks starting tomorrow, so I'd like to
>   do a bugfix Cygwin release, preferredly today, if this patch works as
>   desired.
>
>
> Thanks,
> Corinna
>
>
> P.S.: As a side-note: While this patch (hopefully) reverts this code to
>       work as pre-1.7.34, it seems that the internal Windows access
>       check function is not quite up to the task for Samba shares in
>       scenarios as John's one.  It will always report back the access of
>       the "others" part of POSIX permission bits.  Only with the new
>       mapping of S-1-22-x-y SIDs to real WIndows accounts, or with
>       winbindd-supported mapping, the Windows access check will really
>       work as desired.
>
> --
> Corinna Vinschen                  Please, send mails regarding Cygwin to
> Cygwin Maintainer                 cygwin AT cygwin DOT com
> Red Hat

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