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

Enable vendor experiment earlier in travis script. #247

Merged
merged 1 commit into from
May 18, 2016

Conversation

jrick
Copy link
Member

@jrick jrick commented May 18, 2016

This ensures that the vendor experiment is enabled when building not
just dcrwallet, but glide itself as well (glide vendors dependencies
and checks them into its git repo).

While here, switch from using 'go get $(glide novendor)' to 'go
install . ./cmd/...' to build the project and all related commands.
This won't hide errors when glide fails to restore a required
dependency or the vendor experiment is disabled.

This ensures that the vendor experiment is enabled when building not
just dcrwallet, but glide itself as well (glide vendors dependencies
and checks them into its git repo).

While here, switch from using 'go get $(glide novendor)' to 'go
install . ./cmd/...' to build the project and all related commands.
This won't hide errors when glide fails to restore a required
dependency or the vendor experiment is disabled.
@jcvernaleo
Copy link
Member

Looks good.
OK

@alexlyp
Copy link
Member

alexlyp commented May 18, 2016

kk i'll get this in then rerun

@alexlyp alexlyp merged commit d5c9a96 into decred:master May 18, 2016
@jcvernaleo jcvernaleo added this to the v0.1.4 milestone Jun 6, 2016
beansgum pushed a commit to beansgum/dcrwallet that referenced this pull request Jun 10, 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