[Notes] [Git][BuildStream/buildstream][jmac/virtual_directories] sandbox.py: Use _root in _has_command

[Jim MacArthur <gitlab mg gitlab com>]

Title: GitLab

Jim MacArthur pushed to branch jmac/virtual_directories at BuildStream / buildstream

Commits:

• c18d158b
  by Jim MacArthur at 2018-07-27T15:13:17Z

  sandbox.py: Use _root in _has_command
  
  This will need more attention when we bring in another virtual
  directory backend, however, we've said it is acceptable for the
  sandbox itself to access the underlying directory, and this is
  the best fix in the meantime.
  

1 changed file:

• buildstream/sandbox/sandbox.py

Changes:

buildstream/sandbox/sandbox.py

---

...	...	@@ -329,11 +329,11 @@ class Sandbox():
329	329	def _has_command(self, command, env=None):
330	330	if os.path.isabs(command):
331	331	return os.path.exists(os.path.join(
332		- self.get_directory(), command.lstrip(os.sep)))
	332	+ self._root, command.lstrip(os.sep)))
333	333
334	334	for path in env.get('PATH').split(':'):
335	335	if os.path.exists(os.path.join(
336		- self.get_directory(), path.lstrip(os.sep), command)):
	336	+ self._root, path.lstrip(os.sep), command)):
337	337	return True
338	338
339	339	return False

—
View it on GitLab.
You're receiving this email because of your account on gitlab.com. If you'd like to receive fewer emails, you can adjust your notification settings.