Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Release - patch release 0.14.1 #5340

Closed
dslomov opened this issue Jun 7, 2018 · 17 comments
Closed

Release - patch release 0.14.1 #5340

dslomov opened this issue Jun 7, 2018 · 17 comments
Assignees
Labels

Comments

@dslomov
Copy link
Contributor

dslomov commented Jun 7, 2018

A patch release of 0.14 (#5056) with a fix for #5336

@dslomov dslomov added the release label Jun 7, 2018
@dslomov dslomov self-assigned this Jun 7, 2018
@dslomov
Copy link
Contributor Author

dslomov commented Jun 7, 2018

scripts/release/release.sh create 0.14.1 10a4de9 4b80f2455e7e49a95f3a4c9102a67a57dad52207

@dslomov
Copy link
Contributor Author

dslomov commented Jun 7, 2018

I will cherry-pick a fix for #5308 into this patch release.

@meteorcloudy
Copy link
Member

@lfpino

@dslomov
Copy link
Contributor Author

dslomov commented Jun 7, 2018

rc2: scripts/release/release.sh create 0.14.1 10a4de9 4b80f2455e7e49a95f3a4c9102a67a57dad52207 6b1635279e8b33dc1ac505ac81825e38f8797a14

@dslomov
Copy link
Contributor Author

dslomov commented Jun 7, 2018

rc2 published, downstream tests started: https://buildkite.com/bazel/bazel-with-downstream-projects-bazel/builds/293

@dslomov
Copy link
Contributor Author

dslomov commented Jun 7, 2018

Some stuff in downstream projects fails, but they also fail in trunk pipelines in the same way.
WDYT @aehlig (current Bazel Sheriff)

@jmillikin-stripe
Copy link
Contributor

I've got another cherrypick request: 4ec0a75 to fix a regression in the Skylark implementation of new_git_repository (#5146).

@dslomov
Copy link
Contributor Author

dslomov commented Jun 8, 2018

4ec0a75 does not fix a regression - it is a functionality change, or am I missing something?

@dslomov
Copy link
Contributor Author

dslomov commented Jun 8, 2018

@dslomov
Copy link
Contributor Author

dslomov commented Jun 8, 2018

Ping @petemounce for Chocolatey
Ping @ilovezfs for Homebrew

@ilovezfs
Copy link
Contributor

ilovezfs commented Jun 8, 2018

I've shipped 0.14.1 in Homebrew: Homebrew/homebrew-core#28804

@jmillikin-stripe
Copy link
Contributor

4ec0a75 does not fix a regression - it is a functionality change, or am I missing something?

Long ago (June 2017) I noticed that new_git_repository was broken on MacOS for some repos (#3127), and got a fix merged (#3142).

It was broken again (regressed) by some refactoring in February 2018 (a22a287), but I didn't notice because the repo we needed the fix for had been pinned to an older Bazel version (0.11). We just tried to update it to 0.14.0 and it failed due to a name conflict between BUILD and build/, since the MacOS filesystem is case-insensitive.

@dslomov
Copy link
Contributor Author

dslomov commented Jun 8, 2018

I see. I don't think this meets the bar for patch release for 0.14. Let's cherry pick it into 0.15 (#5057).
I can't help noticing that #3142 has no tests, hence the bug.

@jmillikin-stripe
Copy link
Contributor

Yes, it's definitely my fault for not having tests there. I just hadn't thought there would be so much development velocity in that part of the code.

0.15 should be fine in terms of timeline, it looks like it's already in the RC process. Thanks for the cherrypick.

@petemounce
Copy link
Contributor

0.14.1 out on chocolatey.

@hlopko
Copy link
Member

hlopko commented Jun 25, 2018

Can this issue be closed now?

@meteorcloudy
Copy link
Member

Yes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants