> -----Original Message----- > From: Jim Jagielski [mailto:[email protected]] > Sent: Tuesday, February 20, 2018 6:10 AM > To: [email protected] > Subject: Re: [VOTE] Release httpd-2.4.30 > > Another thing that helps is providing some heads-up > that a T&R will actually be happening... Yeah, you > had noted the you were going to T&R on Monday > (in the "T&R of 2.4.30 Proposal" thread) but sending > out a quick "I plan on doing this in X hours" notice > allows for some possible last-minute items to pop > up. > > Just a suggestion.
Sure, I can do that. > > > On Feb 19, 2018, at 9:44 PM, Jim Jagielski <[email protected]> wrote: > > > > I would suggest using the scripts from > > > > https://svn.apache.org/repos/asf/httpd/site/trunk/tools > > > > for as much of the work as possible since they have been used > > and vetted for several years. Indeed, I did for everything except the tagging (which is why I created a script to do it). I also created one to push to dev/release dist repos for convenience. As mentioned, the goal is to automate the process as much as possible so with the gap in existing tooling, I went ahead and filled it. I guess I just misinterpreted the instructions and didn’t think to review SVN history of the previous releases as Joe mentioned. Aside from this omission, are we otherwise in good shape? > > > >> On Feb 19, 2018, at 7:28 PM, [email protected] wrote: > >> > >> Ah, I see. I created a script that does the tagging based on the directions > here. > >> http://httpd.apache.org/dev/release.html > >> > >> It was unclear in those instructions that one should commit the change to > AP_SERVER_DEVBUILD_BOOLEAN. > >> > >> Instead I did the following: > >> ... > >> # Set AP_SERVER_DEVBUILD_BOOLEAN to 0 in include/ap_release.h. > >> perl -pi -e 's/(#define\s+AP_SERVER_DEVBUILD_BOOLEAN\s+)\d/${1}0/g' > include/ap_release.h > >> > >> # Create an official X.Y.Z tag based on the candidate tree. > >> svn copy "$src_dir" "$tags_dir/$version" > >> > >> # Revert the change to include/ap_release.h setting > AP_SERVER_DEVBUILD_BOOLEAN back to 1, and bump > AP_SERVER_PATCHLEVEL_NUMBER > >> perl -pi -e ' > >> s/(#define\s+AP_SERVER_DEVBUILD_BOOLEAN\s+)\d/${1}1/g; > >> > >> if(/(#define\s+AP_SERVER_PATCHLEVEL_NUMBER\s+)(\d+)$/){ > >> $new = $2 + 1; > >> $_ = "${1}${new}\n"; > >> } > >> ' include/ap_release.h > >> ... > >> > >> This begets a tarball that has the Boolean set to 0, but no > commit/revert/bump (instead just an apparent bump): > >> > http://svn.apache.org/viewvc/httpd/httpd/tags/2.4.30/include/ap_release. > h?view=markup#l47 > >> > >> It makes sense that the tag comes from a specific commit where the > variable was flipped... Should I adjust the script and retry? > >> > >> -- > >> Daniel Ruggeri > >> > >> From: Jim Jagielski [mailto:[email protected]] > >> Sent: Monday, February 19, 2018 9:46 AM > >> To: [email protected] > >> Subject: Re: [VOTE] Release httpd-2.4.30 > >> > >> Hmmm... I'm not seeing the patch where > AP_SERVER_DEVBUILD_BOOLEAN > >> in ap_release.h is set to 0 > >> > >> How does your release process work? What we've always > >> done is make the req changes to the branch and then copy > >> from that branch to the tag. So the tag itself must refer to > >> a specific SVN number on the http-2.4 branch but I'm > >> not seeing where that is done. > >> > >> > >> On Feb 19, 2018, at 9:54 AM, mailto:[email protected] wrote: > >> > >> Hi, all; > >> Please find below the proposed release tarball and signatures: > >> https://dist.apache.org/repos/dist/dev/httpd/ > >> > >> I would like to call a VOTE over the next few days to release this > >> candidate > tarball as 2.4.30: > >> > >> [ ] +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. > >> -- > >> Daniel Ruggeri > >> > >> -- Daniel Ruggeri
