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

srcpkgs/: clean up lots of Go packages #28863

Merged
merged 7 commits into from Feb 19, 2021
Merged

Conversation

ericonr
Copy link
Member

@ericonr ericonr commented Feb 18, 2021

Depending on git for building can sometimes happen due to issue #27690,
where instead of using the unpacked tarball, Go clones the package's
repository itself, since the tarball we provided doesn't contain the
go_import_path we pass to it in the command line.

Since we should require as few dependencies as possible, and this
measure can help catch issues of building the wrong version of a Go
package, we remove git from hostmakedepends where possible.

I'm still pulling in packages that do this, will take a while.

Fix small typos, expand some variables, leave comments for updates.
Upstream force pushed the same tag, so checksum for the tarball changed.
Only changes were timestamps for files and some small differences in
their .travis.yml file, which doesn't affect the build.
@ericonr ericonr changed the title [WIP] srcpkgs/: most Go packages shouldn't have git in hostmakedepends. [WIP] srcpkgs/: clean up lots of Go packages Feb 19, 2021
@ericonr ericonr changed the title [WIP] srcpkgs/: clean up lots of Go packages srcpkgs/: clean up lots of Go packages Feb 19, 2021
@ericonr ericonr merged commit f35df89 into void-linux:master Feb 19, 2021
@ericonr ericonr deleted the goclean branch February 19, 2021 15:08
@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 20, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant