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

Add xxd for data file regenerating tests #112

Merged
merged 1 commit into from
Jul 11, 2023

Conversation

yuhaoth
Copy link
Contributor

@yuhaoth yuhaoth commented Jul 10, 2023

xxd command is used for generating malformed data files. And Mbed-TLS/mbedtls#7866 introduces tests for data files.

Copy link
Contributor

@gilles-peskine-arm gilles-peskine-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. Please run a Linux job on each CI to check that the docker images are built correctly.

@@ -85,6 +85,8 @@ RUN apt-get update -q && apt-get install -yq \
python3-pip \
# for Mbed TLS tests
valgrind \
# for data files generating. xxd is provide by vim
Copy link
Contributor

Choose a reason for hiding this comment

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

By vim-common, actually, but I don't mind installing vim.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

For 16.04, it is provide by vim.

I did not try other distro :)

Copy link
Contributor

Choose a reason for hiding this comment

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

schroot -c xenial -- dpkg -S bin/xxd
vim-common: /usr/bin/xxd

But it doesn't matter.

@gilles-peskine-arm gilles-peskine-arm added needs: ci needs: review needs: reviewer size-xs Estimated task size: extra small (a few hours at most) labels Jul 10, 2023
Copy link
Contributor

@bensze01 bensze01 left a comment

Choose a reason for hiding this comment

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

LGTM, approved pending CI results.

@bensze01 bensze01 added approved Approved in review. May need additional CI. and removed needs: review needs: reviewer labels Jul 10, 2023
@bensze01
Copy link
Contributor

Test runs:

@bensze01
Copy link
Contributor

bensze01 commented Jul 11, 2023

Both CIs built the new images successfully - and both runs failed due to unrelated issues (network issues reaching github on the internal side, and SERVER START TIMEOUT failures the OpenCI that have also shown up in the nightlies)

@bensze01 bensze01 merged commit 0d811e3 into Mbed-TLS:master Jul 11, 2023
1 check passed
@yuhaoth yuhaoth deleted the pr/add-xxd-command branch July 12, 2023 04:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Approved in review. May need additional CI. needs: ci size-xs Estimated task size: extra small (a few hours at most)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants