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

New package: zk-0.14.0 #50400

Merged
merged 1 commit into from
May 23, 2024
Merged

New package: zk-0.14.0 #50400

merged 1 commit into from
May 23, 2024

Conversation

cinerea0
Copy link
Contributor

Testing the changes

  • I tested the changes in this PR: YES

New package

Unfortunately they haven't made a new release since switching to the new org, so the go_import_path will be weird until the next release.


Closes #50398

@cinerea0 cinerea0 added the new-package This PR adds a new package label May 18, 2024
@MIvanchev
Copy link
Contributor

@cinerea0 Be advised that this package downloads tons of dependencies off Github none of which are checksumed... They probably need to be pre-downloaded.

@classabbyamp
Copy link
Member

that's how go works... and they are checksummed

@MIvanchev
Copy link
Contributor

Ah, nice about the checksums, I didn't really look... We can still pre-download the packages in case we don't trust the Go infrastructure to properly verify them.

@classabbyamp
Copy link
Member

we're not going to do that all 380 go packages...

@MIvanchev
Copy link
Contributor

It's about 40, the file includes older versions but yeah, a bit on the populous side here...

@classabbyamp
Copy link
Member

classabbyamp commented May 20, 2024

I meant all the other packages that do the exact same thing because this is how go's build system works

@cinerea0 cinerea0 merged commit b9d6622 into void-linux:master May 23, 2024
8 checks passed
@cinerea0 cinerea0 deleted the zk branch May 23, 2024 19:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new-package This PR adds a new package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Package request: zk
3 participants