go: explicit library version

.github/workflows/ci.yml

@@ -323,6 +323,12 @@ jobs:
         run: go install github.com/golangci/golangci-lint/cmd/golangci-lint@v1.49.0
       - run: make lint
       - run: make test
+      - name: Check library version
+        if: |
+          !github.event.pull_request.head.repo.fork &&
+          startsWith(github.ref, 'refs/tags/go/mcap/v')
+        working-directory: go/cli/check-tag
+        run: make build && bin/check-tag
 
   go-release-cli:
     permissions:

go/cli/check-tag/Makefile

@@ -0,0 +1,4 @@
+.PHONY: build
+build: 
+	mkdir -p bin
+	go build -o bin/check-tag

go/cli/check-tag/go.mod

@@ -0,0 +1,14 @@
+module github.com/foxglove/mcap/go/cli/check-tag
+
+go 1.18
+
+require github.com/foxglove/mcap/go/mcap v0.0.0-20221010221948-8b72814b2c50
+
+require (
+	github.com/davecgh/go-spew v1.1.0 // indirect
+	github.com/klauspost/compress v1.14.1 // indirect
+	github.com/pierrec/lz4/v4 v4.1.12 // indirect
+	github.com/pmezard/go-difflib v1.0.0 // indirect
+	github.com/stretchr/testify v1.7.0 // indirect
+	gopkg.in/yaml.v3 v3.0.0-20200313102051-9f266ea9e77c // indirect
+)

go/cli/check-tag/go.sum

@@ -0,0 +1,17 @@
+github.com/davecgh/go-spew v1.1.0 h1:ZDRjVQ15GmhC3fiQ8ni8+OwkZQO4DARzQgrnXU1Liz8=
+github.com/davecgh/go-spew v1.1.0/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38=
+github.com/foxglove/mcap/go/mcap v0.0.0-20221010221948-8b72814b2c50 h1:JnsYRxGWtsEJQGRfiTW+mstZA97FbKVc9h8PezcDMCo=
+github.com/foxglove/mcap/go/mcap v0.0.0-20221010221948-8b72814b2c50/go.mod h1:LMCS7BVPyHRb7xRXqYuKfM7dUq+MSE7H0dvaYsAQXnc=
+github.com/klauspost/compress v1.14.1 h1:hLQYb23E8/fO+1u53d02A97a8UnsddcvYzq4ERRU4ds=
+github.com/klauspost/compress v1.14.1/go.mod h1:/3/Vjq9QcHkK5uEr5lBEmyoZ1iFhe47etQ6QUkpK6sk=
+github.com/pierrec/lz4/v4 v4.1.12 h1:44l88ehTZAUGW4VlO1QC4zkilL99M6Y9MXNwEs0uzP8=
+github.com/pierrec/lz4/v4 v4.1.12/go.mod h1:gZWDp/Ze/IJXGXf23ltt2EXimqmTUXEy0GFuRQyBid4=
+github.com/pmezard/go-difflib v1.0.0 h1:4DBwDE0NGyQoBHbLQYPwSUPoCMWR5BEzIk/f1lZbAQM=
+github.com/pmezard/go-difflib v1.0.0/go.mod h1:iKH77koFhYxTK1pcRnkKkqfTogsbg7gZNVY4sRDYZ/4=
+github.com/stretchr/objx v0.1.0/go.mod h1:HFkY916IF+rwdDfMAkV7OtwuqBVzrE8GR6GFx+wExME=
+github.com/stretchr/testify v1.7.0 h1:nwc3DEeHmmLAfoZucVR881uASk0Mfjw8xYJ99tb5CcY=
+github.com/stretchr/testify v1.7.0/go.mod h1:6Fq8oRcR53rry900zMqJjRRixrwX3KX962/h/Wwjteg=
+gopkg.in/check.v1 v0.0.0-20161208181325-20d25e280405 h1:yhCVgyC4o1eVCa2tZl7eS0r+SDo693bJlVdllGtEeKM=
+gopkg.in/check.v1 v0.0.0-20161208181325-20d25e280405/go.mod h1:Co6ibVJAznAaIkqp8huTwlJQCZ016jof/cbN4VW5Yz0=
+gopkg.in/yaml.v3 v3.0.0-20200313102051-9f266ea9e77c h1:dUUwHk2QECo/6vqA44rthZ8ie2QXMNeKRTHCNY2nXvo=
+gopkg.in/yaml.v3 v3.0.0-20200313102051-9f266ea9e77c/go.mod h1:K4uyk7z7BCEPqu6E+C64Yfv1cQ7kz7rIZviUmN+EgEM=

go/cli/check-tag/main.go

@@ -0,0 +1,38 @@
+/// Utility for checking that the current git tag matches the internal MCAP library version string.
+package main
+
+import (
+	"fmt"
+	"os"
+	"os/exec"
+	"strings"
+
+	"github.com/foxglove/mcap/go/mcap"
+)
+
+func getGitTags() ([]string, error) {
+	res, err := exec.Command("git", "tag", "--points-at", "HEAD").Output()
+	return strings.Split(string(res), "\n"), err
+}
+
+func main() {
+	tags, err := getGitTags()
+	if err != nil {
+		fmt.Printf("Error getting git tags: %e\n", err)
+		os.Exit(1)
+	}
+	expected := fmt.Sprintf("go/mcap/%s", mcap.Version)
+	found := false
+	for _, tag := range tags {
+		if expected == tag {
+			found = true
+			break
+		}
+	}
+	if !found {
+		fmt.Println("Did not find git tag for library version, expected", expected, "found", tags)
+		os.Exit(1)
+	} else {
+		fmt.Println("Success")
+	}
+}

go/cli/mcap/cmd/version.go

@@ -3,22 +3,27 @@ package cmd
 import (
 	"fmt"
 
+	"github.com/foxglove/mcap/go/mcap"
 	"github.com/spf13/cobra"
 )
 
 var Version string
+var printLibraryVersion bool
 
 // versionCmd represents the version command
-func versionCmd(version string) *cobra.Command {
-	return &cobra.Command{
-		Use:   "version",
-		Short: "Output version information",
-		Run: func(cmd *cobra.Command, args []string) {
+var versionCmd = &cobra.Command{
+	Use:   "version",
+	Short: "Output version information",
+	Run: func(cmd *cobra.Command, args []string) {
+		if printLibraryVersion {
+			fmt.Println(mcap.Version)
+		} else {
 			fmt.Println(Version)
-		},
-	}
+		}
+	},
 }
 
 func init() {
-	rootCmd.AddCommand(versionCmd(Version))
+	versionCmd.PersistentFlags().BoolVarP(&printLibraryVersion, "library", "l", false, "print MCAP library version instead of CLI version")
+	rootCmd.AddCommand(versionCmd)
 }

go/go.work

@@ -2,6 +2,7 @@ go 1.18
 
 use (
 	./cli/mcap
+	./cli/check-tag
 	./conformance/test-streamed-read-conformance
 	./conformance/test-streamed-write-conformance
 	./mcap

go/mcap/mcap.go

@@ -4,7 +4,6 @@ import (
 	"errors"
 	"fmt"
 	"math"
-	"runtime/debug"
 )
 
 // Magic is the magic number for an MCAP file.
@@ -29,14 +28,6 @@ func (c CompressionFormat) String() string {
 	return string(c)
 }
 
-func Version() string {
-	info, ok := debug.ReadBuildInfo()
-	if !ok {
-		return "unknown"
-	}
-	return info.Main.Version
-}
-
 const (
 	OpReserved        OpCode = 0x00
 	OpHeader          OpCode = 0x01

go/mcap/version.go

@@ -0,0 +1,4 @@
+package mcap
+
+// Version of the MCAP library.
+var Version = "v0.1.0"

go/mcap/writer.go

@@ -52,8 +52,8 @@ type Writer struct {
 func (w *Writer) WriteHeader(header *Header) error {
 	var library string
 	if !w.opts.OverrideLibrary {
-		library = fmt.Sprintf("mcap go #%s", Version())
-		if header.Library != "" {
+		library = fmt.Sprintf("mcap go #%s", Version)
+		if header.Library != "" && header.Library != library {
 			library += "; " + header.Library
 		}
 	} else {

go/mcap/writer_test.go

@@ -13,10 +13,11 @@ import (
 func TestMCAPReadWrite(t *testing.T) {
 	t.Run("test header", func(t *testing.T) {
 		buf := &bytes.Buffer{}
-		w, err := NewWriter(buf, &WriterOptions{Compression: CompressionZSTD})
+		w, err := NewWriter(buf, &WriterOptions{Compression: CompressionZSTD, OverrideLibrary: true})
 		assert.Nil(t, err)
 		err = w.WriteHeader(&Header{
 			Profile: "ros1",
+			Library: "libfoo v0",
 		})
 		assert.Nil(t, err)
 		lexer, err := NewLexer(buf)
@@ -30,7 +31,7 @@ func TestMCAPReadWrite(t *testing.T) {
 		assert.Equal(t, "ros1", profile)
 		library, _, err := readPrefixedString(record, offset)
 		assert.Nil(t, err)
-		assert.Equal(t, "mcap go #", library)
+		assert.Equal(t, "libfoo v0", library)
 		assert.Equal(t, TokenHeader, tokenType)
 	})
 	t.Run("zero-valued schema IDs permitted", func(t *testing.T) {