Skip to content

[MOD-10920] Improve BUILD_INTEL_SVS_OPT flag validation#6865

Merged
meiravgri merged 1 commit intomasterfrom
meiravg_explicit_intel_enable
Sep 21, 2025
Merged

[MOD-10920] Improve BUILD_INTEL_SVS_OPT flag validation#6865
meiravgri merged 1 commit intomasterfrom
meiravg_explicit_intel_enable

Conversation

@meiravgri
Copy link
Copy Markdown
Collaborator

@meiravgri meiravgri commented Sep 21, 2025

Align with Redis conventions (yes/no options) while maintaining RediSearch conventions (1/0 flags) for compatibility with both Redis builds and internal usage.

Current: BUILD_INTEL_SVS_OPT enables Intel SVS optimization for any non-zero value
Change: Only accept explicit values "yes" or "1" to enable the feature

@meiravgri meiravgri requested a review from GuyAv46 September 21, 2025 05:05
@meiravgri meiravgri changed the title Fix build script logic for Intel SVS option handling [MOD-10920] Improve BUILD_INTEL_SVS_OPT flag validation Sep 21, 2025
@meiravgri meiravgri enabled auto-merge September 21, 2025 05:30
@meiravgri meiravgri added this pull request to the merge queue Sep 21, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 21, 2025
@meiravgri meiravgri added this pull request to the merge queue Sep 21, 2025
Merged via the queue into master with commit 95695cc Sep 21, 2025
21 of 22 checks passed
@meiravgri meiravgri deleted the meiravg_explicit_intel_enable branch September 21, 2025 12:45
redisearch-backport-pull-request bot pushed a commit that referenced this pull request Sep 21, 2025
Fix build script logic for Intel SVS option handling

(cherry picked from commit 95695cc)
@redisearch-backport-pull-request
Copy link
Copy Markdown
Contributor

Successfully created backport PR for 8.2:

github-merge-queue bot pushed a commit that referenced this pull request Sep 21, 2025
[MOD-10920] Improve BUILD_INTEL_SVS_OPT flag validation (#6865)

Fix build script logic for Intel SVS option handling

(cherry picked from commit 95695cc)

Co-authored-by: meiravgri <109056284+meiravgri@users.noreply.github.com>
nafraf pushed a commit that referenced this pull request Sep 22, 2025
Fix build script logic for Intel SVS option handling
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants