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

bundle: fix issue when bundles would be empty #79

Closed
wants to merge 1 commit into from

Conversation

dscho
Copy link
Member

@dscho dscho commented Nov 13, 2018

And yet another patch coming through Git for Windows...

Change since v1:

  • Using a better oneline now.

Cc: Stefan Beller sbeller@google.com

@dscho
Copy link
Member Author

dscho commented Nov 13, 2018

/submit

@gitgitgadget
Copy link

gitgitgadget bot commented Nov 13, 2018

Submitted as pull.79.git.gitgitgadget@gmail.com

When an user tries to create an empty bundle via `git bundle create
<bundle> <revlist>` where `<revlist>` resolves to an empty list (for
example, like `master..master`), the command fails and warns the user
about how it does not want to create empty bundle.

However, the `.lock` file was still open and on Windows that means
that it could not be deleted properly. This patch fixes that issue.

This closes git-for-windows#790

Signed-off-by: Gaël Lhez <gael.lhez@gmail.com>
Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
@dscho
Copy link
Member Author

dscho commented Nov 14, 2018

/submit

@gitgitgadget
Copy link

gitgitgadget bot commented Nov 14, 2018

Submitted as pull.79.v2.git.gitgitgadget@gmail.com

@dscho
Copy link
Member Author

dscho commented Nov 21, 2018

@peff came up with a better fix that was merged in 9e3dc6b.

@dscho dscho closed this Nov 21, 2018
@dscho dscho deleted the create-empty-bundle branch November 21, 2018 20:15
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