[Notes] [Git][BuildStream/buildstream][master] 2 commits: Check if command is a str and replace by list before checking existence

[Benjamin Schubert <gitlab mg gitlab com>]

Title: GitLab

Benjamin Schubert pushed to branch master at BuildStream / buildstream

Commits:

• 80762ecb
  by Benjamin Schubert at 2018-10-26T09:45:39Z

  Check if command is a str and replace by list before checking existence
  
  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.
  

• f131c407
  by Benjamin Schubert at 2018-10-26T10:09:51Z

  Merge branch 'bschubert/fix-command-sandbox' into 'master'
  
  Check is command is a str and replace by list before checking existence
  
  Closes #728
  
  See merge request BuildStream/buildstream!898

3 changed files:

• buildstream/sandbox/_sandboxbwrap.py
• buildstream/sandbox/_sandboxchroot.py
• buildstream/sandbox/_sandboxdummy.py

Changes:

buildstream/sandbox/_sandboxbwrap.py

---

...	...	@@ -66,15 +66,15 @@ class SandboxBwrap(Sandbox):
66	66	cwd = self._get_work_directory(cwd=cwd)
67	67	env = self._get_environment(cwd=cwd, env=env)
68	68
	69	+ # Convert single-string argument to a list
	70	+ if isinstance(command, str):
	71	+ command = [command]
	72	+
69	73	if not self._has_command(command[0], env):
70	74	raise SandboxError("Staged artifacts do not provide command "
71	75	"'{}'".format(command[0]),
72	76	reason='missing-command')
73	77
74		- # We want command args as a list of strings
75		- if isinstance(command, str):
76		- command = [command]
77		-
78	78	# Create the mount map, this will tell us where
79	79	# each mount point needs to be mounted from and to
80	80	mount_map = MountMap(self, flags & SandboxFlags.ROOT_READ_ONLY)

buildstream/sandbox/_sandboxchroot.py

---

...	...	@@ -57,15 +57,15 @@ class SandboxChroot(Sandbox):
57	57	cwd = self._get_work_directory(cwd=cwd)
58	58	env = self._get_environment(cwd=cwd, env=env)
59	59
	60	+ # Convert single-string argument to a list
	61	+ if isinstance(command, str):
	62	+ command = [command]
	63	+
60	64	if not self._has_command(command[0], env):
61	65	raise SandboxError("Staged artifacts do not provide command "
62	66	"'{}'".format(command[0]),
63	67	reason='missing-command')
64	68
65		- # Command must be a list
66		- if isinstance(command, str):
67		- command = [command]
68		-
69	69	stdout, stderr = self._get_output()
70	70
71	71	# Create the mount map, this will tell us where

buildstream/sandbox/_sandboxdummy.py

---

...	...	@@ -33,6 +33,10 @@ class SandboxDummy(Sandbox):
33	33	cwd = self._get_work_directory(cwd=cwd)
34	34	env = self._get_environment(cwd=cwd, env=env)
35	35
	36	+ # Convert single-string argument to a list
	37	+ if isinstance(command, str):
	38	+ command = [command]
	39	+
36	40	if not self._has_command(command[0], env):
37	41	raise SandboxError("Staged artifacts do not provide command "
38	42	"'{}'".format(command[0]),

—
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.