Skip to content

cmd/continuity: switch to google.golang.org/protobuf/proto#260

Merged
AkihiroSuda merged 1 commit into
containerd:mainfrom
thaJeztah:switch_proto
Oct 28, 2024
Merged

cmd/continuity: switch to google.golang.org/protobuf/proto#260
AkihiroSuda merged 1 commit into
containerd:mainfrom
thaJeztah:switch_proto

Conversation

@thaJeztah

Copy link
Copy Markdown
Member

The github.com/golang/protobuf/proto module is deprecated, and the main module already switched to google.golang.org/protobuf/proto in commit 74a0169. This patch updates the cmd/continuity module to use the new module as well.

Before this patch:

$ cd cmd/continuity
$ go install
$ continuity build ./continuityfs > manifest.pb
$ continuity dump ./manifest.pb
resource: <
  path: "/fuse.go"
  uid: 501
  gid: 20
  mode: 420
  size: 7464
  digest: "sha256:3d86240dcc3ea8ec7ae85d6c4e0bf0be1385b5d1e38564dcec72a69b51fcf919"
>
resource: <
  path: "/provider.go"
  uid: 501
  gid: 20
  mode: 420
  size: 1728
  digest: "sha256:8e3da0fd4a8d5e454ff027d4948d4b116ddaeb5ccdd0e10af806a2ecf2599710"
>

With this patch:

$ cd cmd/continuity
$ go install
$ continuity build ./continuityfs > manifest.pb
$ continuity dump ./manifest.pb
resource:  {
  path:  "/fuse.go"
  uid:  501
  gid:  20
  mode:  420
  size:  7464
  digest:  "sha256:3d86240dcc3ea8ec7ae85d6c4e0bf0be1385b5d1e38564dcec72a69b51fcf919"
}
resource:  {
  path:  "/provider.go"
  uid:  501
  gid:  20
  mode:  420
  size:  1728
  digest:  "sha256:8e3da0fd4a8d5e454ff027d4948d4b116ddaeb5ccdd0e10af806a2ecf2599710"
}

The github.com/golang/protobuf/proto module is deprecated, and the main
module already switched to google.golang.org/protobuf/proto in commit
74a0169. This patch updates the cmd/continuity
module to use the new module as well.

Before this patch:

    $ cd cmd/continuity
    $ go install
    $ continuity build ./continuityfs > manifest.pb
    $ continuity dump ./manifest.pb
    resource: <
      path: "/fuse.go"
      uid: 501
      gid: 20
      mode: 420
      size: 7464
      digest: "sha256:3d86240dcc3ea8ec7ae85d6c4e0bf0be1385b5d1e38564dcec72a69b51fcf919"
    >
    resource: <
      path: "/provider.go"
      uid: 501
      gid: 20
      mode: 420
      size: 1728
      digest: "sha256:8e3da0fd4a8d5e454ff027d4948d4b116ddaeb5ccdd0e10af806a2ecf2599710"
    >

With this patch:

    $ cd cmd/continuity
    $ go install
    $ continuity build ./continuityfs > manifest.pb
    $ continuity dump ./manifest.pb
    resource:  {
      path:  "/fuse.go"
      uid:  501
      gid:  20
      mode:  420
      size:  7464
      digest:  "sha256:3d86240dcc3ea8ec7ae85d6c4e0bf0be1385b5d1e38564dcec72a69b51fcf919"
    }
    resource:  {
      path:  "/provider.go"
      uid:  501
      gid:  20
      mode:  420
      size:  1728
      digest:  "sha256:8e3da0fd4a8d5e454ff027d4948d4b116ddaeb5ccdd0e10af806a2ecf2599710"
    }

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@AkihiroSuda AkihiroSuda merged commit c156753 into containerd:main Oct 28, 2024
@thaJeztah thaJeztah deleted the switch_proto branch October 28, 2024 12:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants