Skip to content

Commit

Permalink
Merge pull request #50 from traversaro/fixprotobufpyconstraint
Browse files Browse the repository at this point in the history
Fix constraint on protobuf version in gz-msgs python package
  • Loading branch information
traversaro authored Nov 4, 2024
2 parents 19e40db + d406e62 commit ad5a1e3
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 25 deletions.

This file was deleted.

14 changes: 7 additions & 7 deletions recipe/meta.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,9 @@
{% set component_version = component_name + major_version %}
{% set cxx_name = "lib" + name %}
{% set python_name = name + "-python" %}
{% set protobuf_major_version = "5" %}

{% set protobuf_version = "5.28.2" %}
{% set protobuf_major_version = protobuf_version.split('.')[0] %}
{% set next_protobuf_major_version = "6" %}

package:
name: {{ name }}
Expand All @@ -20,7 +21,7 @@ source:
- win_enable_py_tests.patch

build:
number: 3
number: 4

outputs:
- name: {{ cxx_name }}
Expand Down Expand Up @@ -94,14 +95,13 @@ outputs:
- libabseil
- libprotobuf
- python
- protobuf {{ protobuf_major_version }}.*
- protobuf {{ protobuf_version }}
- pytest
- dlfcn-win32 # [win]
run:
- python >=3.8
# We need to ensure that at runtime we use the same major protobuf
# version that was used to generate the python messages
- protobuf {{ protobuf_major_version }}.*
# See https://protobuf.dev/support/cross-version-runtime-guarantee/
- protobuf >={{ protobuf_version }},<{{ next_protobuf_major_version }}.0a0
test:
commands:
- pip check
Expand Down

0 comments on commit ad5a1e3

Please sign in to comment.