Skip to content
This repository has been archived by the owner on Apr 14, 2021. It is now read-only.

Allow bundle install --force to work with git specs #5680

Merged
merged 2 commits into from
Jun 1, 2017

Conversation

segiddins
Copy link
Member

@segiddins segiddins commented May 26, 2017

This allows bundle install --force to work when the gemfile includes a git spec, previously it would error nonsensically. This happened because sources needed to be remote!ed for installation to succeed, and this wouldn't happen when --force was called with no other changes to the gemfile.

Closes #5678

@segiddins segiddins added this to the 1.15.1 milestone May 26, 2017
@indirect
Copy link
Member

This looks good. Can you expand the PR description a bit to explain the why behind this PR? Based on the linked ticket it seems like using --force was somehow requesting a checkout even though Bundler wasn't allowing git ops; why wasn't Bundler allowing git ops? Could you also add your rationale for reverting 2dc5f1d in dad4f4d?

@segiddins
Copy link
Member Author

Description updated, the first commit was reverted because it was wrong and broke other things, I can rebase it away if you like.

@segiddins segiddins force-pushed the seg-git-force-no-git-ops branch from dad4f4d to 5161573 Compare May 29, 2017 23:42
@segiddins
Copy link
Member Author

Rebased

@segiddins
Copy link
Member Author

@bundlerbot r+

@bundlerbot
Copy link
Collaborator

📌 Commit 5161573 has been approved by segiddins

@bundlerbot
Copy link
Collaborator

⌛ Testing commit 5161573 with merge 7f1411c...

bundlerbot added a commit that referenced this pull request Jun 1, 2017
Allow bundle install --force to work with git specs

This allows `bundle install --force` to work when the gemfile includes a git spec, previously it would error nonsensically. This happened because sources needed to be `remote!`ed for installation to succeed, and this wouldn't happen when `--force` was called with no other changes to the gemfile.

Closes #5678
@bundlerbot
Copy link
Collaborator

☀️ Test successful - status-travis
Approved by: segiddins
Pushing 7f1411c to master...

@bundlerbot bundlerbot merged commit 5161573 into master Jun 1, 2017
@segiddins segiddins deleted the seg-git-force-no-git-ops branch June 2, 2017 04:58
segiddins pushed a commit that referenced this pull request Jun 2, 2017
Allow bundle install --force to work with git specs

This allows `bundle install --force` to work when the gemfile includes a git spec, previously it would error nonsensically. This happened because sources needed to be `remote!`ed for installation to succeed, and this wouldn't happen when `--force` was called with no other changes to the gemfile.

Closes #5678

(cherry picked from commit 7f1411c)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants