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

Fix CI issues #29

Merged
merged 3 commits into from
Jun 26, 2024
Merged

Fix CI issues #29

merged 3 commits into from
Jun 26, 2024

Conversation

bjosv
Copy link
Collaborator

@bjosv bjosv commented Jun 26, 2024

  • Fixes to build on FreeBSD in CI:

    • Header alloca.h is not available on FreeBSD.
    • Only add libevent includes if found.
    • Don't build tests in CI job on FreeBSD when using CMake.
    • Skip test/examples using glib if pkg-config is not installed.
    • Remove unused test label UT/CT.
  • Run CI workflow "C/C++ CI"

  • Get redis_version if valkey_version missing in INFO.
    Fixes issues in CI job "DB compatibility testing"

Fixes issues in CI job "DB compatibility testing"

Signed-off-by: Björn Svensson <[email protected]>
Signed-off-by: Björn Svensson <[email protected]>
- Header alloca.h is not available on FreeBSD.
- Only add libevent includes if found.
- Don't build tests in CI job on FreeBSD when using CMake.
- Skip test/examples using "glib" if pkg-config is not installed.
- Remove unused test label UT/CT.

Signed-off-by: Björn Svensson <[email protected]>
Copy link
Collaborator

@michael-grunder michael-grunder left a comment

Choose a reason for hiding this comment

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

Awesome! Looking forward to more green tests.

@bjosv bjosv merged commit 4b601d9 into valkey-io:main Jun 26, 2024
13 of 45 checks passed
@bjosv bjosv deleted the ci-fixes branch June 26, 2024 16:02
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