moodle-docker
moodle-docker copied to clipboard
Set the path to python
Thanks for commenting Matteo. I've applied the suggested change.
To be honest, I've always seen paths to binaries specified using the full path, but I have never understood what is the advantage of it over just using the file name. I do understand that the difference when the script using just the bin file name does not know the system where it will be executed, but in this case we do.
TNX @dmonllao!
Using the absolute path is cleaner (IMHO), safer - the main reason, even if running in a container where layers are known - and quicker; otherwise, the system will look up in what defined in an environment variable called PATH
following its ordering, being the env defined by that user context.
HTH, Matteo
:+1: to full path
:+1: thanks for the explanation Matteo. The patch was already updated using the full path so no further changes are required.
Any chance that we could merge this? The patch does not seem problematic.
The failures reported by travis are unrelated to this patch.