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

Ensure gcc is newer >= 11.2.1 < 12.0.0 when building knn lib #1664

Merged

Conversation

peterzhuamazon
Copy link
Member

@peterzhuamazon peterzhuamazon commented Apr 29, 2024

Description

Ensure gcc is newer >= 11.2.1 < 12.0.0 when building knn lib

Issues Resolved

opensearch-project/opensearch-build#4379 (comment)
opensearch-project/opensearch-build#4379 (comment)

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • Commits are signed as per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
@peterzhuamazon peterzhuamazon added Enhancements Increases software capabilities beyond original client specifications release skip-changelog v2.14.0 labels Apr 29, 2024
@peterzhuamazon peterzhuamazon self-assigned this Apr 29, 2024
@peterzhuamazon
Copy link
Member Author

peterzhuamazon commented Apr 29, 2024

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
@peterzhuamazon peterzhuamazon changed the title Ensure gcc is newer >= 10.3.1 < 12.0.0 when building knn lib Ensure gcc is newer >= 11.2.1 < 12.0.0 when building knn lib Apr 30, 2024
Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
@jmazanec15
Copy link
Member

Build might fail because of #1668. Non-blocker this will fix it.

@jmazanec15 jmazanec15 merged commit da4cd1c into opensearch-project:main Apr 30, 2024
59 of 63 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Apr 30, 2024
Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
(cherry picked from commit da4cd1c)
@peterzhuamazon peterzhuamazon deleted the update-gcc10-restrictions-2 branch April 30, 2024 18:09
peterzhuamazon added a commit to peterzhuamazon/k-NN that referenced this pull request Apr 30, 2024
…pensearch-project#1664)"

This reverts commit da4cd1c.

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
navneet1v pushed a commit that referenced this pull request Apr 30, 2024
* Revert "Ensure gcc is newer >= 11.2.1 < 12.0.0 when building knn lib (#1664)"

This reverts commit da4cd1c.

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>

* Revert "Update gcc restrictions to version 12.0.0 (#1634)"

This reverts commit 11342be.

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>

---------

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
opensearch-trigger-bot bot pushed a commit that referenced this pull request Apr 30, 2024
* Revert "Ensure gcc is newer >= 11.2.1 < 12.0.0 when building knn lib (#1664)"

This reverts commit da4cd1c.

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>

* Revert "Update gcc restrictions to version 12.0.0 (#1634)"

This reverts commit 11342be.

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>

---------

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
(cherry picked from commit f432b0e)
peterzhuamazon added a commit that referenced this pull request Apr 30, 2024
* Revert "Ensure gcc is newer >= 11.2.1 < 12.0.0 when building knn lib (#1664)"

This reverts commit da4cd1c.

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>

* Revert "Update gcc restrictions to version 12.0.0 (#1634)"

This reverts commit 11342be.

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>

---------

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
(cherry picked from commit f432b0e)

Co-authored-by: Peter Zhu <zhujiaxi@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 2.x Enhancements Increases software capabilities beyond original client specifications release skip-changelog v2.14.0
Projects
Development

Successfully merging this pull request may close these issues.

4 participants