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

invalid module name in go.mod #583

Closed
cboitel opened this issue Jan 13, 2022 · 3 comments
Closed

invalid module name in go.mod #583

cboitel opened this issue Jan 13, 2022 · 3 comments
Labels
helping with an issue Debugging happening to identify the problem

Comments

@cboitel
Copy link

cboitel commented Jan 13, 2022

What steps did you take:
go get github.com/vmware-tanzu/carvel-ytt@v0

What happened:

go get: github.com/vmware-tanzu/[email protected]: parsing go.mod:
        module declares its path as: github.com/k14s/ytt
                but was required as: github.com/vmware-tanzu/carvel-ytt

What did you expect:
Ability to get the module or install the binary with go install

Anything else you would like to add:
Requires to update module name in go.mod. Without this change, nobody will be able to use ytt as a go library or install it simply using go install standard command.

Environment:

  • ytt version (use ytt --version): N/A
  • OS (e.g. from /etc/os-release): Ubuntu 20.04 LTS + GO 1.17.6

Vote on this request

This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.

👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"

We are also happy to receive and review Pull Requests if you want to help working on this issue.

@cboitel cboitel added bug This issue describes a defect or unexpected behavior carvel triage This issue has not yet been triaged for relevance labels Jan 13, 2022
@pivotaljohn
Copy link
Contributor

Carvel started out as a suite named "k14s" short for Kubernetes Tools.

Some time back, VMware formally sponsored this project and adopted it into its OSS program. The repository was moved at that time, but the module was not renamed.
We have been tracking this across the suite in vmware-tanzu/carvel#153, but haven't yet made it a priority.

Apologies for the inconvenience. Given that we're seeing more and more folks reach for a programmatic integration, I've prioritized doing this rename, soon: #585

In the meantime, feel free to rely on GitHub's redirection service:

go get github.com/k14s/ytt@v0

@pivotaljohn pivotaljohn added helping with an issue Debugging happening to identify the problem and removed bug This issue describes a defect or unexpected behavior carvel triage This issue has not yet been triaged for relevance labels Jan 14, 2022
@pivotaljohn
Copy link
Contributor

FWIW, we have some preliminary guidance around integrating with ytt in this way: https://github.com/vmware-tanzu/carvel-ytt/blob/develop/examples/integrating-with-ytt/apis.md

@pivotaljohn
Copy link
Contributor

We'll be tracking the rename work in #585; closing this issue, for now. @cboitel, if you need more support here than this, please feel free to continue the conversation here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
helping with an issue Debugging happening to identify the problem
Projects
None yet
Development

No branches or pull requests

2 participants