Skip to content

Commit 16e8d67

Browse files
authored
upgrade some dependencies and fix mac conformance tests (#89)
1 parent f00caa4 commit 16e8d67

File tree

9 files changed

+19
-16
lines changed

9 files changed

+19
-16
lines changed

build.gradle.kts

-1
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@ buildscript {
2424

2525
dependencies {
2626
classpath(libraries.kotlinPlugin)
27-
classpath(libraries.protobufPlugin)
2827
}
2928
}
3029

buildSrc/build.gradle.kts

+2-2
Original file line numberDiff line numberDiff line change
@@ -27,8 +27,8 @@ repositories {
2727
}
2828

2929
dependencies {
30-
implementation("com.diffplug.spotless:spotless-plugin-gradle:4.2.0")
31-
implementation("com.google.protobuf:protobuf-gradle-plugin:0.8.12")
30+
implementation("com.diffplug.spotless:spotless-plugin-gradle:5.5.1")
31+
implementation("com.google.protobuf:protobuf-gradle-plugin:0.8.13")
3232
implementation("io.codearte.gradle.nexus:gradle-nexus-staging-plugin:0.21.2")
3333
implementation("com.google.guava:guava:29.0-jre")
3434
implementation("ru.vyarus:gradle-animalsniffer-plugin:1.5.0")

buildSrc/src/main/kotlin/Dependencies.kt

+3-3
Original file line numberDiff line numberDiff line change
@@ -18,13 +18,13 @@ import com.toasttab.protokt.gradle.DEFAULT_PROTOBUF_VERSION
1818
object versions {
1919
const val arrow = "0.10.5"
2020
const val autoService = "1.0-rc7"
21-
const val grpc = "1.29.0"
22-
const val jackson = "2.11.0"
21+
const val grpc = "1.32.1"
22+
const val jackson = "2.11.3"
2323
const val kollection = "0.7"
2424
const val kotlin = "1.3.72"
2525
const val kotlinxCoroutines = "1.3.7"
2626
const val protobuf = DEFAULT_PROTOBUF_VERSION
27-
const val protobufPlugin = "0.8.12"
27+
const val protobufPlugin = "0.8.13"
2828
const val stringTemplate = "4.3"
2929

3030
// Test

buildSrc/src/main/kotlin/Lint.kt

+1-1
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ import org.gradle.kotlin.dsl.apply
1919
import org.gradle.kotlin.dsl.configure
2020

2121
fun Project.lint() {
22-
apply(plugin = "com.diffplug.gradle.spotless")
22+
apply(plugin = "com.diffplug.spotless")
2323

2424
configure<SpotlessExtension> {
2525
kotlin {
+1-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
distributionBase=GRADLE_USER_HOME
22
distributionPath=wrapper/dists
3-
distributionUrl=https\://services.gradle.org/distributions/gradle-6.5-bin.zip
3+
distributionUrl=https\://services.gradle.org/distributions/gradle-6.6.1-bin.zip
44
zipStoreBase=GRADLE_USER_HOME
55
zipStorePath=wrapper/dists

testing/conformance-driver/README.md

+7-5
Original file line numberDiff line numberDiff line change
@@ -11,9 +11,11 @@ Mac conformance tests:
1111
protobuf % ./configure
1212
protobuf % make clean
1313
protobuf % make
14-
protobuf % cd conformance/
14+
protobuf % cd conformance
1515
conformance % make
16-
conformance % cp .libs/conformance-test-runner ../../protokt/testing/conformance-driver/bin/darwin/conformance-test-runner
16+
conformance % cd ..
17+
protobuf % cp conformance/.libs/conformance-test-runner ../protokt/testing/conformance-driver/bin/darwin/conformance-test-runner
18+
protobuf % cp src/.libs/libprotobuf.24.dylib ../protokt/testing/conformance-driver/bin/darwin/.libs/libprotobuf.24.dylib
1719
```
1820

1921
Ubuntu conformance tests:
@@ -24,12 +26,12 @@ root@38f7a53696b9:/# cd tmp/
2426
root@38f7a53696b9:/tmp# ./configure
2527
root@38f7a53696b9:/tmp# make clean
2628
root@38f7a53696b9:/tmp# make
27-
root@38f7a53696b9:/tmp# cd conformance/
29+
root@38f7a53696b9:/tmp# cd conformance
2830
root@38f7a53696b9:/tmp/conformance# make
2931
root@38f7a53696b9:/tmp/conformance# exit
3032
protobuf % cp conformance/.libs/conformance-test-runner ../protokt/testing/conformance-driver/bin/ubuntu-16.04-x86_64/conformance-test-runner
3133
protobuf % cp src/.libs/libprotobuf.so.24 ../protokt/testing/conformance-driver/bin/ubuntu-16.04-x86_64/.libs/libprotobuf.so.24
3234
```
3335

34-
Note that the integer suffix on `libprotobuf.so` may change, and the need to
35-
copy it may disappear as it has for running the conformance tests on Mac.
36+
Note that the version numbers on `libprotobuf.so.xx` and `libprotobuf.xx.dylib`
37+
may change.
Binary file not shown.

testing/conformance-tests/src/test/kotlin/com/toasttab/protokt/conformance/ConformanceTest.kt

+3-3
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ private val command =
6363
"$baseCommand --enforce_recommended $failureList $conformanceDriver"
6464

6565
private val libPathOverride =
66-
pivotOs(
67-
emptyMap(),
68-
mapOf("LD_LIBRARY_PATH" to Paths.get(binDir, ".libs").toString())
66+
mapOf(
67+
pivotOs("DYLD_LIBRARY_PATH", "LD_LIBRARY_PATH") to
68+
Paths.get(binDir, ".libs").toString()
6969
)

testing/util/src/main/kotlin/com/toasttab/protokt/testing/util/ProcessUtils.kt

+2
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,8 @@ fun String.runCommand(
2929
workingDir: Path,
3030
env: Map<String, String> = emptyMap()
3131
): ProcessOutput {
32+
println("Executing $this in $workingDir with $env")
33+
3234
val proc =
3335
ProcessBuilder(*split("\\s".toRegex()).toTypedArray())
3436
.directory(workingDir.toFile())

0 commit comments

Comments
 (0)