Perhaps this should be better an abort().  So the user is not tempted to believe that setting the revision worked.