-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
SDK is unusable with dep and case insensitive filesystem #1540
Comments
#1539 fix /models/apis/Greengrass dep case sensitivity |
Just hit this in Homebrew too in Homebrew/homebrew-core#18363. |
Thanks for posting this issue. Reviewing the change in the PR and repo it seems only the model folders Greengrass item is at fault. I'll investigate this more and correct this issue. It looks like the greengrass folder had it's casing changed some how. Will update this GitHub issue when a fix is available. Temporary fix will be correcting the casing locally, but need to fix this upstream as well so it doesn't happen again. |
I've posted a PR #1541 that should correct the issue. We'll need to make some internally changes to how the release is generated too. As apart of these changes I think when the SDK's models are generated it should fail if two models target the same service's |
…1541) Fixes the greengrass service model folder being duplicated as Greengrass. The duplicate folders occurred due to an upstream error internal SDK release that will be fixed separately. This change should correct the issues with dep and other tools failing because of the duplicate folder name. Fix #1540 Fix #1539
Thanks again for pointing this issue out and alerting us to it. The PR has corrected the issue, and will be apart of our next tagged release. |
Please fill out the sections below to help us address your issue.
Version of AWS SDK for Go?
1.40.49
Version of Go (
go version
)?1.9
What issue did you see?
I'm on Mac with a case insensitive filesystem and I cannot use
dep
tool to add/updateaws-sdk-go
dependency because of theGreengrass
andgreengrass
directories.Steps to reproduce
Try to setup a project with
dep
andaws-sdk-go
. Sorry, the project where I'm having this issue is not opensource.If you have have an runnable example, please include it.
The text was updated successfully, but these errors were encountered: