run_shell_cmd hooks should be applied earlier

[Micket]

The hooks for run_shell_cmd

easybuild-framework/easybuild/tools/run.py

Lines 311 to 317 in 24a0f8a

	if with_hooks:
	hooks = load_hooks(build_option('hooks'))
	hook_res = run_hook(RUN_SHELL_CMD, hooks, pre_step_hook=True, args=[cmd], kwargs={'work_dir': work_dir})
	if hook_res:
	cmd, old_cmd = hook_res, cmd
	cmd_str = to_cmd_str(cmd)
	_log.info("Command to run was changed by pre-%s hook: '%s' (was: '%s')", RUN_SHELL_CMD, cmd, old_cmd)

are applied just before we run the subprocess, but after all of logging, dry-run etc. cmd_str (before hooks) is used in a bunch of places.

I would argue it should be done as early as possible, since it would reflect the true command being run.