-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Fix SPM error codes #9829
Fix SPM error codes #9829
Conversation
Note: these changes were already merged to master and then overwritten by a later PR. This should just add them back in. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
revert 100644 → 100755 file permission change
thanks @0xc0170 , fixed |
@netanelgonen Can you add this to the commit msg:
I knew I saw it previously (this fix). Once added, we will start CI |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
changing the SPM error codes from invalid argument to bad state aligned to PSA spec
8ca4f0f
to
b5b0e98
Compare
fixed , stating CI |
Test run: SUCCESSSummary: 12 of 12 test jobs passed |
Description
reopening #9754 after the changes where overrun
changing the SPM error codes from invalid argument to bad state aligned to PSA spec
Pull request type
Reviewers
@itayzafrir
Release Notes