On Aug  5 17:41, Achim Gratz wrote:
> Corinna Vinschen writes:
> > Same here.  Am I missing something?
> 
> Nothing, but I missed to copy the negation from the corresponding code
> for MD5 even though the description of the patch explicitly makes note
> of the correct logic.  It's fixed already in my repo since I finally
> updated all of Cygwin at work today.
> 
> --8<---------------cut here---------------start------------->8---
> 	Modified   IniDBBuilderPackage.cc
> diff --git a/IniDBBuilderPackage.cc b/IniDBBuilderPackage.cc
> index b41955a..ff92ec2 100644
> --- a/IniDBBuilderPackage.cc
> +++ b/IniDBBuilderPackage.cc
> @@ -268,7 +268,7 @@ IniDBBuilderPackage::buildInstallSize (const std::string &size)
>  void
>  IniDBBuilderPackage::buildInstallSHA512 (unsigned char const *sha512)
>  {
> -  if (sha512 && cbpv.source()->sha512_isSet) {
> +  if (sha512 && !cbpv.source()->sha512_isSet) {
>      memcpy (cbpv.source()->sha512sum, sha512, sizeof cbpv.source()->sha512sum);
>      cbpv.source()->sha512_isSet = true;
>    }
> @@ -277,7 +277,7 @@ IniDBBuilderPackage::buildInstallSHA512 (unsigned char const *sha512)
>  void
>  IniDBBuilderPackage::buildSourceSHA512 (unsigned char const *sha512)
>  {
> -  if (sha512 && cbpv.source()->sha512_isSet) {
> +  if (sha512 && !cbpv.source()->sha512_isSet) {
>      memcpy (cspv.source()->sha512sum, sha512, sizeof cspv.source()->sha512sum);
>      cbpv.source()->sha512_isSet = true;
>    }
> --8<---------------cut here---------------end--------------->8---
> 
> OK for push?

Yes, thank you.


Corinna

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