GoogleCloudPlatform / healthcare-data-harmonization

This is an engine that converts data of one structure to another, based on a configuration file which describes how. There is an accompanying syntax to make writing mappings easier and more robust.
https://cloud.google.com/solutions/healthcare-life-sciences
Apache License 2.0
213 stars 66 forks source link

Building wstl 1 is broken #79

Open hungrymonkey opened 1 month ago

hungrymonkey commented 1 month ago
healthcare-data-harmonization/wstl1> ./build_all.sh 
Entering /home/doof/gitrepo/healthcare-data-harmonization/wstl1/mapping_engine/main
# github.com/GoogleCloudPlatform/healthcare-data-harmonization/mapping_engine/proto
../proto/data_harmonization.pb.go:164:53: predeclared any requires go1.18 or later (-lang was set to go1.14; check go.mod)
../proto/data_harmonization.pb.go:191:69: predeclared any requires go1.18 or later (-lang was set to go1.14; check go.mod)
../proto/harmonization.pb.go:442:48: predeclared any requires go1.18 or later (-lang was set to go1.14; check go.mod)
../proto/harmonization.pb.go:473:64: predeclared any requires go1.18 or later (-lang was set to go1.14; check go.mod)
../proto/harmonization.pb.go:485:64: predeclared any requires go1.18 or later (-lang was set to go1.14; check go.mod)
../proto/harmonization.pb.go:497:64: predeclared any requires go1.18 or later (-lang was set to go1.14; check go.mod)
../proto/http.pb.go:484:39: predeclared any requires go1.18 or later (-lang was set to go1.14; check go.mod)
../proto/library.pb.go:261:42: predeclared any requires go1.18 or later (-lang was set to go1.14; check go.mod)
../proto/mapping.pb.go:1001:42: predeclared any requires go1.18 or later (-lang was set to go1.14; check go.mod)
../proto/unit_config.pb.go:264:46: predeclared any requires go1.18 or later (-lang was set to go1.14; check go.mod)
../proto/harmonization.pb.go:497:64: too many errors

We can fix the issue by bumping the go.mod from 1.14 to 1.18

diff --git a/wstl1/mapping_engine/proto/go.mod b/wstl1/mapping_engine/proto/go.mod
index ecd996d..ce74632 100644
--- a/wstl1/mapping_engine/proto/go.mod
+++ b/wstl1/mapping_engine/proto/go.mod
@@ -1,8 +1,5 @@
 module github.com/GoogleCloudPlatform/healthcare-data-harmonization/mapping_engine/proto

-go 1.14
+go 1.18

-require (
-    github.com/golang/protobuf v1.4.3
-    google.golang.org/protobuf v1.25.0
-)
+require google.golang.org/protobuf v1.25.0
hungrymonkey commented 1 month ago
 go version
go version go1.23.1 linux/amd64