The Conan package manager (a leading C++ package manager) uses Sha256.
On Wed, 16 Oct 2024, 11:11 pm Gary Gregory, <garydgreg...@gmail.com> wrote: > I think it's a bit confusing to have both 256 and 512 sha files. We > shouldn't imo give the impression that you should check both. One is plenty > imo, 512. > > Gary > > On Wed, Oct 16, 2024, 7:05 AM Robert Middleton <rmiddle...@apache.org> > wrote: > > > Notes(none of these are blockers): > > * Fuzzing page doesn't render > > correctly(https://logging.staged.apache.org/log4cxx/1.3.0/fuzzing.html) > > * The review kit link leads to a 404?? > > * SHA256 sums are not needed, but it doesn't hurt to add. I checked > > only the SHA512 sums. > > * The new release is missing a few files that were in the 1.2.0 > > release, but those are all github/administrative files so it doesn't > > cause an issue > > > > Checksums/signatures are good > > Builds and runs(tests pass) on Debian 11. > > I checked the files from SVN > > > > Looks good to me. +1 > > > > -Robert Middleton > > > > On Tue, Oct 15, 2024 at 3:21 AM Piotr P. Karwasz > > <piotr.karw...@gmail.com> wrote: > > > > > > Hi Stephen, > > > > > > On Tue, 15 Oct 2024 at 06:33, Stephen Webb <swebb2...@gmail.com> > wrote: > > > > Please download, test, and cast your votes on this mailing list. > > > > > > > > [ ] +1, release the artifacts > > > > [ ] -1, don't release, because ... > > > > > > +1, release the artifacts > > > > > > I ran the build on my Debian system: > > > > > > piotr@migi:~$ uname -a > > > Linux migi 6.1.0-18-amd64 #1 SMP PREEMPT_DYNAMIC Debian 6.1.76-1 > > > (2024-02-01) x86_64 GNU/Linux > > > piotr@migi:~$ cat /etc/debian_version > > > 12.7 > > > piotr@migi:~$ LANG= apt policy libapr1 libaprutil1 > > > libapr1: > > > Installed: 1.7.2-3 > > > Candidate: 1.7.2-3 > > > Version table: > > > *** 1.7.2-3 500 > > > 500 https://deb.debian.org/debian bookworm/main amd64 Packages > > > 100 /var/lib/dpkg/status > > > libaprutil1: > > > Installed: 1.6.3-1 > > > Candidate: 1.6.3-1 > > > Version table: > > > *** 1.6.3-1 500 > > > 500 https://deb.debian.org/debian bookworm/main amd64 Packages > > > 100 /var/lib/dpkg/status > > > piotr@migi:~/workspace/rc/log4cxx-1.3.0$ g++ --version > > > g++ (Debian 12.2.0-14) 12.2.0 > > > Copyright (C) 2022 Free Software Foundation, Inc. > > > This is free software; see the source for copying conditions. There is > > NO > > > warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR > > PURPOSE. > > > > > > I checked: > > > > > > 1. The hashes and signature of the files in SVN. > > > 2. Ran the build and tests using both source archives. The effective > > > configuration was: > > > -- Available appenders: > > > -- Async Appender .................. : ON > > > -- ODBC Appender ................... : OFF > > > -- DB Appender ..................... : ON > > > -- SMTP Appender ................... : OFF > > > -- XMLSocketAppender ............... : ON > > > -- SocketHubAppender ............... : ON > > > -- SyslogAppender .................. : ON > > > -- Using syslog.h .................. : ON > > > -- TelnetAppender .................. : ON > > > -- NTEventLogAppender .............. : OFF > > > -- OutputDebugStringAppender ....... : ON > > > -- ConsoleAppender ................. : ON > > > -- FileAppender .................... : ON > > > -- RollingFileAppender ............. : ON > > > -- MultiprocessRollingFileAppender . : OFF > > > -- Available layouts: > > > -- HTMLLayout ...................... : ON > > > -- JSONLayout ...................... : ON > > > -- PatternLayout ................... : ON > > > -- SimpleLayout .................... : ON > > > -- XMLLayout ....................... : ON > > > -- FMTLayout ....................... : OFF > > > 3. Verified that the files attached to > > > https://github.com/apache/logging-log4cxx/actions/runs/11049448312 are > > > the same as in the SVN repo. > > > > > > Piotr > > >