[Notes] [Git][BuildStream/buildstream][aevri/affect] 3 commits: Fix pyth

Angelos Evripiotis pushed to branch aevri/affect at BuildStream / buildstream

Commits:

f1aa0974

by Javier Jardón at 2018-10-18T14:13:35Z

Fix python warnings: Use collections.abc instead collections

Since python 3.3, collections has been moved to collections.abc module.
For backwards compatibility, they continue to be visible in this module through Python 3.7.
Subsequently, they will be removed entirely.

See https://docs.python.org/3/library/collections.html

f1550c8e

by Valentin David at 2018-10-18T16:02:45Z

Merge branch 'jjardon/warnings_collections' into 'master'

Fix python warnings: Use collections.abc instead collections

See merge request BuildStream/buildstream!883

1c24979a

by Angelos Evripiotis at 2018-10-19T21:17:01Z

Spelling fixes: 'affect', not 'effect' as verb

Fix most instances of 'effect' being used as a verb instead of a noun.
Skipped generated instance in doc/source/conf.py.

22 changed files:

buildstream/_artifactcache/artifactcache.py
buildstream/_context.py
buildstream/_includes.py
buildstream/_loader/loadelement.py
buildstream/_loader/loader.py
buildstream/_options/optionpool.py
buildstream/_project.py
buildstream/_scheduler/queues/queue.py
buildstream/_yaml.py
buildstream/buildelement.py
buildstream/element.py
buildstream/plugin.py
buildstream/plugins/elements/autotools.yaml
buildstream/plugins/elements/cmake.yaml
buildstream/plugins/elements/junction.py
buildstream/plugins/elements/make.yaml
buildstream/plugins/elements/manual.yaml
buildstream/plugins/elements/meson.yaml
buildstream/plugins/elements/qmake.yaml
buildstream/plugins/sources/git.py
buildstream/source.py
tests/yaml/yaml.py

Changes:

buildstream/_artifactcache/artifactcache.py

@@ -19,7 +19,8 @@
  import os
  import string
 -from collections import Mapping, namedtuple
 +from collections import namedtuple
 +from collections.abc import Mapping
  from ..types import _KeyStrength
  from .._exceptions import ArtifactError, ImplError, LoadError, LoadErrorReason

buildstream/_context.py

@@ -19,7 +19,8 @@
  import os
  import datetime
 -from collections import deque, Mapping
 +from collections import deque
 +from collections.abc import Mapping
  from contextlib import contextmanager
  from . import utils
  from . import _cachekey

buildstream/_includes.py

  import os
 -from collections import Mapping
 +from collections.abc import Mapping
  from . import _yaml
  from ._exceptions import LoadError, LoadErrorReason

buildstream/_loader/loadelement.py

@@ -18,7 +18,7 @@
  #        Tristan Van Berkom <tristan vanberkom codethink co uk>
  # System imports
 -from collections import Mapping
 +from collections.abc import Mapping
  # BuildStream toplevel imports
  from .._exceptions import LoadError, LoadErrorReason

buildstream/_loader/loader.py

@@ -19,7 +19,8 @@
  import os
  from functools import cmp_to_key
 -from collections import Mapping, namedtuple
 +from collections import namedtuple
 +from collections.abc import Mapping
  import tempfile
  import shutil

buildstream/_options/optionpool.py

@@ -18,7 +18,7 @@
  #        Tristan Van Berkom <tristan vanberkom codethink co uk>
+ #
 -from collections import Mapping
 +from collections.abc import Mapping
  import jinja2
  from .. import _yaml

buildstream/_project.py

@@ -19,7 +19,8 @@
  #        Tiago Gomes <tiago gomes codethink co uk>
  import os
 -from collections import Mapping, OrderedDict
 +from collections import OrderedDict
 +from collections.abc import Mapping
  from pluginbase import PluginBase
  from . import utils
  from . import _cachekey

buildstream/_scheduler/queues/queue.py


     # This will have different results for elements depending
     # on the Queue.status() implementation.
     #
-    #   o Elements which are QueueStatus.WAIT will not be effected
+    #   o Elements which are QueueStatus.WAIT will not be affected
     #
     #   o Elements which are QueueStatus.SKIP will move directly
     #     to the dequeue pool

buildstream/_yaml.py


 #
 # The purpose of this is to create a virtual copy-on-write
 # copy of a dictionary, so that mutating it in any way does
-# not effect the underlying dictionaries.
+# not affect the underlying dictionaries.
 #
 # collections.ChainMap covers this already mostly, but fails
 # to record internal state so as to hide keys which have been

buildstream/buildelement.py

@@ -176,7 +176,7 @@ class BuildElement(Element):
          # Specifying notparallel for a given element effects the
          # cache key, while having the side effect of setting max-jobs to 1,
 -        # which is normally automatically resolved and does not effect
 +        # which is normally automatically resolved and does not affect
          # the cache key.
          if self.get_variable('notparallel'):
              dictionary['notparallel'] = True

buildstream/element.py

@@ -76,7 +76,8 @@ import os
  import re
  import stat
  import copy
 -from collections import Mapping, OrderedDict
 +from collections import OrderedDict
 +from collections.abc import Mapping
  from contextlib import contextmanager
  import tempfile
  import shutil

buildstream/plugin.py

@@ -266,7 +266,7 @@ class Plugin():
          such as an sha256 sum of a tarball content.
          Elements and Sources should implement this by collecting any configurations
 -        which could possibly effect the output and return a dictionary of these settings.
 +        which could possibly affect the output and return a dictionary of these settings.
          For Sources, this is guaranteed to only be called if
          :func:`Source.get_consistency() <buildstream.source.Source.get_consistency>`

buildstream/plugins/elements/autotools.yaml

@@ -123,7 +123,7 @@ environment:
    V: 1
  # And dont consider MAKEFLAGS or V as something which may
 -# effect build output.
 +# affect build output.
  environment-nocache:
  - MAKEFLAGS
  - V

buildstream/plugins/elements/cmake.yaml

@@ -66,7 +66,7 @@ environment:
    V: 1
  # And dont consider JOBS or V as something which may
 -# effect build output.
 +# affect build output.
  environment-nocache:
  - JOBS
  - V

buildstream/plugins/elements/junction.py

@@ -124,7 +124,7 @@ the user to resolve possibly conflicting nested junctions by creating a junction
  with the same name in the top-level project, which then takes precedence.
  """
 -from collections import Mapping
 +from collections.abc import Mapping
  from buildstream import Element
  from buildstream._pipeline import PipelineError

buildstream/plugins/elements/make.yaml

@@ -36,7 +36,7 @@ environment:
    V: 1
  # And dont consider MAKEFLAGS or V as something which may
 -# effect build output.
 +# affect build output.
  environment-nocache:
  - MAKEFLAGS
  - V

buildstream/plugins/elements/manual.yaml

@@ -35,7 +35,7 @@ environment:
    V: 1
  # And dont consider MAKEFLAGS or V as something which may
 -# effect build output.
 +# affect build output.
  environment-nocache:
  - MAKEFLAGS
  - V

buildstream/plugins/elements/meson.yaml

@@ -74,6 +74,6 @@ environment:
      %{max-jobs}
  # And dont consider NINJAJOBS as something which may
 -# effect build output.
 +# affect build output.
  environment-nocache:
  - NINJAJOBS

buildstream/plugins/elements/qmake.yaml

@@ -44,7 +44,7 @@ environment:
    V: 1
  # And dont consider MAKEFLAGS or V as something which may
 -# effect build output.
 +# affect build output.
  environment-nocache:
  - MAKEFLAGS
  - V

buildstream/plugins/sources/git.py

@@ -89,7 +89,7 @@ import os
  import errno
  import re
  import shutil
 -from collections import Mapping
 +from collections.abc import Mapping
  from io import StringIO
  from configparser import RawConfigParser
@@ -415,7 +415,7 @@ class GitSource(Source):
      def get_unique_key(self):
          # Here we want to encode the local name of the repository and
          # the ref, if the user changes the alias to fetch the same sources
 -        # from another location, it should not effect the cache key.
 +        # from another location, it should not affect the cache key.
          key = [self.original_url, self.mirror.ref]
          # Only modify the cache key with checkout_submodules if it's something

buildstream/source.py

@@ -155,7 +155,7 @@ Class Reference
  """
  import os
 -from collections import Mapping
 +from collections.abc import Mapping
  from contextlib import contextmanager
  from . import Plugin, Consistency

tests/yaml/yaml.py

  import os
  import pytest
  import tempfile
 -from collections import Mapping
 +from collections.abc import Mapping
  from buildstream import _yaml
  from buildstream._exceptions import LoadError, LoadErrorReason

...	...	@@ -208,7 +208,7 @@ class Queue():
208	208	# This will have different results for elements depending
209	209	# on the Queue.status() implementation.
210	210	#
211		- # o Elements which are QueueStatus.WAIT will not be effected
	211	+ # o Elements which are QueueStatus.WAIT will not be affected
212	212	#
213	213	# o Elements which are QueueStatus.SKIP will move directly
214	214	# to the dequeue pool

...	...	@@ -972,7 +972,7 @@ def node_validate(node, valid_keys):
972	972	#
973	973	# The purpose of this is to create a virtual copy-on-write
974	974	# copy of a dictionary, so that mutating it in any way does
975		-# not effect the underlying dictionaries.
	975	+# not affect the underlying dictionaries.
976	976	#
977	977	# collections.ChainMap covers this already mostly, but fails
978	978	# to record internal state so as to hide keys which have been

[Notes] [Git][BuildStream/buildstream][aevri/affect] 3 commits: Fix python warnings: Use collections.abc instead collections

Angelos Evripiotis pushed to branch aevri/affect at BuildStream / buildstream

Commits:

22 changed files:

Changes: