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

Reply via email to