Fix race condition when launching websockify.

[xavierog]

Description

Fix for the race condition thoroughly investigated in issue #18 Summary of issue #18: noVNC's launch.sh invokes ps. Without it, it exits prematurely, thus ruining websockify's stdout and stderr file descriptors. Depending on how fast websockify starts up, this may or may not prevent it from working.

Types of changes

• [x] Bug fix (non-breaking change which fixes an issue)
• [ ] New feature (non-breaking change which adds functionality)
• [ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

• [x] I have read the contributing document.
• [ ] My change requires a change to the documentation.
• [ ] I have updated the documentation accordingly.
• [ ] I have added tests to cover my changes.
• [ ] All new and existing tests passed.

[jamesmortensen]

From what I can tell, this looks great, but for some reason CircleCI is using pulls/20 as the name of the branch, perhaps because it's a pull request from a forked repository.

To get the CI server to build and test the images, we'll need to replace the VERSION environment variable with something that meets the naming conventions for Docker tags. Docker tags can't have forward slashes.

If you have time, you can make that change yourself, even hard-coding "today" or "now" or "HEAD" for the VERSION environment variable should do the trick, on lines 42 and 102:

https://github.com/xavierog/docker-seleniarm/blob/a8adc841eba0277cfe6c3efa0a82e9d49a722894/.circleci/config.yml#L42

https://github.com/xavierog/docker-seleniarm/blob/a8adc841eba0277cfe6c3efa0a82e9d49a722894/.circleci/config.yml#L102

If you're able to make those changes, then it also would fix https://github.com/seleniumhq-community/docker-seleniarm/issues/21 as well as fix https://github.com/seleniumhq-community/docker-seleniarm/issues/18

[jamesmortensen]

Thank you @xavierog