Fix #283 - Refactor rsync to only use absolute paths

So it's always an array with exactly 2 elements?  Why do we use an array in that case?  Why don't we have an object with two properties?  Seems an odd design choice, since you can't name the two properties.