[engine] undefined names in BUILD files trigger a double trace

[baroquebobcat]

I was looking at #4528, and wondered if v2 looked okay with when it hit an undefined name. I think it could be better

v1

$ ./pants --no-print-exception-stacktrace --no-enable-v2-engine list :
Exception caught: (<class 'pants.build_graph.address_lookup_error.AddressLookupError'>)

Exception message: name 'undeclared' is not defined
 while executing BUILD file BuildFile(BUILD.tools, FileSystemProjectTree(/.../pants))
 Loading addresses from '' failed.

v2

$ ./pants --no-print-exception-stacktrace list :
Exception caught: (<class 'pants.build_graph.address_lookup_error.AddressLookupError'>)

Exception message: Build graph construction failed for (SiblingAddresses(directory=u''), Select(Collection.of(HydratedTarget))):
Computing Select(SiblingAddresses(directory=u''), =Collection.of(HydratedTarget))
  Computing Task(<function transitive_hydrated_targets at 0x102116ed8>, SiblingAddresses(directory=u''), =Collection.of(HydratedTarget))
    Computing TransitiveDependencies( =HydratedTarget, =BuildFileAddresses)
      Computing Task(<function addresses_from_address_families at 0x101f771b8>, SiblingAddresses(directory=u''), =BuildFileAddresses)
        Computing Task(<function parse_address_family at 0x101f746e0>, Dir(path=''), =AddressFamily)
          Throw(Failed to parse BUILD.tools:
name 'undeclared' is not defined)

Computing Select(SiblingAddresses(directory=u''), =BuildFileAddresses)
  Computing Task(<function addresses_from_address_families at 0x101f771b8>, SiblingAddresses(directory=u''), =BuildFileAddresses)
    Computing Task(<function parse_address_family at 0x101f746e0>, Dir(path=''), =AddressFamily)
      Throw(Failed to parse BUILD.tools:
name 'undeclared' is not defined)

[stuhood]

Related to #3695

... or perhaps just a dupe?

[stuhood]

Dupe.