Skip to content
This repository was archived by the owner on Oct 4, 2022. It is now read-only.

Format paths for Linux platform containers#192

Closed
cdhunt wants to merge 1 commit into
dagger:mainfrom
cdhunt:main
Closed

Format paths for Linux platform containers#192
cdhunt wants to merge 1 commit into
dagger:mainfrom
cdhunt:main

Conversation

@cdhunt
Copy link
Copy Markdown
Contributor

@cdhunt cdhunt commented Sep 9, 2022

This isn't a permanent solution. If cloak ever wants to support Windows containers, I'm not sure this will work. But, it gets me past the extension install phase so I can actually test other parts.

@netlify
Copy link
Copy Markdown

netlify Bot commented Sep 9, 2022

👷 Deploy request for cloak-docs pending review.

A Netlify team Owner will need to approve the deploy before you can run your build.

Are you a team Owner? Visit the deploys page to approve it

Need more help? Learn more in the Netlify docs

Name Link
🔨 Latest commit 5e12daf

Signed-off-by: Chris Hunt <chunt@stackoverflow.com>
@marcosnils
Copy link
Copy Markdown
Contributor

does this still works in unix?

@cdhunt
Copy link
Copy Markdown
Contributor Author

cdhunt commented Sep 9, 2022

It works for me in Ubuntu in WSL.

@sipsma
Copy link
Copy Markdown
Contributor

sipsma commented Sep 16, 2022

Hey @cdhunt we are working on migrating PRs to a different repo and wanted to make that as easy as possible for you, so I went ahead and rebased this and merged in a different PR: #228

Appreciate the contribution! There is a follow up described here but I was not even aware of this issue before your change, so it's extremely helpful to have the initial one in and to know that we have to deal with this going forward.

@sipsma sipsma closed this Sep 16, 2022
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.

3 participants