Re: [PATCH v4 0/2] credential: improvements to erase in helpers

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Jun 15, 2023 at 02:09:15PM -0700, Junio C Hamano wrote:

> "M Hickford via GitGitGadget" <gitgitgadget@xxxxxxxxx> writes:
> 
> > M Hickford (2):
> >   credential: avoid erasing distinct password
> >   credential: erase all matching credentials
> >
> >  Documentation/git-credential.txt   |   2 +-
> >  Documentation/gitcredentials.txt   |   2 +-
> >  builtin/credential-cache--daemon.c |  17 +++--
> >  builtin/credential-store.c         |  15 +++--
> >  credential.c                       |   7 +-
> >  credential.h                       |   2 +-
> >  t/lib-credential.sh                | 103 +++++++++++++++++++++++++++++
> >  7 files changed, 128 insertions(+), 20 deletions(-)
> 
> It is helpful to reviewers to describe/summarize, in your own words,
> what changed since the previous version, in the cover letter.
> 
> The range-diff generated for the versions can serve as a good
> supporting material, and it would help you while writing that
> summary, but not a substitute for the summary.

Yeah, I agree that would have made reviewing much easier. :)

That said, I just re-reviewed the patches themselves, and everything now
looks good to me.

-Peff



[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux