* [PATCH v2] libio: Correctly link tst-popen-fork against libpthread
@ 2024-10-25 7:45 Arjun Shankar
2024-10-25 8:31 ` Florian Weimer
0 siblings, 1 reply; 2+ messages in thread
From: Arjun Shankar @ 2024-10-25 7:45 UTC (permalink / raw)
To: libc-alpha; +Cc: Joseph Myers, Florian Weimer
tst-popen-fork failed to build for Hurd due to not being linked with
libpthread. This commit fixes that.
Tested with build-many-glibcs.py for i686-gnu.
---
v1 here:
https://sourceware.org/pipermail/libc-alpha/2024-October/160928.html
Changes in v2: Removed unnecessary check for have-thread-library.
libio/Makefile | 2 ++
1 file changed, 2 insertions(+)
diff --git a/libio/Makefile b/libio/Makefile
index 018c26d971..cd5a3afeeb 100644
--- a/libio/Makefile
+++ b/libio/Makefile
@@ -144,6 +144,8 @@ tests = \
tst_wscanf \
# tests
+$(objpfx)tst-popen-fork: $(shared-thread-library)
+
tests-internal = tst-vtables tst-vtables-interposed
ifeq (yes,$(build-shared))
--
2.46.2
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH v2] libio: Correctly link tst-popen-fork against libpthread
2024-10-25 7:45 [PATCH v2] libio: Correctly link tst-popen-fork against libpthread Arjun Shankar
@ 2024-10-25 8:31 ` Florian Weimer
0 siblings, 0 replies; 2+ messages in thread
From: Florian Weimer @ 2024-10-25 8:31 UTC (permalink / raw)
To: Arjun Shankar; +Cc: libc-alpha, Joseph Myers
* Arjun Shankar:
> tst-popen-fork failed to build for Hurd due to not being linked with
> libpthread. This commit fixes that.
>
> Tested with build-many-glibcs.py for i686-gnu.
> ---
> v1 here:
> https://sourceware.org/pipermail/libc-alpha/2024-October/160928.html
>
> Changes in v2: Removed unnecessary check for have-thread-library.
>
> libio/Makefile | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/libio/Makefile b/libio/Makefile
> index 018c26d971..cd5a3afeeb 100644
> --- a/libio/Makefile
> +++ b/libio/Makefile
> @@ -144,6 +144,8 @@ tests = \
> tst_wscanf \
> # tests
>
> +$(objpfx)tst-popen-fork: $(shared-thread-library)
> +
> tests-internal = tst-vtables tst-vtables-interposed
>
> ifeq (yes,$(build-shared))
This version looks okay to me.
Reviewed-by: Florian Weimer <fweimer@redhat.com>
Thanks,
Florian
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2024-10-25 8:31 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-10-25 7:45 [PATCH v2] libio: Correctly link tst-popen-fork against libpthread Arjun Shankar
2024-10-25 8:31 ` Florian Weimer
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).