Skip to content

Commit

Permalink
Update docs for protoc minimum installed version (#11006)
Browse files Browse the repository at this point in the history
* bumps `protoc` minimum version from 3.12 -> 3.15

* Minor: Fix link for Protoc installation

* minor: removes reference to 3.12
  • Loading branch information
jcsherin authored Jun 20, 2024
1 parent 1e7c38b commit 5bfc11b
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion datafusion/proto/CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -29,4 +29,4 @@ valid installation of [protoc] (see [installation instructions] for details).
```

[protoc]: https://github.com/protocolbuffers/protobuf#protocol-compiler-installation
[installation instructions]: https://datafusion.apache.org/contributor-guide/#protoc-installation
[installation instructions]: https://datafusion.apache.org/contributor-guide/getting_started.html#protoc-installation
4 changes: 2 additions & 2 deletions docs/source/contributor-guide/getting_started.md
Original file line number Diff line number Diff line change
Expand Up @@ -50,11 +50,11 @@ $ pacman -S protobuf
$ brew install protobuf
```

You will want to verify the version installed is `3.12` or greater, which introduced support for explicit [field presence](https://github.com/protocolbuffers/protobuf/blob/v3.12.0/docs/field_presence.md). Older versions may fail to compile.
You will want to verify the version installed is `3.15` or greater, which has support for explicit [field presence](https://github.com/protocolbuffers/protobuf/blob/v3.15.0/docs/field_presence.md). Older versions may fail to compile.

```shell
$ protoc --version
libprotoc 3.12.4
libprotoc 3.15.0
```

Alternatively a binary release can be downloaded from the [Release Page](https://github.com/protocolbuffers/protobuf/releases) or [built from source](https://github.com/protocolbuffers/protobuf/blob/main/src/README.md).
Expand Down

0 comments on commit 5bfc11b

Please sign in to comment.