Don't mention linuxthreads in Depend files
diff mbox

Message ID Pine.LNX.4.64.1405192208520.743@digraph.polyomino.org.uk
State New
Headers show

Commit Message

Joseph Myers May 19, 2014, 10:09 p.m. UTC
I noticed that some of the Depend files, used to determine the
subdirectory build order in sysd-sorted, still mentioned linuxthreads,
although it hasn't been supported for many years.  This patch removes
those references.  In the case of nscd, it substitutes an nptl
reference, since I believe there is a fact a thread library dependence
there; the others already mentioned nptl.

Note that I am not at all confident in the completeness of these
Depend files.

Note also that references to linuxthreads remain in a comment in
sysdeps/unix/sysv/linux/ia64/Versions, and in manual/maint.texi,
manual/signal.texi and scripts/documented.sh.

Tested x86_64 that the installed shared libraries are unchanged by the
patch (as is sysd-sorted).

2014-05-19  Joseph Myers  <joseph@codesourcery.com>

	* nscd/Depend (linuxthreads): Remove.
	(nptl): Add.
	* resolv/Depend (linuxthreads): Remove.
	* rt/Depend (linuxthreads): Remove.

Comments

Siddhesh Poyarekar May 20, 2014, 4:42 a.m. UTC | #1
On Mon, May 19, 2014 at 10:09:55PM +0000, Joseph S. Myers wrote:
> I noticed that some of the Depend files, used to determine the
> subdirectory build order in sysd-sorted, still mentioned linuxthreads,
> although it hasn't been supported for many years.  This patch removes
> those references.  In the case of nscd, it substitutes an nptl
> reference, since I believe there is a fact a thread library dependence
> there; the others already mentioned nptl.
> 
> Note that I am not at all confident in the completeness of these
> Depend files.
> 
> Note also that references to linuxthreads remain in a comment in
> sysdeps/unix/sysv/linux/ia64/Versions, and in manual/maint.texi,
> manual/signal.texi and scripts/documented.sh.
> 
> Tested x86_64 that the installed shared libraries are unchanged by the
> patch (as is sysd-sorted).
> 
> 2014-05-19  Joseph Myers  <joseph@codesourcery.com>
> 
> 	* nscd/Depend (linuxthreads): Remove.
> 	(nptl): Add.
> 	* resolv/Depend (linuxthreads): Remove.
> 	* rt/Depend (linuxthreads): Remove.

Looks good to me.

Siddhesh

> 
> diff --git a/nscd/Depend b/nscd/Depend
> index fe673ba..6c1aa44 100644
> --- a/nscd/Depend
> +++ b/nscd/Depend
> @@ -1 +1 @@
> -linuxthreads
> +nptl
> diff --git a/resolv/Depend b/resolv/Depend
> index 8d2587b..6c1aa44 100644
> --- a/resolv/Depend
> +++ b/resolv/Depend
> @@ -1,2 +1 @@
> -linuxthreads
>  nptl
> diff --git a/rt/Depend b/rt/Depend
> index 8d2587b..6c1aa44 100644
> --- a/rt/Depend
> +++ b/rt/Depend
> @@ -1,2 +1 @@
> -linuxthreads
>  nptl
> 
> -- 
> Joseph S. Myers
> joseph@codesourcery.com

Patch
diff mbox

diff --git a/nscd/Depend b/nscd/Depend
index fe673ba..6c1aa44 100644
--- a/nscd/Depend
+++ b/nscd/Depend
@@ -1 +1 @@ 
-linuxthreads
+nptl
diff --git a/resolv/Depend b/resolv/Depend
index 8d2587b..6c1aa44 100644
--- a/resolv/Depend
+++ b/resolv/Depend
@@ -1,2 +1 @@ 
-linuxthreads
 nptl
diff --git a/rt/Depend b/rt/Depend
index 8d2587b..6c1aa44 100644
--- a/rt/Depend
+++ b/rt/Depend
@@ -1,2 +1 @@ 
-linuxthreads
 nptl