Benjamin Schubert authored Oct 25, 2018 and Benjamin Schubert committed Oct 26, 2018

Currently we would make sure the sandbox had a command before
converting it to a list if it was given as a string. That meant that a string
command would never exist and the check be invalid.

This also adds the same logic in the dummy sandbox for consistency.

Check is command is a str and replace by list before checking existence

Closes #728

See merge request !898

Daniel Silverstone authored Oct 26, 2018 and Phil Dawson committed Oct 26, 2018

In cases where elements end up with large numbers of dependencies,
the n-squared performance in `_extract_depends_from_node` can produce
significant delays during project load.  This patch corrects that.
In addition we were double-retrieving provenance which was unnecessary.

Signed-off-by: Daniel Silverstone <daniel.silverstone@codethink.co.uk>