Remote exec: Remove early warning and check directory is not None

Jim MacArthurrequested to merge
jmac/remote_exec_checkout_fix into master

Description

Fix for #653 (closed), which should allow bst checkout to work again with remote execution.

Proposed changes

Changes proposed in this merge request:

  • Remove early warning check which attempted to warn the user if bst was about to use remote execution with no remotes set
  • Don't use SandboxRemote if no directory is passed to Element.__sandbox.

Fixes

This merge request, when approved, will close issue/bug: #653 (closed)

Caveats

This error occurs because we're using SandboxRemote for checkouts. There aren't many reasons to do this, and a better way to do it would be to alter Element.__sandbox such that it uses platform.create_sandbox(). If we do this, we can keep the early warning. However, I don't yet know how to determine if we're being used for a checkout.

Merge request reports