-
Notifications
You must be signed in to change notification settings - Fork 9.6k
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
New provider arukas #11171
New provider arukas #11171
Conversation
b1a7c01
to
7d772b1
Compare
7d772b1
to
9f5dae3
Compare
rebased. |
a2e9f00
to
32695f9
Compare
2641045
to
bb02258
Compare
bb02258
to
9ae1d66
Compare
Thanks for this - @jbardin has checked out the vendor file and is happy with it Pulling it locally to run the tests again Thanks Paul |
Test results post merge conflicts:
|
* Add a Arukas provider * Add dependencies for the Arukas provider * Add documents for the Arukas
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
==UPDATED==
By updating upstream, dependency library
gopkg.in/alecthomas/kingpin.v2
is no longer used, so the following problem is solved.And updating the ignore section of
vendor/vendor.json
is no longer necessary.This PR was recreated to solve the problem occurring at #10862.
Problems
This provider includes a dependency that at time of writing requires a *nix system, and will not build on Windows.
Cause
The cause of this case is probably in
govendor
orgopkg.in/alecthomas/kingpin.v2
package.First of all, the direct cause of this case was that
vendor/gopkg.in/alecthomas/kingpin.v2/guesswidth.go
file necessary for building on Windows was not included in the vendor directory.Next, it is about why this file was not included in the vendor directory, but this seems to be caused by the existing "appengine" in the ignore section of vendor.json as follows.
https://github.com/hashicorp/terraform/blob/v0.8.4/vendor/vendor.json#L3
The build tag of the target file and the fallback file is described as follows.
https://github.com/alecthomas/kingpin/blob/v2.2.3/guesswidth.go#L1
// +build appengine !linux,!freebsd,!darwin,!dragonfly,!netbsd,!openbsd
https://github.com/alecthomas/kingpin/blob/v2.2.3/guesswidth_unix.go#L1
// +build !appengine,linux freebsd darwin dragonfly netbsd openbsd
On the other hand, as discussed in this Issue, in govendor it seems that if the build tag matches even one of the ignore in vendor.json, it does not include the target file under the vendor directory.
In this case, since "appengine" is included in the ignore of vendor.json,
guesswidth.go
will not be copied under the vendor directory even if you executegovendor add or fetch
.Solution
In this case, the size of the vendor directory will become large.
And there may be other problems as well.
What do you think about this method?
Looking forward to your feedback!