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

Cargo.lock should not be in gitignore #105

Merged
merged 3 commits into from
Sep 30, 2020
Merged

Cargo.lock should not be in gitignore #105

merged 3 commits into from
Sep 30, 2020

Conversation

lukehinds
Copy link
Member

@lukehinds lukehinds commented Sep 30, 2020

"If you’re building a non-end product, such as a rust library that
other rust packages will depend on, put Cargo.lock in your
.gitignore. If you’re building an end product, which are executable
like command-line tool or an application, or a system library with
crate-type of staticlib or cdylib, check Cargo.lock into git."

https://doc.rust-lang.org/cargo/guide/cargo-toml-vs-cargo-lock.html

Luke Hinds added 3 commits September 29, 2020 15:52
We may need to think of better ways of syncing the conf between
both projects. For now I have moved the file over in its entirety
as we will require integration tests and so forth (so keys around
the verifier and register will be used there).
"If you’re building a non-end product, such as a rust library that
other rust packages will depend on, put Cargo.lock in your
.gitignore. If you’re building an end product, which are executable
like command-line tool or an application, or a system library with
crate-type of staticlib or cdylib, check Cargo.lock into git."

https://doc.rust-lang.org/cargo/guide/cargo-toml-vs-cargo-lock.html
@lukehinds lukehinds changed the title Lock ignore Cargo.lock should not be in gitignore Sep 30, 2020
@ashcrow
Copy link
Contributor

ashcrow commented Sep 30, 2020

test crypto::tests::test_do_hmac ... ok
error: test failed, to rerun pass '--bin keylime_agent'
Error: Process completed with exit code 1.

CI failure looks unrelated.

@ashcrow ashcrow merged commit 527aaf0 into keylime:master Sep 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants