Skip to content

Conversation

@guibescos
Copy link
Contributor

@guibescos guibescos commented Jun 30, 2022

Motivation : the oracle contract compiles with solana 1.7.2, but we'd like to use a recent version of solana for the rust parts that are gonna come next. Keeping the c code with 1.7.2 would require us having 2 versions of solana in the docker.
We'd much rather bump solana to the lastest version and using the same version to compile the c code and the rust code.

This PR bumps solana to 1.10.29. Fixing the CI required the following changes.

  • No longer including limits.h in upd_aggregate.h. The solana c sdk doesn't play nicely with libc and solana_sdk.h probably has this header.
  • If SOL_TEST is defined include solana_sdk.h instead of stdint.h. This change only affects criterion tests.
  • Reduce the scope of the solana.patch to only add warning flags. We remove a patch that would declared an unused function as __attribute__(( weak )).

@guibescos guibescos merged commit ff4009f into main Jul 1, 2022
@guibescos guibescos deleted the bump-solana-2 branch July 1, 2022 14:42
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.

3 participants