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

cmd/go: Followup step for CL 12192 external test repo. #12769

Closed
dmitshur opened this issue Sep 28, 2015 · 3 comments
Closed

cmd/go: Followup step for CL 12192 external test repo. #12769

dmitshur opened this issue Sep 28, 2015 · 3 comments
Milestone

Comments

@dmitshur
Copy link
Contributor

This is an issue that should be resolved by @rsc. Ian Lance Taylor has asked me to create an issue for this:

Please open an issue for this at https://golang.org/issue . Thanks.

I am following up with one remaining step that should be done by @rsc for a merged CL https://go-review.googlesource.com/#/c/12192/. This is really really minor cleanup issue, but it should not take more than 1 minute for @rsc to resolve (and I just want to get it checked off from my TODO list).

I will quote the CL conversation here for context:

Dmitri Shuralyov

Just in case it's at all helpful, I've rewritten the history as it
should be with new import paths ("github.com/rsc/go-get-issue-9224-{cmd,lib}")
and pushed that on "rsc-import-paths" branches:
https://github.com/shurcooL/go-get-test-cmd/commits/rsc-import-paths
https://github.com/shurcooL/go-get-test-lib/commits/rsc-import-paths

Russ Cox

Just in case it's at all helpful, I've rewritten the history as it
should be with new import paths ("github.com/rsc/go-get-issue-9224-{cmd,lib}")
and pushed that on "rsc-import-paths" branches:
https://github.com/shurcooL/go-get-test-cmd/commits/rsc-import-paths
https://github.com/shurcooL/go-get-test-lib/commits/rsc-import-paths

Thanks. Pushed those.

Dmitri Shuralyov

Pushed those.

https://github.com/rsc/go-get-issue-9224-lib looks good.

But https://github.com/rsc/go-get-issue-9224-cmd still has the wrong default branch "work". Its "master" branch should be set default (in GitHub repo settings at https://github.com/rsc/go-get-issue-9224-cmd/settings) and "work" branch can be deleted.

Dmitri Shuralyov

Just checking in here, the above followup step is still not done.

Issue and Resolution

The issue is that Russ Cox has pushed the changes to the repo, but the branch he pushed to is not the default branch.

The resolution, as I commented above, is:

But https://github.com/rsc/go-get-issue-9224-cmd still has the wrong default branch "work". Its "master" branch should be set default (in GitHub repo settings at https://github.com/rsc/go-get-issue-9224-cmd/settings) and "work" branch can be deleted.

Note that go get ignores the default branch set on GitHub, so master is used anyway. But it can be confusing to see the wrong default branch in the github repo.

@adg adg assigned rsc Sep 28, 2015
@adg adg added this to the Go1.6 milestone Sep 28, 2015
@dmitshur
Copy link
Contributor Author

Note that go get ignores the default branch set on GitHub, so master is used anyway.

#9032 suggests that this might've changed, although in my testing just now it doesn't seem to be:

$ go get github.com/rsc/go-get-issue-9224-cmd
$ cd $GOPATH/src/github.com/rsc/go-get-issue-9224-cmd
$ git branch
* master
  work

(This was done while work branch was still the default branch on that GitHub repo.)

@rsc
Copy link
Contributor

rsc commented Nov 5, 2015

Made master the default branch and deleted work.

@rsc rsc closed this as completed Nov 5, 2015
@dmitshur
Copy link
Contributor Author

dmitshur commented Nov 5, 2015

Thank you very much! Confirmed, this issue is resolved.

@golang golang locked and limited conversation to collaborators Nov 4, 2016
@rsc rsc removed their assignment Jun 23, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants