Re: [PATCH] popen: set gl_cv_func_popen_works to no on OS/2

2025-02-10 Thread Bruno Haible via Gnulib discussion list
KO Myung-Hun wrote: > Done. Thanks. Patch applied, with a wording tweak in doc/posix-functions/popen.texi. Bruno

Re: [PATCH] popen: set gl_cv_func_popen_works to no on OS/2

2025-02-10 Thread KO Myung-Hun
Bruno Haible wrote: > KO Myung-Hun wrote: >>> If you set gl_cv_func_popen_works to 'no', it will activate the workaround >>> in lib/popen.c. Does this workaround work? Does 'test-[p]open' succeed? >> >> Do you mean 'test-popen'? Then it works. > > Good. Then lib/popen.c does not need to be exten

Re: [PATCH] popen: set gl_cv_func_popen_works to no on OS/2

2025-02-10 Thread Bruno Haible via Gnulib discussion list
KO Myung-Hun wrote: > > If you set gl_cv_func_popen_works to 'no', it will activate the workaround > > in lib/popen.c. Does this workaround work? Does 'test-[p]open' succeed? > > Do you mean 'test-popen'? Then it works. Good. Then lib/popen.c does not need to be extended. > > If yes, we'll also

Re: [PATCH] popen: set gl_cv_func_popen_works to no on OS/2

2025-02-10 Thread KO Myung-Hun
Hi/2. Bruno Haible wrote: > Myung-Hun wrote: >> * m4/popen.m4 (gl_FUNC_POPEN): Set gl_cv_func_popen_works to no on OS/2. > > If you set gl_cv_func_popen_works to 'no', it will activate the workaround > in lib/popen.c. Does this workaround work? Does 'test-open' succeed? Do you mean 'test-popen'?

Re: [PATCH] popen: set gl_cv_func_popen_works to no on OS/2

2025-02-10 Thread Bruno Haible via Gnulib discussion list
Myung-Hun wrote: > * m4/popen.m4 (gl_FUNC_POPEN): Set gl_cv_func_popen_works to no on OS/2. If you set gl_cv_func_popen_works to 'no', it will activate the workaround in lib/popen.c. Does this workaround work? Does 'test-open' succeed? If yes, we'll also need an edit to doc/posix-functions/popen.

[PATCH] popen: set gl_cv_func_popen_works to no on OS/2

2025-02-10 Thread KO Myung-Hun
On OS/2, popen() does not work with closed stdios. And configure hangs on with the test. * m4/popen.m4 (gl_FUNC_POPEN): Set gl_cv_func_popen_works to no on OS/2. --- m4/popen.m4 | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/m4/popen.m4 b/m4/popen.m4 index a68c1e606f..b301