Code review comment for ~info-martin-konrad/epics-base:fix-broken-epicshostarch-script

Revision history for this message
Andrew Johnson (anj) wrote :

I will change the file permissions to be 0755, but since the arguments to EpicsHostArch.pl should never contain spaces there is no real difference between $* and "$@" here. If anything using $* is slightly safer since the final target name would get a '-' instead of any spaces, but the right place to remove spaces would actually be in the Perl script, and either way I don't think anyone is ever likely to hit this problem.

See https://unix.stackexchange.com/questions/41571/what-is-the-difference-between-and for a nice description of how $* and "$@" differ.

« Back to merge proposal