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

unable to get go get github.com/Jeffail/gabs #38

Closed
Rajkumrjsn opened this issue Sep 12, 2017 · 6 comments
Closed

unable to get go get github.com/Jeffail/gabs #38

Rajkumrjsn opened this issue Sep 12, 2017 · 6 comments

Comments

@Rajkumrjsn
Copy link

github.com/Jeffail/gabs

../go/src/github.com/Jeffail/gabs/gabs.go:429: e.SetEscapeHTML undefined (type *json.Encoder has no field or method SetEscapeHTML)
../go/src/github.com/Jeffail/gabs/gabs.go:436: e.SetIndent undefined (type *json.Encoder has no field or method SetIndent)
../go/src/github.com/Jeffail/gabs/gabs.go:447: encoder.SetEscapeHTML undefined (type *json.Encoder has no field or method SetEscapeHTML)

@Jeffail
Copy link
Owner

Jeffail commented Sep 12, 2017

Hey @Rajkumrjsn, I can pull it fine on my machine. Which version of Go are you building with?

@pjebs
Copy link

pjebs commented Oct 13, 2017

I noticed that /jeffail got changed to /Jeffail which is breaking all my package imports.
When did this happen?

@cliffano
Copy link
Contributor

@Jeffail It failed only on golang 1.6 for me. Fine on 1.7 all the way to tip.

@KinoLien
Copy link

Version 1.2.1 failed too, maybe these methods exist on version >= 1.7 ?

@pjebs
Copy link

pjebs commented Dec 31, 2017

Go Dep: dep init produces these results when I run dep init on a project with old dependency:

...
safe write of manifest and lock: error while writing out vendor tree: failed to write dep tree: failed to export github.com/jeffail/gabs: : exit status 128

These discussions all say it's due to case-sensitive changes to repo name:
golang/dep#1072
golang/dep#797
golang/dep#1199

@Jeffail
Copy link
Owner

Jeffail commented Dec 31, 2017

Hey @pjebs I think unfortunately you're forced at this stage to rewrite the imports as Jeffail/gabs. I've had to rewrite them already in most of my projects since I used to prefer the lowercase but came across similar issues that seemed to never end.

@Jeffail Jeffail closed this as completed Jul 2, 2018
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

No branches or pull requests

5 participants