Skip to content

Conversation

@guptask
Copy link

@guptask guptask commented May 4, 2023

This change is Reviewable

@guptask guptask requested a review from igchor May 4, 2023 17:42
@igchor
Copy link

igchor commented May 4, 2023

LGTM, let's just wait and see if it builds

@vinser52
Copy link

vinser52 commented May 5, 2023

The old compiler is still used during the build;

-- The C compiler identification is GNU 8.5.0
-- The CXX compiler identification is GNU 8.5.0

@raema
Copy link

raema commented May 9, 2023

The old compiler is still used during the build;

-- The C compiler identification is GNU 8.5.0
-- The CXX compiler identification is GNU 8.5.0

cmake was still detecting the old 8.5 gcc when building Cachelib.. I added lines to enable gcc 12 prior to running cmake and it is using gcc 12 as expected now.

@igchor igchor merged commit ab98751 into intel:develop May 9, 2023
vinser52 pushed a commit that referenced this pull request Jul 17, 2023
updated the docker gcc version to 12

---------

Co-authored-by: Matt Rae <[email protected]>
byrnedj pushed a commit that referenced this pull request Jul 23, 2023
updated the docker gcc version to 12

---------

Co-authored-by: Matt Rae <[email protected]>
vinser52 pushed a commit that referenced this pull request Feb 29, 2024
updated the docker gcc version to 12

---------

Co-authored-by: Matt Rae <[email protected]>
vinser52 pushed a commit that referenced this pull request Mar 1, 2024
updated the docker gcc version to 12

---------

Co-authored-by: Matt Rae <[email protected]>
byrnedj pushed a commit that referenced this pull request May 21, 2024
updated the docker gcc version to 12

---------

Co-authored-by: Matt Rae <[email protected]>
byrnedj pushed a commit that referenced this pull request Jun 25, 2024
updated the docker gcc version to 12

---------

Co-authored-by: Matt Rae <[email protected]>
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.

4 participants