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

+encore.dev #2897

Merged
merged 4 commits into from
Sep 7, 2023
Merged

+encore.dev #2897

merged 4 commits into from
Sep 7, 2023

Conversation

kevinmichaelchen
Copy link
Contributor

@kevinmichaelchen kevinmichaelchen commented Aug 14, 2023

blocked by #2896

@kevinmichaelchen
Copy link
Contributor Author

go mod download is failing

go: cuelang.org/[email protected]: GOPROXY list is not the empty string, but contains no entries

@jhheider
Copy link
Contributor

Peculiar. No package in our pantry sets GOPROXY. Maybe try explicitly setting it to the empty string?

@jhheider
Copy link
Contributor

seems like it might be related to 1.21.0. i'll see what I can find.

https://github.com/teaxyz/pantry/actions/runs/5860883067/job/15889888782

@kevinmichaelchen
Copy link
Contributor Author

golang/go#61928

@jhheider
Copy link
Contributor

yup d3a903f

@jhheider
Copy link
Contributor

Based on their build instructions and release logic, it looks like this is going to involved building and depending on encore-go:

https://github.com/encoredev/encore/blob/main/.github/workflows/release.yml

and setting some envvars: https://github.com/encoredev/encore/blob/main/CONTRIBUTING.md

@kevinmichaelchen
Copy link
Contributor Author

I didn't realize they forked the Go language! Is best practice for https://github.com/encoredev/go to be a separate Tea package?

@jhheider
Copy link
Contributor

I think so. Since it is renamed, not much risk of conflict.

@jhheider
Copy link
Contributor

jhheider commented Sep 7, 2023

gonna mark this as go. thanks, @kevinmichaelchen !

@jhheider jhheider marked this pull request as ready for review September 7, 2023 21:41
@jhheider jhheider merged commit b44d191 into pkgxdev:main Sep 7, 2023
20 checks passed
@kevinmichaelchen kevinmichaelchen deleted the new/encore.dev branch September 16, 2023 23:20
@mxcl
Copy link
Member

mxcl commented Sep 28, 2023

like what is this exactly?

Since it causes conflict with google’s go and it seems to just be some kind of fork of go, I'm wondering what it is? I can't find mention of it at encore.dev.

Refs pkgxdev/pkgx#748

@jhheider
Copy link
Contributor

We can almost certainly add a link to encore-go and omit the provides for vanilla go. I think that's even in their docs. I'll do that today, to remove this conflict.

@mxcl
Copy link
Member

mxcl commented Sep 28, 2023

IMO it's ok just to remove the provides and then encore users can just do tea +encore.dev go. However the longer form encore-go in addition as a provides is also a good idea

@jhheider
Copy link
Contributor

c90d397

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants