pants: Rename {lib,}thrift-0.5.0 to {lib,}thrift

Summary: Problem When we change the version of thrift, we don't want to have to rename the lib, which entails changing many BUILD files. Solution Since we only have one thrift, we can drop the version-specific suffix. JIRA Issues: CSL-5593 TBR=true NO_USER_HOOK=1 Differential Revision: https://phabricator.twitter.biz/D149423
twitter · Mar 22, 2018 · 12dcacf · 12dcacf
1 parent ab44378
commit 12dcacf
Show file tree

Hide file tree

Showing 9 changed files with 10 additions and 10 deletions.
diff --git a/examples/java-thrift-server/thrift-example-server/src/main/java/BUILD b/examples/java-thrift-server/thrift-example-server/src/main/java/BUILD
@@ -4,7 +4,7 @@ java_library(
     dependencies = [
         "3rdparty/jvm/com/google/inject:guice",
         "3rdparty/jvm/javax/inject:javax.inject",
-        "3rdparty/jvm/org/apache/thrift:libthrift-0.5.0",
+        "3rdparty/jvm/org/apache/thrift:libthrift",
         "3rdparty/jvm/org/slf4j:slf4j-api",
         "finagle/finagle-core/src/main/scala",
         "finagle/finagle-http/src/main/scala",

diff --git a/examples/java-thrift-server/thrift-example-server/src/test/java/BUILD b/examples/java-thrift-server/thrift-example-server/src/test/java/BUILD
@@ -5,7 +5,7 @@ junit_tests(
         "3rdparty/jvm/com/google/guava",
         "3rdparty/jvm/com/google/inject:guice",
         "3rdparty/jvm/junit",
-        "3rdparty/jvm/org/apache/thrift:libthrift-0.5.0",
+        "3rdparty/jvm/org/apache/thrift:libthrift",
         "3rdparty/jvm/org/scalatest",
         "3rdparty/jvm/org/slf4j:slf4j-api",
         "3rdparty/jvm/org/slf4j:slf4j-simple",

diff --git a/http/src/main/scala/BUILD b/http/src/main/scala/BUILD
@@ -24,7 +24,7 @@ scala_library(
         "3rdparty/jvm/io/netty",
         "3rdparty/jvm/javax/inject:javax.inject",
         "3rdparty/jvm/net/codingwell:scala-guice",
-        "3rdparty/jvm/org/apache/thrift:libthrift-0.5.0",
+        "3rdparty/jvm/org/apache/thrift:libthrift",
         "3rdparty/jvm/org/scala-lang/modules:scala-parser-combinators",
         "3rdparty/jvm/org/slf4j:jcl-over-slf4j",
         "3rdparty/jvm/org/slf4j:jul-to-slf4j",

diff --git a/http/src/test/scala/BUILD b/http/src/test/scala/BUILD
@@ -23,7 +23,7 @@ junit_tests(
         "3rdparty/jvm/javax/inject:javax.inject",
         "3rdparty/jvm/joda-time",
         "3rdparty/jvm/net/codingwell:scala-guice",
-        "3rdparty/jvm/org/apache/thrift:libthrift-0.5.0",
+        "3rdparty/jvm/org/apache/thrift:libthrift",
         "3rdparty/jvm/org/scala-lang/modules:scala-xml",
         "3rdparty/jvm/org/scalatest",
         "3rdparty/jvm/org/slf4j:slf4j-api",

diff --git a/inject/inject-server/src/test/scala/BUILD b/inject/inject-server/src/test/scala/BUILD
@@ -12,7 +12,7 @@ junit_tests(
         "3rdparty/jvm/com/google/inject/extensions:guice-testlib",
         "3rdparty/jvm/commons-lang",
         "3rdparty/jvm/net/codingwell:scala-guice",
-        "3rdparty/jvm/org/apache/thrift:libthrift-0.5.0",
+        "3rdparty/jvm/org/apache/thrift:libthrift",
         "3rdparty/jvm/org/scala-lang/modules:scala-parser-combinators",
         "3rdparty/jvm/org/scalatest",
         "3rdparty/jvm/org/slf4j:slf4j-api",
@@ -53,7 +53,7 @@ scala_library(
         "3rdparty/jvm/com/google/inject/extensions:guice-testlib",
         "3rdparty/jvm/commons-lang",
         "3rdparty/jvm/net/codingwell:scala-guice",
-        "3rdparty/jvm/org/apache/thrift:libthrift-0.5.0",
+        "3rdparty/jvm/org/apache/thrift:libthrift",
         "3rdparty/jvm/org/scalatest",
         "3rdparty/jvm/org/slf4j:slf4j-api",
         "3rdparty/jvm/org/specs2",

diff --git a/jackson/src/main/scala/BUILD b/jackson/src/main/scala/BUILD
@@ -25,7 +25,7 @@ scala_library(
         "3rdparty/jvm/javax/inject:javax.inject",
         "3rdparty/jvm/joda-time",
         "3rdparty/jvm/net/codingwell:scala-guice",
-        "3rdparty/jvm/org/apache/thrift:libthrift-0.5.0",
+        "3rdparty/jvm/org/apache/thrift:libthrift",
         "3rdparty/jvm/org/scala-lang:scala-compiler",
         "3rdparty/jvm/org/scala-lang:scalap",
         "3rdparty/jvm/org/slf4j:slf4j-api",

diff --git a/jackson/src/test/scala/BUILD b/jackson/src/test/scala/BUILD
@@ -21,7 +21,7 @@ junit_tests(
         "3rdparty/jvm/joda-time",
         "3rdparty/jvm/junit",
         "3rdparty/jvm/net/codingwell:scala-guice",
-        "3rdparty/jvm/org/apache/thrift:libthrift-0.5.0",
+        "3rdparty/jvm/org/apache/thrift:libthrift",
         "3rdparty/jvm/org/mockito:mockito-all",
         "3rdparty/jvm/org/scalacheck",
         "3rdparty/jvm/org/scalatest",

diff --git a/thrift/src/main/scala/BUILD b/thrift/src/main/scala/BUILD
@@ -15,7 +15,7 @@ scala_library(
         "3rdparty/jvm/javax/inject:javax.inject",
         "3rdparty/jvm/joda-time",
         "3rdparty/jvm/net/codingwell:scala-guice",
-        "3rdparty/jvm/org/apache/thrift:libthrift-0.5.0",
+        "3rdparty/jvm/org/apache/thrift:libthrift",
         "3rdparty/jvm/org/slf4j:jcl-over-slf4j",
         "3rdparty/jvm/org/slf4j:jul-to-slf4j",
         "3rdparty/jvm/org/slf4j:log4j-over-slf4j",

diff --git a/thrift/src/test/java/BUILD b/thrift/src/test/java/BUILD
@@ -11,7 +11,7 @@ junit_tests(
         "3rdparty/jvm/com/google/inject/extensions:guice-testlib",
         "3rdparty/jvm/javax/inject:javax.inject",
         "3rdparty/jvm/net/codingwell:scala-guice",
-        "3rdparty/jvm/org/apache/thrift:libthrift-0.5.0",
+        "3rdparty/jvm/org/apache/thrift:libthrift",
         "3rdparty/jvm/org/scalatest",
         "3rdparty/jvm/org/slf4j:slf4j-api",
         "finagle/finagle-core/src/main/scala",