Sigh. There's a catch 22 here. If we tag an RC as 1.9.0, we'll have to keep editing the tag every time there's a new RC. I agree that removing the check seems like the best course of action.
On Sun, Dec 17, 2017 at 12:01 AM, Bolke de Bruin <bdbr...@gmail.com> wrote: > This is just matter of setting the tag in the repo right? > > We should remove that check or make it not fail at least. It is ridiculous. > > B. > > Verstuurd vanaf mijn iPad > > > Op 17 dec. 2017 om 07:32 heeft Joy Gao <j...@wepay.com> het volgende > geschreven: > > > > Ahh, tested the build on a fresh virtualenv, which succeeded *the first > > time* given gitpython was not installed and it skipped the assertion > check > > <https://github.com/apache/incubator-airflow/blob/1.9. > 0rc8/setup.py#L68-L73>. > > Build fails on re-installs :( > > > >> On Fri, Dec 15, 2017 at 6:01 PM, Feng Lu <fen...@google.com.invalid> > wrote: > >> > >> +0.5 (non-binding) > >> > >> Looks like the version(1.9.0) and tag(1.9.0rc8) is mismatched, > >> <https://github.com/apache/incubator-airflow/blob/1.9.0rc8/setup.py#L87 > > > >> which will cause the installation (pip install or python setup) to error > >> out and fail. > >> nit: mind also updating the release log " > >> https://github.com/apache/incubator-airflow/blob/1.9.0rc8/CHANGELOG.txt > " > >> > >> > >> On Fri, Dec 15, 2017 at 3:21 PM, Driesprong, Fokko <fo...@driesprong.frl > > > >> wrote: > >> > >>> +1 binding > >>> > >>> Op vr 15 dec. 2017 om 23:39 schreef Bolke de Bruin <bdbr...@gmail.com> > >>> > >>>> +1, binding > >>>> > >>>> Checked sigs, version, source is there (did not check build), bin is > >>> there. > >>>> > >>>> Bolke > >>>> > >>>> Verstuurd vanaf mijn iPad > >>>> > >>>>> Op 15 dec. 2017 om 23:31 heeft Joy Gao <j...@wepay.com> het volgende > >>>> geschreven: > >>>>> > >>>>> +1, binding > >>>>> > >>>>> Thank you Chris! > >>>>> > >>>>> On Fri, Dec 15, 2017 at 2:30 PM, Chris Riccomini < > >>> criccom...@apache.org> > >>>>> wrote: > >>>>> > >>>>>> Hey all, > >>>>>> > >>>>>> (Last time, I hope)^2 > >>>>>> > >>>>>> I have cut Airflow 1.9.0 RC8. This email is calling a vote on the > >>>> release, > >>>>>> which will last for 72 hours. Consider this my (binding) +1. > >>>>>> > >>>>>> Airflow 1.9.0 RC8 is available at: > >>>>>> > >>>>>> https://dist.apache.org/repos/dist/dev/incubator/airflow/1.9.0rc8/ > >>>>>> > >>>>>> apache-airflow-1.9.0rc8+incubating-source.tar.gz is a source > >> release > >>>> that > >>>>>> comes with INSTALL instructions. > >>>>>> apache-airflow-1.9.0rc8+incubating-bin.tar.gz is the binary Python > >>>> "sdist" > >>>>>> release. > >>>>>> > >>>>>> Public keys are available at: > >>>>>> > >>>>>> https://dist.apache.org/repos/dist/release/incubator/airflow/ > >>>>>> > >>>>>> The release contains no new JIRAs. Just a version fix. > >>>>>> > >>>>>> I also had to change the version number to exclude the `rc6` string > >> as > >>>> well > >>>>>> as the "+incubating" string, so it's now simply 1.9.0. This will > >> allow > >>>> us > >>>>>> to rename the artifact without modifying the artifact checksums when > >>> we > >>>>>> actually release. > >>>>>> > >>>>>> See JIRAs that were in 1.9.0RC7 and before (see previous VOTE email > >>> for > >>>>>> full list). > >>>>>> > >>>>>> Cheers, > >>>>>> Chris > >>>>>> > >>>> > >>> > >> >