Re: [VOTE] Release apr-1.7.4-rc1 as apr-1.7.4

2023-04-13 Thread Eric Covener
On Wed, Apr 12, 2023 at 10:25 PM Eric Covener  wrote:
>
> Hi all,
>
> Please find below the proposed release tarball and signatures:
>
> https://dist.apache.org/repos/dist/dev/apr/
>
> I would like to call a VOTE over the next few days to release
> this candidate tarball apr-1.7.4-rc1 as 1.7.4:
> [x] +1: It's not just good, it's good enough!
> [ ] +0: Let's have a talk.
> [ ] -1: There's trouble in paradise. Here's what's wrong.


my +1 aix/xlc/ppc64

usual suspects only

Failed TestsTotal   FailFailed %
===
testprocmutex   6  2 33.33%
testsock   17  1  5.88%


[GitHub] [apr] tumatanquang opened a new pull request, #41: Change the minimum OS version from XP to Vista.

2023-04-13 Thread via GitHub


tumatanquang opened a new pull request, #41:
URL: https://github.com/apache/apr/pull/41

   As these topics:
   [Apache 2.2 build with new VC++ 2008 
RTM](https://www.apachelounge.com/viewtopic.php?t=2095)
   [apr 1.7.0 build issues](https://www.apachelounge.com/viewtopic.php?t=8260)
   From version 1.6.5 and earlier, this value does not affect compilation, but 
it will break compilation starting with version 1.7.0. I also tried compiling 
on Visual Studio 2013 and confirmed that the value `0x0501` would break the 
compilation, but after changing to `0x0600`, the compilation went normally.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@apr.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [VOTE] Release apr-1.7.4-rc1 as apr-1.7.4

2023-04-13 Thread Evgeny Kotkov via dev
Eric Covener  writes:

> Hi all,
>
> Please find below the proposed release tarball and signatures:
>
> https://dist.apache.org/repos/dist/dev/apr/
>
> I would like to call a VOTE over the next few days to release
> this candidate tarball apr-1.7.4-rc1 as 1.7.4:

  [X] +1: It's not just good, it's good enough!

Checks performed:
— Verify SHA256 for apr-1.7.4-rc1-candidate.tar.bz2
— Verify SHA256 for apr-1.7.4-rc1-candidate.tar.gz
— Verify SHA256 for apr-1.7.4-rc1-candidate-win32-src.zip
— Verify diff between apr-1.7.4-rc1-candidate-win32-src.zip and
  published apr-1.7.3-win32-src.zip
— Build using apr-1.7.4-rc1-candidate-win32-src.zip on Windows x64/CMake
— Run APR tests on Windows x64
— Run Subversion 1.14.x tests with this APR version on Windows x64


Thanks,
Evgeny Kotkov


Re: [VOTE] Release apr-1.7.4-rc1 as apr-1.7.4

2023-04-13 Thread Eric Covener
On Thu, Apr 13, 2023 at 1:08 PM Evgeny Kotkov
 wrote:
>
> Eric Covener  writes:
>
> > Hi all,
> >
> > Please find below the proposed release tarball and signatures:
> >
> > https://dist.apache.org/repos/dist/dev/apr/
>
> First time voting for an APR release, but isn't apr-1.7.4-rc1-win32-src.zip
> missing from the set of artifacts?

yes, omission in porting the httpd automation. I manually added now.


Re: [VOTE] Release apr-1.7.4-rc1 as apr-1.7.4

2023-04-13 Thread Evgeny Kotkov via dev
Eric Covener  writes:

> Hi all,
>
> Please find below the proposed release tarball and signatures:
>
> https://dist.apache.org/repos/dist/dev/apr/

First time voting for an APR release, but isn't apr-1.7.4-rc1-win32-src.zip
missing from the set of artifacts?

I think that it was present during the vote for APR 1.7.3:
[[[
> svn log https://dist.apache.org/repos/dist/dev/apr -c60838 -v

Changed paths:
   M /dev/apr/Announcement1.x.html
   M /dev/apr/Announcement1.x.txt
   A /dev/apr/apr-1.7.3-rc1-win32-src.zip
   A /dev/apr/apr-1.7.3-rc1-win32-src.zip.asc
   A /dev/apr/apr-1.7.3-rc1-win32-src.zip.sha256
   A /dev/apr/apr-1.7.3-rc1.tar.bz2
   A /dev/apr/apr-1.7.3-rc1.tar.bz2.asc
   A /dev/apr/apr-1.7.3-rc1.tar.bz2.sha256
   A /dev/apr/apr-1.7.3-rc1.tar.gz
   A /dev/apr/apr-1.7.3-rc1.tar.gz.asc
   A /dev/apr/apr-1.7.3-rc1.tar.gz.sha256

1.7.3. rc1
]]]


Thanks,
Evgeny Kotkov


Re: [VOTE] Release apr-1.7.4-rc1 as apr-1.7.4

2023-04-13 Thread Yann Ylavic
On Thu, Apr 13, 2023 at 1:08 PM Yann Ylavic  wrote:
>
> On Thu, Apr 13, 2023 at 1:02 PM Eric Covener  wrote:
> >
> > New for this release we are only toggling it on the release tag (sic), 
> > similar to httpd release process.
>
> Maybe a bump (to 1.7.5 here) is missing in 1.7.x after tagging now?
> Currently apr_version.h is still at 1.7.4-dev it seems.

Nevermind, this probably happens when the release is accepted..


Re: [VOTE] Release apr-1.7.4-rc1 as apr-1.7.4

2023-04-13 Thread Yann Ylavic
On Thu, Apr 13, 2023 at 4:25 AM Eric Covener  wrote:
>
> I would like to call a VOTE over the next few days to release
> this candidate tarball apr-1.7.4-rc1 as 1.7.4:

[X] +1: It's not just good, it's good enough!

All tests pass on Debian 11 and 12, sums/sigs/version/dirname OK.

Thanks Eric for RMing!


Re: [VOTE] Release apr-1.7.4-rc1 as apr-1.7.4

2023-04-13 Thread Yann Ylavic
On Thu, Apr 13, 2023 at 1:02 PM Eric Covener  wrote:
>
> New for this release we are only toggling it on the release tag (sic), 
> similar to httpd release process.

Maybe a bump (to 1.7.5 here) is missing in 1.7.x after tagging now?
Currently apr_version.h is still at 1.7.4-dev it seems.


Re: [VOTE] Release apr-1.7.4-rc1 as apr-1.7.4

2023-04-13 Thread Eric Covener
New for this release we are only toggling it on the release tag (sic),
similar to httpd release process.

On Thu, Apr 13, 2023, 6:42 AM Yann Ylavic  wrote:

> On Thu, Apr 13, 2023 at 12:16 PM SteffenAL  wrote:
> >
> > it's me.  I build from svn, and there is :
> >
> > #define APR_IS_DEV_VERSION
>
> Yeah, this is restored immediately after tagging to not let a branch
> in a "release" state.
> You could checkout tags/apr-1.7.4-rc1 rather than branches/1.7.x for
> testing, the vote is about the tarballs but I suppose testing the tag
> is fine too if it helps (maybe related with Windows' CRLF vs tarball's
> LF in the source files in your case..).
>
>
> Regards;
> Yann.
>


Re: [VOTE] Release apr-1.7.4-rc1 as apr-1.7.4

2023-04-13 Thread Yann Ylavic
On Thu, Apr 13, 2023 at 12:16 PM SteffenAL  wrote:
>
> it's me.  I build from svn, and there is :
>
> #define APR_IS_DEV_VERSION

Yeah, this is restored immediately after tagging to not let a branch
in a "release" state.
You could checkout tags/apr-1.7.4-rc1 rather than branches/1.7.x for
testing, the vote is about the tarballs but I suppose testing the tag
is fine too if it helps (maybe related with Windows' CRLF vs tarball's
LF in the source files in your case..).


Regards;
Yann.


Re: Release APR 1.7.4?

2023-04-13 Thread Yann Ylavic
On Thu, Apr 13, 2023 at 12:15 PM Evgeny Kotkov
 wrote:
>
> Yann Ylavic  writes:
>
> > Do we want to include r1909117 in 1.7.4 (to call abort() instead of
> > assert() in apr_base64 when NDEBUG)?
> > Sorry for the lateness, but should there be an -rc2 (or in any case..).
>
> If I recall correctly, APR 1.7.x doesn't have apr_base64, because it is a
> part of APR-util in that timeline.

Ah yes sorry for the noise, I keep forgetting which ones are in apr or
in apr-util :)


Regards;
Yann.


Re: [VOTE] Release apr-1.7.4-rc1 as apr-1.7.4

2023-04-13 Thread SteffenAL


Re: Release APR 1.7.4?

2023-04-13 Thread Evgeny Kotkov via dev
Yann Ylavic  writes:

> Do we want to include r1909117 in 1.7.4 (to call abort() instead of
> assert() in apr_base64 when NDEBUG)?
> Sorry for the lateness, but should there be an -rc2 (or in any case..).

If I recall correctly, APR 1.7.x doesn't have apr_base64, because it is a
part of APR-util in that timeline.


Thanks,
Evgeny Kotkov


Re: Release APR 1.7.4?

2023-04-13 Thread Yann Ylavic
On Thu, Apr 13, 2023 at 10:47 AM Evgeny Kotkov via dev
 wrote:
>
> Eric Covener  writes:
>
> > I will RM but it might be a bit piecemeal between now and the weekend.
> >   At the mercy of a lot of inflexible $bigco stuff.
> >
> > Hoping to improve a little this time by porting more of icings amazing
> > httpd release work.
>
> Awesome, thanks!

Do we want to include r1909117 in 1.7.4 (to call abort() instead of
assert() in apr_base64 when NDEBUG)?
Sorry for the lateness, but should there be an -rc2 (or in any case..).


Regards;
Yann.


Re: [VOTE] Release apr-1.7.4-rc1 as apr-1.7.4

2023-04-13 Thread Yann Ylavic
On Thu, Apr 13, 2023 at 9:38 AM SteffenAL  wrote:
>
> It is identified as 1.7.4-dev
>
> In apr_version.h dev version is defined.

I don't see this in the release candidate tarballs (.gz or .bz2),
apr_version.h contains:
/* #undef APR_IS_DEV_VERSION */
so it should not include the "-dev" suffix in APR_VERSION_STRING.
Likewise here: 
https://github.com/apache/apr/blob/1.7.4-rc1-candidate/include/apr_version.h#L72

Could it be from a local /D of yours or something?


Regards;
Yann.


Re: Release APR 1.7.4?

2023-04-13 Thread Evgeny Kotkov via dev
Eric Covener  writes:

> I will RM but it might be a bit piecemeal between now and the weekend.
>   At the mercy of a lot of inflexible $bigco stuff.
>
> Hoping to improve a little this time by porting more of icings amazing
> httpd release work.

Awesome, thanks!


Regards,
Evgeny Kotkov


Re: [VOTE] Release apr-1.7.4-rc1 as apr-1.7.4

2023-04-13 Thread SteffenAL


It is identified as 1.7.4-dev

In apr_version.h dev version is defined.

Steffen



--- Original message ---
Subject: [VOTE] Release apr-1.7.4-rc1 as apr-1.7.4
From: Eric Covener 
To: dev@apr.apache.org 
Date: Thursday, 13/04/2023 04:25

Hi all,

Please find below the proposed release tarball and signatures:

https://dist.apache.org/repos/dist/dev/apr/

I would like to call a VOTE over the next few days to release
this candidate tarball apr-1.7.4-rc1 as 1.7.4:
[ ] +1: It's not just good, it's good enough!
[ ] +0: Let's have a talk.
[ ] -1: There's trouble in paradise. Here's what's wrong.

The computed digests of the tarball up for vote are:


The SVN candidate source is found at tags/1.7.4-rc1-candidate.

--
Eric Covener
cove...@gmail.com