Skip to content

Commit

Permalink
Don't check picard.jar in to lib, but instead declare as dependency i…
Browse files Browse the repository at this point in the history
…n gradle.

I'm checking this for posterity, but I won't merge because until broadinstitute/picard#1919 is merged and we upgrade to a version including this change, we need to declare all the transitive dependencies we use in our code.
  • Loading branch information
alecw committed Nov 30, 2023
1 parent 252ccb0 commit 8754ddd
Show file tree
Hide file tree
Showing 2 changed files with 15 additions and 0 deletions.
15 changes: 15 additions & 0 deletions dropseq/build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,21 @@ dependencies {

// From https://stackoverflow.com/questions/54166069/how-do-you-add-local-jar-file-dependency-to-build-gradle-kt-file
implementation(fileTree(mapOf("dir" to "../lib", "include" to listOf("*.jar"))))
implementation("com.github.broadinstitute:picard:3.1.0")
testFixturesImplementation("com.github.broadinstitute:picard:3.1.0")
// The following picard transitive dependencies can be removed when we upgrade to picard with this PR included:
// https://github.com/broadinstitute/picard/pull/1919
implementation("com.github.samtools:htsjdk:3.0.5")
testFixturesImplementation("com.github.samtools:htsjdk:3.0.5")
implementation("org.broadinstitute:barclay:5.0.0")
implementation("org.apache.commons:commons-lang3:3.12.0")
implementation("commons-io:commons-io:2.11.0")
implementation("com.google.guava:guava:32.1.1-jre")
implementation("org.apache.commons:commons-math3:3.6.1")
implementation("com.google.code.gson:gson:2.10.1")
implementation("org.apache.commons:commons-collections4:4.4")
// Might be good to eliminate this dependency
testImplementation("org.freemarker:freemarker:2.3.30")

// testFixtures call testng and use dependent jar classes
testFixturesImplementation(fileTree(mapOf("dir" to "../lib", "include" to listOf("*.jar"))))
Expand Down
Binary file removed lib/picard-3.1.0.jar
Binary file not shown.

0 comments on commit 8754ddd

Please sign in to comment.