Skip to content
This repository was archived by the owner on Mar 26, 2020. It is now read-only.

Fix to escape options containing ‘<>’ on windows platform#419

Open
chrisg15 wants to merge 1 commit intodropbox:masterfrom
chrisg15:windows-fix
Open

Fix to escape options containing ‘<>’ on windows platform#419
chrisg15 wants to merge 1 commit intodropbox:masterfrom
chrisg15:windows-fix

Conversation

@chrisg15
Copy link
Copy Markdown

No description provided.

@artwyman
Copy link
Copy Markdown
Contributor

artwyman commented Nov 28, 2018

Some questions to understand the intent of this PR:
What's the use case and problem this solves? What arguments in particular have run into this?
Why isn't the escaping done by the caller who actually specifies the arguments?

@xianwen
Copy link
Copy Markdown

xianwen commented Dec 26, 2018

Hi, @chrisg15:
I noticed you haven't signed the CLA yet, could you please sign it here: https://opensource.dropbox.com/cla/
Thanks a lot!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants