Skip to content

fix(chore): Handle spaces in git command arguments#5375

Merged
rashil2000 merged 3 commits intodevelopfrom
fix-git-quotes
Feb 6, 2023
Merged

fix(chore): Handle spaces in git command arguments#5375
rashil2000 merged 3 commits intodevelopfrom
fix-git-quotes

Conversation

@rashil2000
Copy link
Copy Markdown
Member

Description

Fix regression introduced in #5122, i.e. handling of Scoop home paths which have spaces in them.

How Has This Been Tested?

Run commands like scoop bucket ... or scoop info ... which use Git internally.

Checklist:

  • I have read the Contributing Guide.
  • I have ensured that I am targeting the develop branch.
  • I have updated the documentation accordingly.
  • I have updated the tests accordingly.
  • I have added an entry in the CHANGELOG.

@rashil2000
Copy link
Copy Markdown
Member Author

@niheaven could you please review? This is a crucial fix for me as I have a non-standard Scoop home folder.

Comment thread lib/core.ps1 Outdated
Copy link
Copy Markdown
Member

@niheaven niheaven left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@rashil2000 rashil2000 merged commit e2558ac into develop Feb 6, 2023
@rashil2000 rashil2000 deleted the fix-git-quotes branch February 6, 2023 23:25
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