actually do we need this at all?  this is either immediate (as so probably shouldn't be a task) or it does the `ssh` -- in which case it is already setting up tasks two levels deep (the `BasicMachineDetails.forSshMachineLocation()` which in turn wraps an `ssh` task) and wrapping in a third level is ugly.

suggest the callers just do `sshMachineLocation.getMachineDetails()` (or `BasicMachineDetails.forSshMachineLocation(...)` if they really need a task) and drop this class altogether