nielsbasjes commented on a change in pull request #12837:
URL: https://github.com/apache/beam/pull/12837#discussion_r496657647



##########
File path: website/www/site/content/en/contribute/_index.md
##########
@@ -94,6 +94,11 @@ $ go get github.com/linkedin/goavro
 
 gLinux users should configure their machines for sudoless Docker.
 
+Alternatively, you can use the 
[Dockerfile](https://github.com/apache/beam/blob/master/Dockerfile) to setup a 
container meeting the requirements above, and mounting the directory that 
contains the Beam repo to the container. e.g.:
+```shell script
+docker run -it --mount 
type=bind,source=/Users/jsmith/Desktop/beam,target=/workspaces/beam 
<CONTAINER_ID> bash
+```

Review comment:
       I would prefer a script similar (same) as what is present in the Hadoop 
project:
   https://github.com/apache/hadoop/blob/trunk/start-build-env.sh
   It makes starting/building the development environment for the developers a 
lot easier as it also mounts the working directory and has fixes and 
workarounds for various situations (like selinux, MacOS, etc.)




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to