Skip to content

Conversation

@gilmishal
Copy link
Contributor

Added quotes to the commands that caused issues on my machine - Managed to build it and deploy with no issues.

Copy link
Member

@miniksa miniksa left a comment

Choose a reason for hiding this comment

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

I know there's other outstanding issues/plans to replace xcopy with robocopy or something that doesn't have language/locale dependent flags. But this fix seems like goodness we can take now until that is done.

@miniksa miniksa added the Area-Build Issues pertaining to the build system, CI, infrastructure, meta label May 14, 2019
@miniksa
Copy link
Member

miniksa commented May 14, 2019

I was thinking of #741 specifically...

@gilmishal
Copy link
Contributor Author

This is me trying to address #438. The quotes are also for the mkdir that was causing issues for me, but you don't have to accept my PR if #741 fixes it as well.

@miniksa
Copy link
Member

miniksa commented May 14, 2019

#741 is WIP, so I'm willing to take this in the interim.

@adiviness adiviness merged commit ea5270e into microsoft:master May 14, 2019
ObsidianPhoenix pushed a commit to ObsidianPhoenix/Terminal that referenced this pull request May 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area-Build Issues pertaining to the build system, CI, infrastructure, meta

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants