Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update protoc #2421

Open
wants to merge 2 commits into
base: release-1.1
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/workflows/install-rust-and-protoc/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,5 +28,5 @@ runs:
- name: Install protoc (protobuf)
uses: arduino/setup-protoc@v3
with:
version: "25.1"
version: "25.5"
repo-token: ${{ inputs.github-token }}
4 changes: 2 additions & 2 deletions .github/workflows/java-cd.yml
Original file line number Diff line number Diff line change
Expand Up @@ -105,7 +105,7 @@ jobs:
- name: Install protoc (protobuf)
uses: arduino/setup-protoc@v3
with:
version: "26.1"
version: "28.1"
repo-token: ${{ secrets.GITHUB_TOKEN }}

- name: Create secret key ring file
Expand Down Expand Up @@ -245,7 +245,7 @@ jobs:
- name: Install protoc (protobuf)
uses: arduino/setup-protoc@v3
with:
version: "26.1"
version: "28.1"
repo-token: ${{ secrets.GITHUB_TOKEN }}

- name: Start standalone Valkey server
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/java.yml
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,7 @@ jobs:
- name: Install protoc (protobuf)
uses: arduino/setup-protoc@v3
with:
version: "26.1"
version: "28.1"
repo-token: ${{ secrets.GITHUB_TOKEN }}

- name: Build java client
Expand Down Expand Up @@ -161,7 +161,7 @@ jobs:
- name: Install protoc (protobuf)
uses: arduino/setup-protoc@v3
with:
version: "26.1"
version: "28.1"
repo-token: ${{ secrets.GITHUB_TOKEN }}

- name: Install Java
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/pypi-cd.yml
Original file line number Diff line number Diff line change
Expand Up @@ -176,8 +176,8 @@ jobs:
echo "Running on unsupported architecture: $ARCH. Expected one of: ['x86_64', 'aarch64']"
exit 1
fi
curl -LO $PB_REL/download/v3.20.3/protoc-3.20.3-linux-${PROTOC_ARCH}.zip
unzip protoc-3.20.3-linux-${PROTOC_ARCH}.zip -d $HOME/.local
curl -LO $PB_REL/download/v4.25.5/protoc-4.25.5-linux-${PROTOC_ARCH}.zip
unzip protoc-4.25.5-linux-${PROTOC_ARCH}.zip -d $HOME/.local
export PATH="$PATH:$HOME/.local/bin"

- name: Build Python wheels (macos)
Expand Down
2 changes: 2 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,7 @@
#### Changes

* Java, Python: Bump protobuf (protoc) version ([#2421](https://github.com/valkey-io/valkey-glide/pull/2421))

#### Breaking Changes

#### Fixes
Expand Down
12 changes: 6 additions & 6 deletions java/DEVELOPER.md
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ The Valkey GLIDE Java wrapper consists of both Java and Rust code. Rust bindings
- git
- GCC
- pkg-config
- protoc (protobuf compiler) >= 26.1
- protoc (protobuf compiler) >= 28.1
- openssl
- openssl-dev
- rustup
Expand Down Expand Up @@ -64,17 +64,17 @@ Continue with **Install protobuf compiler** below.
To install protobuf for MacOS, run:
```bash
brew install protobuf
# Check that the protobuf compiler version 26.1 or higher is installed
# Check that the protobuf compiler version 28.1 or higher is installed
protoc --version
```

For the remaining systems, do the following:
```bash
PB_REL="https://github.com/protocolbuffers/protobuf/releases"
curl -LO $PB_REL/download/v26.1/protoc-26.1-linux-x86_64.zip
unzip protoc-26.1-linux-x86_64.zip -d $HOME/.local
curl -LO $PB_REL/download/v28.1/protoc-28.1-linux-x86_64.zip
unzip protoc-28.1-linux-x86_64.zip -d $HOME/.local
export PATH="$PATH:$HOME/.local/bin"
# Check that the protobuf compiler version 26.1 or higher is installed
# Check that the protobuf compiler version 28.1 or higher is installed
protoc --version
```

Expand Down Expand Up @@ -165,7 +165,7 @@ Some troubleshooting issues:
- Failed to find `cargo` after `rustup`.
- No Protobuf compiler (protoc) found.
- If build fails because of rust compiler fails, make sure submodules are updated using `git submodule update`.
- If protobuf 26.0 or earlier is detected, upgrade to the latest protobuf release.
- If protobuf 28.0 or earlier is detected, upgrade to the latest protobuf release.

## Running Examples App

Expand Down
10 changes: 5 additions & 5 deletions java/client/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ repositories {
}

dependencies {
implementation group: 'com.google.protobuf', name: 'protobuf-java', version: '4.27.1'
implementation group: 'com.google.protobuf', name: 'protobuf-java', version: '4.28.1'
implementation group: 'org.apache.commons', name: 'commons-lang3', version: '3.13.0'

implementation group: 'io.netty', name: 'netty-handler', version: '4.1.100.Final'
Expand All @@ -39,11 +39,11 @@ dependencies {

ext {
checkProtocVersion = { String output ->
// Line in format like: libprotoc 26.1
// Line in format like: libprotoc 28.1
int majorVersion = Integer.parseInt(output.split(" ")[1].split("\\.")[0].trim());
int minorVersion = Integer.parseInt(output.split(" ")[1].split("\\.")[1].trim());
if (majorVersion < 26 || (majorVersion == 26 && minorVersion < 1)) {
throw new GradleException("Protobuf compiler (protoc) version 26.1 or newer is required. Current version: $output");
if (majorVersion < 28) {
throw new GradleException("Protobuf compiler (protoc) version 28.0 or newer is required. Current version: $output");
}
return output.split(" ")[1]
}
Expand All @@ -62,7 +62,7 @@ tasks.register('protobuf', Exec) {
}
} catch (Exception e) {
if (e.getMessage().startsWith("A problem occurred starting process")) {
throw new GradleException("No Protobuf compiler (protoc) found. Protobuf compiler version 26.1 or newer is required.");
throw new GradleException("No Protobuf compiler (protoc) found. Protobuf compiler version 28.0 or newer is required.");
}
throw e
}
Expand Down
12 changes: 6 additions & 6 deletions python/DEVELOPER.md
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ Software Dependencies
- git
- GCC
- pkg-config
- protoc (protobuf compiler) >= v3.20.0
- protoc (protobuf compiler) >= v4.25.5
- openssl
- openssl-dev
- rustup
Expand All @@ -34,9 +34,9 @@ rustc --version
# Install protobuf compiler
PB_REL="https://github.com/protocolbuffers/protobuf/releases"
# For other arch type from x86 example below, the signature of the curl url should be protoc-<version>-<os>-<arch>.zip,
# e.g. protoc-3.20.3-linux-aarch_64.zip for ARM64.
curl -LO $PB_REL/download/v3.20.3/protoc-3.20.3-linux-x86_64.zip
unzip protoc-3.20.3-linux-x86_64.zip -d $HOME/.local
# e.g. protoc-4.25.5-linux-aarch_64.zip for ARM64.
curl -LO $PB_REL/download/v4.25.5/protoc-4.25.5-linux-x86_64.zip
unzip protoc-4.25.5-linux-x86_64.zip -d $HOME/.local
export PATH="$PATH:$HOME/.local/bin"
# Check that the protobuf compiler is installed
protoc --version
Expand All @@ -55,8 +55,8 @@ source "$HOME/.cargo/env"
rustc --version
# Install protobuf compiler
PB_REL="https://github.com/protocolbuffers/protobuf/releases"
curl -LO $PB_REL/download/v3.20.3/protoc-3.20.3-linux-x86_64.zip
unzip protoc-3.20.3-linux-x86_64.zip -d $HOME/.local
curl -LO $PB_REL/download/v4.25.5/protoc-4.25.5-linux-x86_64.zip
unzip protoc-4.25.5-linux-x86_64.zip -d $HOME/.local
export PATH="$PATH:$HOME/.local/bin"
# Check that the protobuf compiler is installed
protoc --version
Expand Down
2 changes: 1 addition & 1 deletion python/requirements.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
async-timeout==4.0.2
google-api-python-client==2.85.0
maturin==0.13.0
protobuf==3.20.*
protobuf==4.25.*
pytest==7.1.2
pytest-asyncio==0.19.0
typing_extensions==4.8.0
Expand Down
Loading