Skip to content

Update to use hard versions of dependencies#9

Closed
ryano144 wants to merge 2 commits intoaws:masterfrom
ryano144:patch-1
Closed

Update to use hard versions of dependencies#9
ryano144 wants to merge 2 commits intoaws:masterfrom
ryano144:patch-1

Conversation

@ryano144
Copy link

The open ended [1.10.5,) causes maven/ivy to go back to the central repo to look for the latest version every time the build runs. At least in SBT, this forces the latest version of the SDK to download every time it is released, which has caused problems for our build. It also makes dependency resolution take an unnecessarily long time. This update chooses the current latest version of the SDK.

ryano144 and others added 2 commits March 25, 2016 11:43
The open ended [1.10.5,) causes maven/ivy to go back to the central repo to look for the latest version every time the build runs.  At least in SBT, this forces the latest version of the SDK to download every time it is released, which has caused problems for our build.  It also makes dependency resolution take an unnecessarily long time.  This update chooses the current latest version of the SDK.
@torgeir
Copy link

torgeir commented Mar 30, 2016

Second this one, as we've hit several issues with NoClassDefFoundError errors in our lambdas due to the lax version ranges of the aws-lambda-java-events pom. While minor and patch releases generally should not break apis this way, these version ranges would also bump major versions which definitely would break apis.

Temporarily we exclude all dependencies we actually depend on from the aws-lambda-java-events module to lock down version numbers in explicit dependencies instead

<dependency>
  <groupId>com.amazonaws</groupId>
  <artifactId>aws-lambda-java-events</artifactId>
  <version>${aws.lambda.version}</version>
  <exclusions>
    <exclusion>
      <groupId>com.amazonaws</groupId>
      <artifactId>aws-java-sdk-s3</artifactId>
    </exclusion>
    <exclusion>
      <groupId>com.amazonaws</groupId>
      <artifactId>aws-java-sdk-sqs</artifactId>
    </exclusion>
  </exclusions>
</dependency>

@petehayes
Copy link

This can be closed as the latest version removes the dynamic version ranges.

@bmoffatt
Copy link
Contributor

617af84 65ef84f

@bmoffatt bmoffatt closed this May 23, 2016
smirnoal pushed a commit that referenced this pull request Mar 10, 2023
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.

4 participants

Comments