DvorakDwarf / Infinite-Storage-Glitch

ISG lets you use YouTube as cloud storage for ANY files, not just video
GNU General Public License v3.0
11.43k stars 910 forks source link

Windows docker requires "%cd%" instead of ${PWD} #61

Closed Shyvadi closed 1 year ago

beyse commented 1 year ago

This is interesting. Perhaps there is a difference in our Windows systems 🤔

I have Windows 11 and I am using WSL2. In my Docker Desktop app I have Enabled WSL Integration for my Ubuntu distro.

When I run

docker run -it --rm -v ${PWD}:/home/Infinite-Storage-Glitch isg cargo build --release

from my PowerShell (not inside a WSL shell) it works just fine. If you share your environment, perhaps we can find why it works differently.

Shyvadi commented 1 year ago

This is interesting. Perhaps there is a difference in our Windows systems 🤔

I have Windows 11 and I am using WSL2. In my Docker Desktop app I have Enabled WSL Integration for my Ubuntu distro.

When I run

docker run -it --rm -v ${PWD}:/home/Infinite-Storage-Glitch isg cargo build --release

from my PowerShell (not inside a WSL shell) it works just fine. If you share your environment, perhaps we can find why it works differently.

I use Windows 11 as well and am using Docker Desktop downloaded fresh. I am running it without any ubuntu subsystem enabled if that's what you are referring to.

image image

beyse commented 1 year ago

I guess it is probably easiest to update the readme file to point this out.

ayhamo commented 1 year ago

i would like to add that if you run the command from powershell, it will work, but cmd will not

beyse commented 1 year ago

i would like to add that if you run the command from powershell, it will work, but cmd will not

I added it as part of #72

beyse commented 1 year ago

Since #72 is merged we can close this.