Skip to content

Backward support PSA_ITS #9862

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

Merged
merged 2 commits into from
Feb 27, 2019
Merged

Conversation

netanelgonen
Copy link
Contributor

@netanelgonen netanelgonen commented Feb 26, 2019

Description

Add support to previous PSA ITS spec

adding an alias to the old PSA error codes and types
return invalid argument in case of receiving an uid==0

This is part of PSA compliance tests effort

Pull request type

[X] Fix
[ ] Refactor
[ ] Target update
[ ] Functionality change
[ ] Docs update
[ ] Test update
[ ] Breaking change

Reviewers

@itayzafrir @Alonof @davidsaada

Release Notes

Copy link
Contributor

@itayzafrir itayzafrir left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@davidsaada davidsaada left a comment

Choose a reason for hiding this comment

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

LGTM.

Copy link
Contributor

@0xc0170 0xc0170 left a comment

Choose a reason for hiding this comment

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

💯 backward compatibility

@cmonr
Copy link
Contributor

cmonr commented Feb 26, 2019

CI started

@cmonr cmonr added the risk: A label Feb 26, 2019
@mbed-ci
Copy link

mbed-ci commented Feb 26, 2019

Test run: FAILED

Summary: 1 of 1 test jobs failed
Build number : 1
Build artifacts

Failed test jobs:

  • jenkins-ci/mbed-os-ci_unittests

@cmonr
Copy link
Contributor

cmonr commented Feb 26, 2019

Started CI prematurely. Need IAR8 work to land first.

@cmonr
Copy link
Contributor

cmonr commented Feb 26, 2019

CI started

@mbed-ci
Copy link

mbed-ci commented Feb 27, 2019

Test run: SUCCESS

Summary: 12 of 12 test jobs passed
Build number : 2
Build artifacts

@cmonr cmonr merged commit d96c9e8 into ARMmbed:master Feb 27, 2019
@cmonr cmonr removed the needs: CI label Feb 27, 2019
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.

6 participants