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

Docker: Allow setting a mapping for docker paths #46

Merged
merged 3 commits into from
Aug 30, 2022

Conversation

cmorty
Copy link
Contributor

@cmorty cmorty commented Aug 16, 2022

Bug 579944 introduced using Windows paths, to allow putting your project into the WSL's file system.
While Docker for Windows (the commercial version) supports Windows paths, this does not work with a OSS-docker installed in the WSL.
[See discussion in Bug 579944]] (https://bugs.eclipse.org/bugs/show_bug.cgi?id=579944#c5)
It is now possible to provide a path mapping to work around this.

@jonahgraham jonahgraham requested a review from jjohnstn August 17, 2022 10:46
cmorty added 2 commits August 25, 2022 18:44
Bug 579944 introduced using Windows paths, to allow putting your project
into the WSL's file system.
While Docker for Windows (the commercial version) supports Windows
paths, this does not work with a OSS-docker installed in the WSL.
It is now possible to provide a path mapping to work around this.
Copy link
Contributor

@jjohnstn jjohnstn left a comment

Choose a reason for hiding this comment

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

Just a minor comment about unnecessary blocks, but it should not block merging. Don't know why the build keeps failing, but will keep trying.

Change-Id: I61ff533fb496e19dbcd187a8f7ce60a6face599e
@akurtakov
Copy link
Contributor

These tests are in unrelated plugins so ignore if you want this in 2022-09 release.

@jjohnstn jjohnstn merged commit 52483c0 into eclipse-cdt:main Aug 30, 2022
@jonahgraham
Copy link
Member

These tests are in unrelated plugins so ignore if you want this in 2022-09 release.

BTW Current Main branch is for 2022-12 release. For 2022-09 we have already released 10.7.1 - of course we can do a 10.7.2 if we want to backport things. Please let me know.

@jonahgraham jonahgraham added this to the 11.0.0 milestone Aug 30, 2022
@jjohnstn
Copy link
Contributor

I didn't know that 11.0.0 wasn't 2022-09. The fix can't be back-ported as it does a major version bump in a plug-in so needs to be part of 11.0.0.

@jonahgraham jonahgraham added the noteworthy Pull requests and fixed issues that should be highlighted to users label Sep 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
noteworthy Pull requests and fixed issues that should be highlighted to users
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants