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

x/build/cmd/release: go1.12rc1 source tarball includes empty gocache and tmp directories #30181

Closed
dmitshur opened this issue Feb 12, 2019 · 5 comments
Labels
Builders x/build issues (builders, bots, dashboards) FrozenDueToAge NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. release-blocker
Milestone

Comments

@dmitshur
Copy link
Contributor

dmitshur commented Feb 12, 2019

Issue #29906 has been mostly resolved, but there's still a minor issue where the go1.12rc1.src.tar.gz release archive (from #30175) has empty gocache and tmp directories:

$ tree
.
├── go1.12rc1.src.tar.gz
└── go1.12rc1.src
    ├── go
    │   ├── AUTHORS
    │   ├── CONTRIBUTING.md
    │   ├── CONTRIBUTORS
    │   └── ...
    ├── gocache
    └── tmp

All other release archives do not have gocache and tmp directories.

These directories are empty, so the impact is low, but we should still clean this up further. /cc @bradfitz @FiloSottile @julieqiu @andybons

The source tarball also has a slightly different structure: it contains a top-level "go1.12rc1.src" directory, which then has a "go" directory inside. All others have "go" at the root. This isn't a problem, just mentioning it because it might be helpful for identifying the cause.

@dmitshur dmitshur added Builders x/build issues (builders, bots, dashboards) NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. labels Feb 12, 2019
@bradfitz
Copy link
Contributor

Sent https://go-review.googlesource.com/c/build/+/161967 (not sure what gopherbot is doing)

@gopherbot
Copy link

Change https://golang.org/cl/161967 mentions this issue: cmd/release: clean gocache and tmp in source mode too

@rsc rsc modified the milestones: Unreleased, Go1.12 Feb 12, 2019
@rsc
Copy link
Contributor

rsc commented Feb 12, 2019

This gets to be "Go1.12" not "Unreleased" because it affects the bits that go into the release (even though the command itself is not released).

@bradfitz
Copy link
Contributor

The source tarball also has a slightly different structure: it contains a top-level "go1.12rc1.src" directory, which then has a "go" directory inside. All others have "go" at the root. This isn't a problem, just mentioning it because it might be helpful for identifying the cause.

I think the cause is you double clicking it on a Mac and macOS auto-extracting it using its prefix as the directory name. I don't see such a top-level directory actually existing:

bradfitz@gdev:~/src/golang.org/x/build/cmd/release$ curl --silent https://dl.google.com/go/go1.10.4.src.tar.gz | tar -zt | grep -v ^go/
bradfitz@gdev:~/src/golang.org/x/build/cmd/release$ curl --silent https://dl.google.com/go/go1.11.5.src.tar.gz | tar -zt | grep -v ^go/
gocache/
tmp/
bradfitz@gdev:~/src/golang.org/x/build/cmd/release$ curl --silent https://dl.google.com/go/go1.11.4.src.tar.gz | tar -zt | grep -v ^go/
bradfitz@gdev:~/src/golang.org/x/build/cmd/release$ 

@dmitshur
Copy link
Contributor Author

@bradfitz You're absolutely right, that's what happened. I didn't know about that behavior of Archive Utility.app on macOS.

I also confirmed the "go" directory is indeed top-level in go1.12rc1.src.tar.gz by making quick use of the compress/gzip and archive/tar packages. Thanks for helping me understand this.

@golang golang locked and limited conversation to collaborators Feb 12, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Builders x/build issues (builders, bots, dashboards) FrozenDueToAge NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. release-blocker
Projects
None yet
Development

No branches or pull requests

4 participants