This is the mail archive of the cygwin-patches mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Add pldd(1)


On Feb 24 15:31, Christopher Faylor wrote:
> On Fri, Feb 24, 2012 at 10:47:07AM +0100, Corinna Vinschen wrote:
> >On Feb 24 02:38, Yaakov (Cygwin/X) wrote:
> >> The pldd(1) command apparently originates from Solaris and was added to
> >> glibc-2.15[1].  Patches and new file attached.
> >
> >Looks good, works fine.  Please apply.
> 
> It's not entirely fine.  Minor nits:
> 
> - The comment says "pldd.cc" but it's pldd.c.
> 
> - There was no ChangeLog entry announcing its checkin.

Oh, right.  The ChangeLog entry was part of the patch submission, though.
Yaakov, could you apply the ChangeLog as well, please?


Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Project Co-Leader          cygwin AT cygwin DOT com
Red Hat


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]