Skip to content
This repository has been archived by the owner on Jul 29, 2024. It is now read-only.

Windows doc + scripts #9

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

maxime-gueriau
Copy link

No description provided.

@maxime-gueriau maxime-gueriau requested a review from lcodeca as a code owner August 7, 2020 09:09
IMAGE_FOLDER="docker-image"
GPU=true
GPU_OPT="--gpus all"
OPTIRUN=false
Copy link
Owner

Choose a reason for hiding this comment

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

optirun does not exist for Mac OSX

if [[ "$CACHE" = true ]]; then
echo "Building the docker container using the cache, if present."
$OPTIRUN_OPT docker build \
--build-arg USER_ID=$(id -u ${USER}) \
Copy link
Owner

Choose a reason for hiding this comment

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

This is how the parameters are passed to the Dockerfile.
You can add a command-line parameter to read them from the user instead of pulling them from the system like it's done in Linux.

#USER alice

# Working user
RUN groupadd --gid 197121 alice && \
Copy link
Owner

Choose a reason for hiding this comment

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

UID and GID should not be hardcoded.


3. build the docker image using the docker cmd bash script

`./docker-cmd-windows.sh -n=marl-parking-devel -b -c -r`
Copy link
Owner

Choose a reason for hiding this comment

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

This was from our specific project, it should be more generic in a howto.


5. use the following command line to run your docker image:

(of course replace "GroupID, UserID and FolderPath" accordingly
Copy link
Owner

Choose a reason for hiding this comment

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

Can this be embedded in docker-cmd-windows.sh with

  • DISPLAY
  • UID & GID
    as command-line parameters?

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.

2 participants