DMARC-Filter: OpenDMARC Filter v1.4.2 delorie.com 51JDu8V23966580
DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org B19CB3858401
Date: Wed, 19 Feb 2025 22:54:37 +0900
To: cygwin@cygwin.com
Subject: [Bug] libtool (was Re: lost output after pipe redirection)
Message-Id: <20250219225437.a2d307e940c92916ef1fb89f@nifty.ne.jp>
In-Reply-To: <20250219215849.81bc27632686b1ed9f44b7d5@nifty.ne.jp>
References: <6bc117eb-ece2-43a7-91ae-6ab17d62f6ca@gmail.com>
 <20250218205647.41b64c0fded65dc9bc681589@nifty.ne.jp>
 <32abf18d-488e-4658-b006-96c62cfd1f00@gmail.com>
 <20250219073813.7b26b8db73e055b9f1d2c75b@nifty.ne.jp>
 <20250219204205.ba1dfa7265ba68cf0deee14e@nifty.ne.jp>
 <20250219215849.81bc27632686b1ed9f44b7d5@nifty.ne.jp>
Mime-Version: 1.0
Content-Type: multipart/mixed;
 boundary="Multipart=_Wed__19_Feb_2025_22_54_37_+0900_FS+jHSBrg1CA4_v."
Precedence: list
From: Takashi Yano via Cygwin <cygwin@cygwin.com>
Reply-To: Takashi Yano <takashi.yano@nifty.ne.jp>
Errors-To: cygwin-bounces~archive-cygwin=delorie.com@cygwin.com
Sender: "Cygwin" <cygwin-bounces~archive-cygwin=delorie.com@cygwin.com>

This is a multi-part message in MIME format.

--Multipart=_Wed__19_Feb_2025_22_54_37_+0900_FS+jHSBrg1CA4_v.
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit

Hi Achim,

On Wed, 19 Feb 2025 21:58:49 +0900
Takashi Yano wrote:
> On Wed, 19 Feb 2025 20:42:05 +0900
> Takashi Yano wrote:
> > On Wed, 19 Feb 2025 07:38:13 +0900
> > Takashi Yano wrote:
> > > On Tue, 18 Feb 2025 21:59:36 +0100
> > > Marco Atzeriwrote:
> > > > On 18/02/2025 12:56, Takashi Yano via Cygwin wrote:
> > > > > Hi Marco,
> > > > > 
> > > > > On Mon, 17 Feb 2025 09:28:11 +0100
> > > > > Marco Atzeri wrote:
> > > > >> Hi Takashi,
> > > > >>
> > > > >> I think there is still issue on pipe redirection output lost.
> > > > >> I have two cases where during package tests the test are reported as
> > > > >> failures as the output is missing, while running the test stand alone
> > > > >> the output is produced as expected.
> > > > >>
> > > > 
> > > > >> --------------------------------------------------------------
> > > > > 
> > > > > This is not a pipe problem, but just a path problem for gdbmtool.
> > > > > Please try the patch attached.
> > > > 
> > > > Thanks Takashi,
> > > > 
> > > > it works.
> > > > I reported the solution upstream.
> > > 
> > > I forgot to mention, but this problem does not occur in Linux.
> > > I don't know why.
> > 
> > I digged this deeper.
> > 
> > I noticed that this is not a simple path problem. Even if gdbmtool.exe
> > is in the path, running it fails if the directory named gdbmtool exists
> > in the path prior to gdmtool.exe.
> > 
> > Simple test case which reproduce this problem is attached. This problem
> > occurs if the executable is linked with libtool.
> > 
> > The expected output of the test case is:
> > Hello 1
> > Hello 2
> > Hello 3
> > 
> > However, the output in cygwin is:
> > Hello 1
> > 127
> > Hello 3
> > 
> > The culprit would be libtool or cygwin1.dll. libtool is the large
> > scale shell script, so I do not find the cause so far.
> 
> This seems to be a problem of libtool. With the test case I attached
> previous mail, the stub executable should call execv() with:
> <path to the test case>/b/.libs/hello.exe
> because hello.exe is in the directory "b".
> However, it calls execv() with:
> <path to he test case>/a/.libs/hello.exe
> 
> Achim, any idea?

I think I have found the cause. It is an upstream bug of libtool.
The function check_executable() generated by ltmain.sh returns 1
for directory, but this is not correct behaviour, I think.

I confirmed that the patch attached solves the issue.

What do you think?

-- 
Takashi Yano <takashi.yano@nifty.ne.jp>

--Multipart=_Wed__19_Feb_2025_22_54_37_+0900_FS+jHSBrg1CA4_v.
Content-Type: text/plain;
 name="check-executable.patch"
Content-Disposition: attachment;
 filename="check-executable.patch"
Content-Transfer-Encoding: base64

LS0tIG9yaWdzcmMvbGlidG9vbC0yLjUuMy9idWlsZC1hdXgvbHRtYWluLmluCTIwMjUtMDItMTkg
MjE6MDc6NDMuMTU0MDI0NzAwICswOTAwDQorKysgc3JjL2xpYnRvb2wtMi41LjMvYnVpbGQtYXV4
L2x0bWFpbi5pbgkyMDI1LTAyLTE5IDIyOjM4OjI1Ljc2NTYxOTMwMCArMDkwMA0KQEAgLTQwODUs
NyArNDA4NSw3IEBAIGNoZWNrX2V4ZWN1dGFibGUgKGNvbnN0IGNoYXIgKnBhdGgpDQogICBpZiAo
KCFwYXRoKSB8fCAoISpwYXRoKSkNCiAgICAgcmV0dXJuIDA7DQogDQotICBpZiAoKHN0YXQgKHBh
dGgsICZzdCkgPj0gMCkNCisgIGlmICgoc3RhdCAocGF0aCwgJnN0KSA+PSAwKSAmJiAhU19JU0RJ
UiAoc3Quc3RfbW9kZSkNCiAgICAgICAmJiAoc3Quc3RfbW9kZSAmIChTX0lYVVNSIHwgU19JWEdS
UCB8IFNfSVhPVEgpKSkNCiAgICAgcmV0dXJuIDE7DQogICBlbHNlDQo=

--Multipart=_Wed__19_Feb_2025_22_54_37_+0900_FS+jHSBrg1CA4_v.
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Content-Disposition: inline


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

--Multipart=_Wed__19_Feb_2025_22_54_37_+0900_FS+jHSBrg1CA4_v.--