Update Dockerfile and install.sh

umccr / umccrise

:snake: DRAGEN Tumor/Normal workflow post-processing

https://umccr.github.io/umccrise/

MIT License

22 stars 8 forks source link

Update Dockerfile and install.sh #68

Closed scwatts closed 3 years ago

scwatts commented 3 years ago

These changes resolve the numerous warnings Snakemake emits wrt input paths containing double slashes '//'. The default root user in the Docker container has also been replaced with a non-root user.

Change summary:

avoid use of the HOME environment variable in Dockerfile
use a non-root user in the Docker container
forcefully set HOME to non-root user home dir in Docker container
normalise base install path by stripping leading '//' in install.sh

/cc @brainstorm @alexiswl @ohofmann

scwatts commented 3 years ago

Good rebuild for the Docker image and had a successful manual run on an AWS instance using the test bcbio project from https://github.com/umccr/umccrise_test_data.