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

[tests-only] Mention go minimal version #2035

Merged
merged 1 commit into from
Sep 29, 2021
Merged

Conversation

michielbdejong
Copy link
Contributor

If you try to build reva with the go version 1.13 which ships standard with Ubuntu, you'll get this error during make build:

build github.com/cs3org/reva/cmd/revad: cannot load embed: malformed module path "embed": missing dot in first path element

The use of embed requires go >= 1.16, see https://blog.jetbrains.com/go/2021/06/09/how-to-use-go-embed-in-go-1-16/

If you try to build reva with the go version 1.13 which ships standard with Ubuntu, you'll get this error during `make build`:
```
build github.com/cs3org/reva/cmd/revad: cannot load embed: malformed module path "embed": missing dot in first path element
```

The use of embed requires go >= 1.16, see https://blog.jetbrains.com/go/2021/06/09/how-to-use-go-embed-in-go-1-16/
@update-docs
Copy link

update-docs bot commented Sep 3, 2021

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@labkode labkode changed the title Mention go minimal version [tests-only] Mention go minimal version Sep 16, 2021
@ishank011 ishank011 merged commit ceb92c3 into master Sep 29, 2021
@ishank011 ishank011 deleted the mention-go-minimal-version branch September 30, 2021 07:08
glpatcern pushed a commit to glpatcern/reva that referenced this pull request Nov 12, 2021
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