-
Notifications
You must be signed in to change notification settings - Fork 6
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 image is out of date. #67
Comments
I believe this was an experimental feature that was never officially supported |
@afdaniele that is a very old version of the shell, right? |
@david-boles where did you get the instructions about the |
@afdaniele They might have got the instructions here: https://hub.docker.com/r/duckietown/duckietown-shell, in the |
@david-boles In case you still want to run your command, you may try using my image: tl;dr Below are the steps I did to create the above image (just for reference). Could have used a Dockerfile, but not sure if all of this is proper. I figured that the duckietown-shell docker image has python2, so it's not easy to directly use that image to upgrade dts. But luckily, it seems creating a new image is not hard. For reference, here are the steps I did to create the docker image (with reference to this Dockerfile duckietown/duckietown-shell's Dockerfile on Docker Hub)
, and then
|
@jasonhu5 Thanks! We'll try it at some point. @afdaniele We just got the Docker command from the README. See #66 for why we had trouble running it on Windows directly, though not installing it from Powershell might have been the issue (besides not being able to install docker at all on some computers). |
Running
docker run -it --rm -v /var/run/docker.sock:/var/run/docker.sock -w $PWD -v $PWD:$PWD -v ~/.dt-shell:/root/.dt-shell -v ~/.docker:/root/.docker duckietown/duckietown-shell:latest dts
gives:The text was updated successfully, but these errors were encountered: