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

Rewrite quickstart example command line interface #186

Merged
merged 2 commits into from Dec 20, 2019
Merged

Conversation

@mssun
Copy link
Member

mssun commented Dec 20, 2019

Description

Two sub-commands:

echo:

./release/examples/quickstart echo -e release/examples/enclave_info.toml -i "Hello, World!"

psi:

MESATEE_STORAGE_DIR=/tmp/mesatee_storage ./release/examples/quickstart psi -e release/examples/enclave_info.toml -f tests/integration_test/test_data/psi1.txt  -f tests/integration_test/test_data/psi2.txt

Note that this PR is to have a better interfaces. I'll come back to polish the code. Thanks.

Type of change (select applied and DELETE the others)

  • Example rewrite.

How Has This Been Tested?

CI: https://ci.mesalock-linux.org/mssun/incubator-mesatee/216

Checklist (check ALL before submitting PR, even not applicable)

  • Fork the repo and create your branch from master.
  • If you've added code that should be tested, add tests.
  • If you've changed APIs, update the documentation.
  • Ensure the tests pass (see CI results).
  • Make sure your code lints/format.
mssun added 2 commits Dec 20, 2019
@mssun mssun mentioned this pull request Dec 20, 2019
@mssun mssun requested a review from m4sterchain Dec 20, 2019
Copy link
Contributor

m4sterchain left a comment

LGTM

@mssun mssun merged commit 708831d into apache:master Dec 20, 2019
@mssun mssun deleted the mssun:quickstart branch Dec 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.