Skip to content

Conversation

jgiannuzzi
Copy link
Member

  • CI: improve version management
    • make sure that only the specific version of librdkafka.redist can be restored
    • make sure that the tag and project versions match
    • handle versions with build metadata
  • Hotfix: use system-provided cyrus-sasl/libsasl2 at runtime (use librdkafka.redist version 2.11.0.1-RC1+gr)
  • Bump version to 2.11.0.1-RC1+gr

- make sure that the specific version of librdkafka.redist can be restored
- make sure that the tag and project versions match
- handle versions with build metadata
<Title>Confluent.Kafka</Title>
<AssemblyName>Confluent.Kafka</AssemblyName>
<VersionPrefix>2.11.0</VersionPrefix>
<VersionPrefix>2.11.0.1-RC1+gr</VersionPrefix>
Copy link

@marcin-krystianc marcin-krystianc Aug 15, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess that should be a Version, not a VersionPrefix if we specify the entire version here. But as far as I remember, it doesn't matter - VersionPrefix works too.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It works with VersionPrefix, but I'm happy to change it to Version. I was trying to minimise changes from upstream!


<ItemGroup>
<PackageReference Include="librdkafka.redist" Version="2.11.0-gr">
<PackageReference Include="librdkafka.redist" Version="2.11.0.1-RC1+gr">

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is fine, but note that the +gr suffix doesn't have any effect here; it would work the same with 2.11.0.1-RC1

@jgiannuzzi jgiannuzzi merged commit 22fb1ef into G-Research:2.11.0-gr Aug 15, 2025
5 checks passed
@jgiannuzzi jgiannuzzi deleted the hotfix/2.11.0.1 branch August 15, 2025 08:56
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