X-Recipient: archive-cygwin AT delorie DOT com
DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org D74AA3858430
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cygwin.com;
	s=default; t=1694170791;
	bh=k39Aw4V7IAJHkZXVUJROqt2vRU0gStZj1JYp3byp09M=;
	h=Date:To:Subject:References:In-Reply-To:List-Id:List-Unsubscribe:
	 List-Archive:List-Post:List-Help:List-Subscribe:From:Reply-To:Cc:
	 From;
	b=AKiavFgFrjP7VJvDqQjR/vh0RtbOahaDJHlqoQTSYcxGnA6Lc9Cso/MAXgrdEHFOu
	 8xw/Eg946z3anhrw+2UsjnxqL2AKJCJD/J+9fQAFqoVfovu+uuZNJhv+KY1CarXxcI
	 CJrLzKNDq00RA3rFPDS9Iu+pekAKvtUF9xvdkwRM=
X-Original-To: cygwin AT cygwin DOT com
Delivered-To: cygwin AT cygwin DOT com
DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org C24543858CD1
Date: Fri, 8 Sep 2023 12:59:36 +0200
To: cygwin AT cygwin DOT com
Subject: Re: NFS mkfifo support in cygwin 3.5.0 Re: [ANNOUNCEMENT] cygwin
 3.4.9-1
Message-ID: <ZPr+mFTvEfYfD+8T@calimero.vinschen.de>
Mail-Followup-To: cygwin AT cygwin DOT com
References: <announce DOT 20230906172530 DOT 526501-1-corinna-cygwin AT cygwin DOT com>
 <CANH4o6MSXBxMBsqgtWbNe9rPETNgY-D5QxJO9AsLwe+tm0WkDw AT mail DOT gmail DOT com>
 <CALXu0UcRuepZC_jH1+rzqLxDuhCtC4h94wgfVxkBxH9fSDEH7A AT mail DOT gmail DOT com>
MIME-Version: 1.0
Content-Disposition: inline
In-Reply-To: <CALXu0UcRuepZC_jH1+rzqLxDuhCtC4h94wgfVxkBxH9fSDEH7A@mail.gmail.com>
X-BeenThere: cygwin AT cygwin DOT com
X-Mailman-Version: 2.1.30
Precedence: list
List-Id: General Cygwin discussions and problem reports <cygwin.cygwin.com>
List-Unsubscribe: <https://cygwin.com/mailman/options/cygwin>,
 <mailto:cygwin-request AT cygwin DOT com?subject=unsubscribe>
List-Archive: <https://cygwin.com/pipermail/cygwin/>
List-Post: <mailto:cygwin AT cygwin DOT com>
List-Help: <mailto:cygwin-request AT cygwin DOT com?subject=help>
List-Subscribe: <https://cygwin.com/mailman/listinfo/cygwin>,
 <mailto:cygwin-request AT cygwin DOT com?subject=subscribe>
From: Corinna Vinschen via Cygwin <cygwin AT cygwin DOT com>
Reply-To: cygwin AT cygwin DOT com
Cc: Corinna Vinschen <corinna-cygwin AT cygwin DOT com>
Content-Type: text/plain; charset="utf-8"
Errors-To: cygwin-bounces+archive-cygwin=delorie DOT com AT cygwin DOT com
Sender: "Cygwin" <cygwin-bounces+archive-cygwin=delorie DOT com AT cygwin DOT com>
Content-Transfer-Encoding: 8bit
X-MIME-Autoconverted: from base64 to 8bit by delorie.com id 388AxqXs020521

On Sep  8 06:48, Cedric Blancher via Cygwin wrote:
> On Thu, 7 Sept 2023 at 23:20, Martin Wege via Cygwin <cygwin AT cygwin DOT com> wrote:
> >
> > On Wed, Sep 6, 2023 at 5:27 PM Corinna Vinschen via Cygwin-announce
> > via Cygwin <cygwin AT cygwin DOT com> wrote:
> > >
> > > The following packages have been uploaded to the Cygwin distribution:
> > >
> > > * cygwin-3.4.9-1
> > > * cygwin-devel-3.4.9-1
> > > * cygwin-doc-3.4.9-1
> > >
> > > Bug Fixes
> > > ---------
> > >
> > > - Fix a bug introduced in cygwin 3.4.0 that switch_to_nat_pipe flag is
> > >   not cleared properly when non-cygwin app is terminated in the case
> > >   where pseudo console is not activated.
> > >
> > > - For the time being, disable creating special files using mknod/mkfifo
> > >   on NFS.
> > >   Addresses: https://cygwin.com/pipermail/cygwin/2023-August/254266.html
> >
> > While I am unhappy that you disabled this for Cygwin 3.4.9-1, we
> > noticed that you added mkfifo support for Cygwin 3.5.0 in
> > https://cygwin.com/git/?p=newlib-cygwin.git;a=commit;h=622fb0776ea333dd708ff312f08ec98311138fbe
> > 🙏😁 :-)
> >
> > Thanks
> >
> > We are testing 3.5.0-0.423.g7ced682549ae.x86_64 - which includes
> > 622fb0776ea333dd708ff312f08ec98311138fbe - right now, feedback will
> > come on Monday
> 
> That's amazing news.
> 
> But an old bug crept in:
> H is NFS file system
> $ uname -a
> CYGWIN_NT-10.0-19045 bastille0x44ef 3.5.0-0.423.g7ced682549ae.x86_64
> 2023-09-07 17:19 UTC x86_64 Cygwin
> $ mount
> C:/cygwin64/bin on /usr/bin type ntfs (binary,auto)
> C:/cygwin64/lib on /usr/lib type ntfs (binary,auto)
> C:/cygwin64 on / type ntfs (binary,auto)
> C: on /cygdrive/c type ntfs (binary,posix=0,user,noumount,auto)
> H: on /cygdrive/h type nfs (binary,posix=0,user,noumount,auto)
> M: on /cygdrive/m type nfs (binary,posix=0,user,noumount,auto)
> N: on /cygdrive/n type nfs (binary,posix=0,user,noumount,auto)
> O: on /cygdrive/o type nfs (binary,posix=0,user,noumount,auto)
> P: on /cygdrive/p type nfs (binary,posix=0,user,noumount,auto)
> $ cd /cygdrive/h/tmp/
> $ mkfifo f1.fifo ; echo $?
> 0
> $ mkfifo -m 0600 f2.fifo ; echo $?
> mkfifo: cannot set permissions of 'f2.fifo': No such file or directory
> 1
> 
> So chmod() for a FIFO inode on NFS fails. Tested with MSFT NFSv3 and
> new builds of the NFSv4.1 ms-nfs41-client filesystems.

Did you actually test this with 3.4.8?  It never worked on NFS.

Just to be clear, the above creates a Cygwin FIFO.  The situation
is different with native FIFOs, created on the host.  With those,
chmod worked before because native FIFOs were handled like normal files,
except in stat(2).  Now that they are handled as FIFOs, the mechanism
to change the file mode doesn't work anymore, because it depends on
FIFOs being WIndows shortcuts.

I will look into that at one point, but it's not a regression.


Corinna

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