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

Upgrade dependencies #12

Merged
merged 1 commit into from
Sep 8, 2020
Merged

Upgrade dependencies #12

merged 1 commit into from
Sep 8, 2020

Conversation

hug-dev
Copy link
Member

@hug-dev hug-dev commented Sep 2, 2020

Signed-off-by: Hugues de Valon hugues.devalon@arm.com

@hug-dev
Copy link
Member Author

hug-dev commented Sep 2, 2020

There is a bug in psa-crypto when it is compiled with --no-default-features --features interface, it does not compile.

@hug-dev
Copy link
Member Author

hug-dev commented Sep 2, 2020

This will need to pull the newer version of psa-crypto once parallaxsecond/rust-psa-crypto#61 is merged.

@hug-dev hug-dev force-pushed the upgrade branch 2 times, most recently from 2c80d0a to d6ec419 Compare September 7, 2020 18:10
@hug-dev
Copy link
Member Author

hug-dev commented Sep 7, 2020

CI fails because of parallaxsecond/rust-psa-crypto#63

@@ -1,5 +1,6 @@
# Executing our tests on Arm64 with Travis CI
arch: arm64
dist: bionic
Copy link
Member Author

Choose a reason for hiding this comment

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

Was needed to have a recent version of libclang

Copy link
Member

@ionut-arm ionut-arm left a comment

Choose a reason for hiding this comment

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

LGTM!

@hug-dev
Copy link
Member Author

hug-dev commented Sep 8, 2020

Added a patch section in the Cargo.toml, to be replaced with a newer version of psa-crypto when released.

Signed-off-by: Hugues de Valon <hugues.devalon@arm.com>
@hug-dev hug-dev merged commit 68a1ec1 into parallaxsecond:master Sep 8, 2020
@hug-dev hug-dev deleted the upgrade branch September 8, 2020 11:40
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