Skip to content

Conversation

@laijs
Copy link
Contributor

@laijs laijs commented Mar 22, 2016

go get https://github.com/pquerna/ffjson
package https:/github.com/pquerna/ffjson: "https://" not allowed in
import path

so we remove the "https://"

Signed-off-by: Lai Jiangshan [email protected]

go get https://github.com/pquerna/ffjson
package https:/github.com/pquerna/ffjson: "https://" not allowed in
import path

so we remove the "https://"

Signed-off-by: Lai Jiangshan <[email protected]>
@wking
Copy link
Contributor

wking commented Mar 22, 2016

On Mon, Mar 21, 2016 at 07:25:25PM -0700, Lai Jiangshan wrote:

go get https://github.com/pquerna/ffjson
package https:/github.com/pquerna/ffjson: "https://" not allowed in
import path

This is also covered by a79fdef (Makefile: add target for tools,
2016-03-17, #349), where it's a drive-by. I expect it doesn't matter
particularly much which PR lands the fix, but the sooner we land one
of them the less likely we'll be to get another duplicate ;).

@laijs
Copy link
Contributor Author

laijs commented Mar 23, 2016

thanks, close it

@laijs laijs closed this Mar 23, 2016
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.

2 participants