Re: pipe hang issue when running as SYSTEM

2022-03-29 Thread Takashi Yano
On Fri, 25 Mar 2022 10:01:53 -0700 (PDT) Jeremy Drake wrote: > On Fri, 25 Mar 2022, Takashi Yano wrote: > > > I will submit v3 patch shortly. > > I applied your v3 patch to 3.3.4 in > https://github.com/msys2/msys2-runtime/pull/88 and re-ran my test action, > and it worked as expected this time.

Re: pipe hang issue when running as SYSTEM

2022-03-25 Thread Jeremy Drake via Cygwin
On Fri, 25 Mar 2022, Takashi Yano wrote: > I will submit v3 patch shortly. I applied your v3 patch to 3.3.4 in https://github.com/msys2/msys2-runtime/pull/88 and re-ran my test action, and it worked as expected this time. I've put out a call to test on the msys2-runtime issue to confirm that it

Re: pipe hang issue when running as SYSTEM

2022-03-25 Thread Takashi Yano
On Fri, 25 Mar 2022 00:09:59 -0700 (PDT) Jeremy Drake wrote: > On Fri, 25 Mar 2022, Takashi Yano wrote: > > > I can confirm the sample script hangs indeed if the script is > > running as SYSTEM account. However, in my test, the patch solves > > the issue. > > > > It would be very helpfull if a sim

Re: pipe hang issue when running as SYSTEM

2022-03-25 Thread Roumen Petrov
Hi, Takashi Yano wrote: [SNIP] Just got another report to MSYS2 of this behavior, this time from a Gitlab CI runner that is running as a service and trying to pipe output from Microsoft cl.exe while running configure. This user reports that 3.3.4 with your "[PATCH v2] Cygwin: pipe: Avoid deadl

Re: pipe hang issue when running as SYSTEM

2022-03-25 Thread Jeremy Drake via Cygwin
On Fri, 25 Mar 2022, Takashi Yano wrote: > I can confirm the sample script hangs indeed if the script is > running as SYSTEM account. However, in my test, the patch solves > the issue. > > It would be very helpfull if a simple github repository with > github actions, which can reproduce the issue,

Re: pipe hang issue when running as SYSTEM

2022-03-24 Thread Takashi Yano
On Thu, 24 Mar 2022 20:42:50 -0700 (PDT) Jeremy Drake wrote: > On Tue, 22 Mar 2022, Takashi Yano wrote: > > > > Thanks for the report. This is expected problem as mentioned > > > in b531d6b commit message. However, I could not imagin the > > > situation that the service has multiple writer for the

Re: pipe hang issue when running as SYSTEM

2022-03-24 Thread Jeremy Drake via Cygwin
On Tue, 22 Mar 2022, Takashi Yano wrote: > > Thanks for the report. This is expected problem as mentioned > > in b531d6b commit message. However, I could not imagin the > > situation that the service has multiple writer for the pipe > > and one of them is a non-cygwin app. > > > > Question is: Doe

Re: pipe hang issue when running as SYSTEM

2022-03-22 Thread Takashi Yano
On Tue, 22 Mar 2022 19:18:07 +0900 Takashi Yano wrote: > On Mon, 21 Mar 2022 11:45:57 -0700 (PDT) > Jeremy Drake wrote: > > This issue was reported to MSYS2 as a hang when trying to build libxml2 in > > a Windows docker container. Another user was able to come up with a > > simple reproducer and a

Re: pipe hang issue when running as SYSTEM

2022-03-22 Thread Takashi Yano
On Mon, 21 Mar 2022 11:45:57 -0700 (PDT) Jeremy Drake wrote: > This issue was reported to MSYS2 as a hang when trying to build libxml2 in > a Windows docker container. Another user was able to come up with a > simple reproducer and a reasonable theory as to why it happens. The msys2 > issue is ht

pipe hang issue when running as SYSTEM

2022-03-21 Thread Jeremy Drake via Cygwin
This issue was reported to MSYS2 as a hang when trying to build libxml2 in a Windows docker container. Another user was able to come up with a simple reproducer and a reasonable theory as to why it happens. The msys2 issue is https://github.com/msys2/msys2-runtime/issues/77, and I will quote the