Skip to content

Commit

Permalink
Upgrade scrypt library.
Browse files Browse the repository at this point in the history
  • Loading branch information
@mikehearn
mikehearn committed May 21, 2014
1 parent a07a810 commit 028a1cc
Showing 1 changed file with 1 addition and 6 deletions.
7 changes: 1 addition & 6 deletions core/pom.xml
View file
Original file line number Diff line number Diff line change
Expand Up @@ -104,11 +104,6 @@
<target>1.6</target>
</configuration>
</plugin>
<!-- Surefire plugin specified for Maven2 compatibility -->
<plugin>
<artifactId>maven-surefire-plugin</artifactId>
<version>2.9</version>
</plugin>

<plugin>
<groupId>org.apache.maven.plugins</groupId>
Expand Down Expand Up @@ -269,7 +264,7 @@
<dependency>
<groupId>com.lambdaworks</groupId>
<artifactId>scrypt</artifactId>
<version>1.3.3</version>
<version>1.4.0</version>
</dependency>

<!-- Add in to test/use Postgres blockstore -->
Expand Down

1 comment on commit 028a1cc

@schildbach
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can't help this commit feels half-baked to me.

  1. Why did you update scrypt? It should be noted in the commit comment imho.
  2. Why did you remove the surefire version fixing and what does it have to do with scrypt?
  3. The scrypt checksum wasn't updated.

Please sign in to comment.