failing test for setgid
only chown if we have to

We might want to just get rid of this chown and rely on setgid, etc. to
control this.  In fact, the reason to make the chown only execute if
needed is because it removes setgid on root_squashed NFS volumes.
(sometimes) fix setgid being removed by Genome::Sys->create_directory