[Notes] [Git][BuildStream/buildstream][master] 2 commits: buildstream/_yaml.py: Import Mapping from collections.abc

[Jürg Billeter <gitlab mg gitlab com>]

Title: GitLab

Jürg Billeter pushed to branch master at BuildStream / buildstream

Commits:

• fca37d0d
  by Chandan Singh at 2019-02-19T06:37:04Z

  buildstream/_yaml.py: Import Mapping from collections.abc
  
  Instead of importing `Mapping` from `collections`, it must be imported
  from `collections.abc`, as the old method will stop working from Python
  3.8.
  

• 5f0571d1
  by Jürg Billeter at 2019-02-19T07:37:21Z

  Merge branch 'chandan/fix-mapping-import' into 'master'
  
  buildstream/_yaml.py: Import Mapping from collections.abc
  
  See merge request BuildStream/buildstream!1161

1 changed file:

• buildstream/_yaml.py

Changes:

buildstream/_yaml.py

---

...	...	@@ -940,7 +940,7 @@ def node_sanitize(node):
940	940	return [node_sanitize(elt) for elt in node]
941	941
942	942	# Finally ChainMap and dict, and other Mappings need special handling
943		- if node_type in (dict, ChainMap) or isinstance(node, collections.Mapping):
	943	+ if node_type in (dict, ChainMap) or isinstance(node, collections.abc.Mapping):
944	944	result = SanitizedDict()
945	945
946	946	key_list = [key for key, _ in node_items(node)]

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