smoke_test: Run git commit
to work around cargo bug
#9405
Merged
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.
This is a workaround for rust-lang/cargo#14354, which currently prevents us from running our smoke tests.
cargo currently (Rust 1.81) crashes when
cargo publish
(orcargo package
) is used on a git repository that has no commits yet. Since that is the setup we had been using for our smoke tests we are currently running into this issue. The solution in this PR runsgit add --all
andgit commit
as part of creating our dummy project, which resolves the problem on our end. As a nice bonus, this allows us to remove the--allow-dirty
argument from thecargo publish
call, since we have now committed all of our changes.