Re: Review Request 42117: Change release script to use rel/ tag prefix.

2016-01-10 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42117/#review113653 --- Ship it! Master (6b768bd) is green with this patch.

Review Request 42117: Change release script to use rel/ tag prefix.

2016-01-10 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42117/ --- Review request for Aurora, John Sirois and Zameer Manji. Repository: aurora

Re: Review Request 42117: Change release script to use rel/ tag prefix.

2016-01-10 Thread Bill Farner
> On Jan. 10, 2016, 10:43 a.m., Zameer Manji wrote: > > Will we also retroactively push `rel/` tags for previous releases? Once this patch lands, yes - i will do this retroactively for old releases, update the website, etc. - Bill ---

Re: Review Request 42117: Change release script to use rel/ tag prefix.

2016-01-10 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42117/#review113655 --- build-support/release/release (line 137)

Re: Review Request 42117: Change release script to use rel/ tag prefix.

2016-01-10 Thread John Sirois
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42117/#review113660 --- Ship it! Ship It! - John Sirois On Jan. 10, 2016, 11:26 a.m.,

Re: Review Request 42117: Change release script to use rel/ tag prefix.

2016-01-10 Thread John Sirois
> On Jan. 10, 2016, 12:08 p.m., John Sirois wrote: > > build-support/release/release, line 137 > > > > > > IIUC, this commit will be pushed and reachable from the tag, but not > > from master. Even if thats not

Re: Review Request 42117: Change release script to use rel/ tag prefix.

2016-01-10 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42117/#review113650 --- build-support/release/release (line 138)

Re: Review Request 42117: Change release script to use rel/ tag prefix.

2016-01-10 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42117/#review113651 --- Ship it! Will we also retroactively push `rel/` tags for previous

Re: Review Request 42117: Change release script to use rel/ tag prefix.

2016-01-10 Thread Bill Farner
> On Jan. 10, 2016, 11:08 a.m., John Sirois wrote: > > build-support/release/release, line 137 > > > > > > IIUC, this commit will be pushed and reachable from the tag, but not > > from master. Even if thats not

Re: Review Request 42117: Change release script to use rel/ tag prefix.

2016-01-10 Thread Bill Farner
> On Jan. 10, 2016, 11:08 a.m., John Sirois wrote: > > build-support/release/release, line 140 > > > > > > No longer true. Removed comment and `refs/tags` prefix. - Bill

Re: Review Request 42117: Change release script to use rel/ tag prefix.

2016-01-10 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42117/ --- (Updated Jan. 10, 2016, 11:58 a.m.) Review request for Aurora, John Sirois and