Terminology correction
Make legacy turn on when needed

Originally legacy results would be on by default, but turn off when
'modern' was used. This makes no sense. Instead we have it off by
default and turn it on when a non-modern builder is used. This avoids
bad interactions
fix a test