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

Fixing Windows not playing nicely with mkdir. #149

Merged
merged 3 commits into from
Jul 17, 2020
Merged

Fixing Windows not playing nicely with mkdir. #149

merged 3 commits into from
Jul 17, 2020

Conversation

nitz
Copy link
Contributor

@nitz nitz commented Jul 17, 2020

I really don't know why Windows is such a pain sometimes:

  • The -p doesn't get handled as a switch, and instead actually creates a directory named "-p"...
  • The forward slash in the path being created causes cmd.exe to be sad instead of interpreting it as a path. Quotes around the path makes it happy.

One day things will be sane on Windows.

Today is not that day. 😂

I really don't know why Windows is such a pain sometimes:
 * The `-p` doesn't get handled as a switch, and instead actually creates a directory named "-p"...
 * The forward slash in the path being created causes cmd.exe to be sad instead of interpreting it as a path. Quotes around the path makes it happy.

One day things will be sane on Windows.

Today is not that day. 😂
Makefile Outdated Show resolved Hide resolved
Makefile Outdated Show resolved Hide resolved
Makefile Outdated Show resolved Hide resolved
Copy link
Member

@hathach hathach left a comment

Choose a reason for hiding this comment

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

perfect now, thank you very much for the PR

@hathach hathach merged commit 0b3c515 into adafruit:master Jul 17, 2020
@nitz nitz deleted the sorry-my-build-env-is-absolute-nonsense branch July 17, 2020 15:33
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