Remove Roland McGrath et al. from libc.so banner

Message ID CAMe9rOpmHhacPAtrQGYfCJSML+n_oTWJ8ykQTYjt9rF2b7RHag@mail.gmail.com
State New, archived
Headers

Commit Message

H.J. Lu Sept. 1, 2017, 5:14 p.m. UTC
  On Fri, Sep 1, 2017 at 9:41 AM, Joseph Myers <joseph@codesourcery.com> wrote:
> On Fri, 1 Sep 2017, H.J. Lu wrote:
>
>> This is what I am going to check in:
>>
>> * csu/version.c (banner): Remove "by Roland McGrath et al.".
>> * nptl/Banner: Remove "by Ulrich Drepper et al.".
>
> Please see what I said about being sure to update contrib.texi when
> removing such credits.  I think NPTL merits adding to Ulrich Drepper's
> entry in contrib.texi (but given that, the nptl/Banner file could be
> removed completely).

I checked in my patch.   Here is a patch to credit Ulrich Drepper for
libpthread in contrib.texi.   OK for master?

> I think crypt/Banner can be removed (the credit is already in
> contrib.texi).  The reference to a BIND version in contrib.texi should
> probably be updated when removing resolv/Banner.  More credits in
> contrib.texi are needed to remove soft-fp/Banner.  (libidn/Banner serves
> the purpose of indicating whether the last remaining add-on is configured
> in, so I don't suggest removing it until the add-on mechanism is also
> removed and libidn is built unconditionally - however, the corresponding
> credit could be added to contrib.texi now.)

I will get to these when I find time if no one beats me to it.
  

Comments

Joseph Myers Sept. 1, 2017, 5:16 p.m. UTC | #1
On Fri, 1 Sep 2017, H.J. Lu wrote:

> On Fri, Sep 1, 2017 at 9:41 AM, Joseph Myers <joseph@codesourcery.com> wrote:
> > On Fri, 1 Sep 2017, H.J. Lu wrote:
> >
> >> This is what I am going to check in:
> >>
> >> * csu/version.c (banner): Remove "by Roland McGrath et al.".
> >> * nptl/Banner: Remove "by Ulrich Drepper et al.".
> >
> > Please see what I said about being sure to update contrib.texi when
> > removing such credits.  I think NPTL merits adding to Ulrich Drepper's
> > entry in contrib.texi (but given that, the nptl/Banner file could be
> > removed completely).
> 
> I checked in my patch.   Here is a patch to credit Ulrich Drepper for
> libpthread in contrib.texi.   OK for master?

OK.
  

Patch

From 628c52afccdf8f8738855cb046b38f21a4f21bfe Mon Sep 17 00:00:00 2001
From: "H.J. Lu" <hjl.tools@gmail.com>
Date: Fri, 1 Sep 2017 10:05:47 -0700
Subject: [PATCH] Credit Ulrich Drepper for libpthread in contrib.texi

	* manual/contrib.texi: Credit Ulrich Drepper for the POSIX
	Threads Library.
---
 manual/contrib.texi | 1 +
 1 file changed, 1 insertion(+)

diff --git a/manual/contrib.texi b/manual/contrib.texi
index ffdd77191c..fff45b92b9 100644
--- a/manual/contrib.texi
+++ b/manual/contrib.texi
@@ -75,6 +75,7 @@  x86.
 Ulrich Drepper for his many contributions in almost all parts of
 @theglibc{}, including:
 @itemize @bullet
+@item writing most of the POSIX Threads Library originally.
 @item internationalization support, including the @code{locale} and
 @code{localedef} utilities.
 @item Linux i386/ELF support
-- 
2.13.5