Add coverity push script for 3.6 #215
Draft
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Paul Elliott <[email protected]>
Signed-off-by: Paul Elliott <[email protected]>
Mistaken usage of hashlib.md5(), much simpler form is available. Signed-off-by: Paul Elliott <[email protected]>
-j without a number means unlimited, and is likely a bad idea on limited systems. Signed-off-by: Paul Elliott <[email protected]>
(Rather than empty string) Signed-off-by: Paul Elliott <[email protected]>
Signed-off-by: Paul Elliott <[email protected]>
Signed-off-by: Paul Elliott <[email protected]>
Signed-off-by: Paul Elliott <[email protected]>
Signed-off-by: Paul Elliott <[email protected]>
Refactor previous function as now too small to really be required (error handling now done by exceptions thrown by run(), and only a single API call now required) Signed-off-by: Paul Elliott <[email protected]>
Signed-off-by: Paul Elliott <[email protected]>
Also make branch change recurse into submodules, as we now have one. Signed-off-by: Paul Elliott <[email protected]>
Use 'with' context, copy to backup file if required. Signed-off-by: Paul Elliott <[email protected]>
The addition of the framework necessitates this. Signed-off-by: Paul Elliott <[email protected]>
Missed as logic causes the script to just redownload the tools. Signed-off-by: Paul Elliott <[email protected]>
Instead of erroring out if the specified backup directory is non-existent, just create it. Signed-off-by: Paul Elliott <[email protected]>
Signed-off-by: Paul Elliott <[email protected]>
Useful for debug more than anything else. Signed-off-by: Paul Elliott <[email protected]>
.. and update documentation to add new options. Signed-off-by: Paul Elliott <[email protected]>
If config.h was changed upstream, it would be backed up prior to these changes being applied locally, and then restored into the wrong state. Signed-off-by: Paul Elliott <[email protected]>
Moving the backup of config files fixed the previous issue, but created another one. Signed-off-by: Paul Elliott <[email protected]>
Signed-off-by: Paul Elliott <[email protected]>
Enable testing without consuming a coverity credit Signed-off-by: Paul Elliott <[email protected]>
With the addition of tf-psa-crypto as a submodule, which itself has framework as a submodule, git submodule calls now need to be recursive Signed-off-by: Paul Elliott <[email protected]>
Required some re-work to have two files. May not entirely be the optimal way of doing this, but probably only worth more work if more files are required. Signed-off-by: Paul Elliott <[email protected]>
Signed-off-by: David Horstmann <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
A tweaked version of #165 that allows pushing builds for the
mbedtls-3.6branch (rather thandevelopmentas was happening before).