remove sudos and chmods that should not be necessary after we introduced setfacl

Author: oesteban

docker/files/run_builddocs.sh

@@ -6,6 +6,4 @@ set -u
 mkdir -p /scratch/docs
 make html 2>&1 | tee /scratch/builddocs.log
 cp -r /root/src/nipype/doc/_build/html/* /scratch/docs/
-chmod 777 -R /scratch/docs
-chmod 777 /scratch/builddocs.log
 cat /scratch/builddocs.log && if grep -q "ERROR" /scratch/builddocs.log; then false; else true; fi

docker/files/run_examples.sh

@@ -15,5 +15,3 @@ coverage run /root/src/nipype/tools/run_examples.py $@
 arr=$@
 tmp_var=$( IFS=$' '; echo "${arr[*]}" )
 coverage xml -o "/scratch/smoketest_${tmp_var//[^A-Za-z0-9_-]/_}.xml"
-
-chmod 777 -R /scratch/logs

docker/files/run_pytests.sh

@@ -36,4 +36,3 @@ fi
 
 # Copy crashfiles to scratch
 for i in $(find /root/src/nipype/ -name "crash-*" ); do cp $i /scratch/crashfiles/; done
-chmod 777 -R /scratch/*

docker/files/teardown.sh

@@ -7,9 +7,9 @@ set -u
 set -e
 
 mkdir -p ${CIRCLE_TEST_REPORTS}/pytest
-sudo mv ~/scratch/*.xml ${CIRCLE_TEST_REPORTS}/pytest
+mv ~/scratch/*.xml ${CIRCLE_TEST_REPORTS}/pytest
 mkdir -p ~/docs
-sudo mv ~/scratch/docs/* ~/docs/
+mv ~/scratch/docs/* ~/docs/
 mkdir -p ~/logs
-sudo mv ~/scratch/builddocs.log ~/logs/builddocs.log
-sudo mv ~/scratch/logs/* ~/logs/
+mv ~/scratch/builddocs.log ~/logs/builddocs.log
+mv ~/scratch/logs/* ~/logs/