can't build about terraform-provider-elasticsearch HOT 7 CLOSED

Hello, I'll need a bit more context to help debug.

The steps to build this project are documented in the readme. (The travis build script is also a good resource.)

Can you list all the commands you used to generate that error?

from terraform-provider-elasticsearch.

Hey @chicagobuss took me a second, but I think you found a bug in the README 😀, instead of being Golang 1.7 it should be >= 1.7. Locally I'm on go 1.9 (although, yes a mac).

If you trying building in a container with 1.9 and glide does that work for you?

I'd definitely be open to migrating this package to go dep as well (and adding some CI) - this was thrown together in a bit of a rush, so totally can use some love!

from terraform-provider-elasticsearch.

Ha, I went through a lot of effort just to get on 1.7 because I assumed it was required. Will try again soon, thanks.

from terraform-provider-elasticsearch.

So in a bit of irony, I did try to convert to dep (https://github.com/phillbaker/terraform-provider-elasticsearch/tree/dep-not-glide), but I am seeing the same error you posted above when using dep. On https://travis-ci.org/phillbaker/terraform-provider-elasticsearch/jobs/314409199.

Using glide on travis on master seems to build fine 😕 . Other posts about this seem to indicate that it's a a dependency issue.

from terraform-provider-elasticsearch.

Ah, so it looks like dep strips the vendor directories of dependencies which is an issue for at least terraform, aws-sdk-go, see golang/dep#1075 (comment).

However, dep did merge support for govendor (which terraform uses) a couple of weeks ago, although it hasn't been included in a release yet, which I think solve the issue of recursive dependencies?

Until it's clear how dep support dependencies like terraform, seems better to stick with glide.

from terraform-provider-elasticsearch.

I am trying to build on mac but I am getting the following:

unexpected directory layout:
    import path: github.com/hashicorp/terraform/plugin
    root: /Users/username/.gvm/pkgsets/go1.10.1/global/src
    dir: /Users/username/.gvm/pkgsets/go1.10.1/global/src/github.com/phillbaker/terraform-provider-elasticsearch/vendor/github.com/hashicorp/terraform/plugin
    expand root: /Users/username/.gvm/pkgsets/go1.10.1/global/src
    expand dir: /Users/username/.gvm/pkgsets/go1.10.1/global/src/github.com/phillbaker/terraform-provider-elasticsearch/vendor/github.com/hashicorp/terraform/plugin
    separator: /

from terraform-provider-elasticsearch.

@phillbaker Thanks for your reply, sorry for the late reply.

I tried with the master updates and it worked, I can not reproduce the issue.

from terraform-provider-elasticsearch.