DevContainer initialization fails with "Syntaxfehler"

[bl81hi]

Description

On my Windows 11 machine running Docker Desktop, the DevContainer initializationCommand

"initializeCommand": "mkdir -p ${localEnv:HOME}/.cache/bazel"

becomes

C:\WINDOWS\system32\cmd.exe /c mkdir -p C:\Users\bl81hi/.cache/bazel

which quits with "Syntaxfehler."

Error Occurrence Rate

Reproducible

How to reproduce

clone repo

Start VS Code and confirm "Run in DevContainer"

Supporting Information

No response

Classification

minor

Affected Version

0.5

Expected Closure Version

0.5

Category

• Safety Related
• Security Related

[lurtz]

Can you check if #41 works for you? I have no Windows with Docker available to test it.

[bl81hi]

It works one time when everything is clean, but believe it or not, a directory "-p" is created, too. In addition, the Windows command exits with an error if the directories are already existing.

[2353 ms] Start: Run: C:\WINDOWS\system32\cmd.exe /c mkdir -p C:\Users<user>.cache\bazel
Ein Unterverzeichnis oder eine Datei mit dem Namen "C:\Users<user>.cache\bazel
" existiert bereits.
Folgender Fehler trat auf: C:\Users<user>.cache\bazel.
[2585 ms] Error: Command failed: C:\WINDOWS\system32\cmd.exe /c mkdir -p C:\Users<user>.cache\bazel

[AlexanderLanin]

Maybe this is better fix? Someone with more docker knowledge has to comment :)

"mounts": [
  "source=${localEnv:HOME}/.cache/bazel,target=/home/vscode/.cache/bazel,type=bind"
]

[lurtz]

@bl81hi I pushed an update to the branch. If this does not work, I need your help. I have no Windows available to test it.

[lurtz]

Maybe this is better fix? Someone with more docker knowledge has to comment :)

"mounts": [
  "source=${localEnv:HOME}/.cache/bazel,target=/home/vscode/.cache/bazel,type=bind"
]

This is already done the reason we need to be sure the directory exists: https://github.com/eclipse-score/devcontainer/blob/a73b33534cef30aea5257b8669acb0831a6d0191/src/s-core-devcontainer/.devcontainer/s-core-local/devcontainer-feature.json#L34-L39

[bl81hi]

@bl81hi I pushed an update to the branch. If this does not work, I need your help. I have no Windows available to test it.

Great job, seems to work now at my place, maybe someone else makes some tests, too.

[lurtz]

@bl81hi Can you then approve the PR? This helps the formal reviewers

[LittleHuba]

Fix is merged.