[Notes] [Git][BuildStream/buildstream][aevri/empty

Angelos Evripiotis pushed to branch aevri/empty_manual 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.

43a2eee3

by Angelos Evripiotis at 2018-10-19T21:48:59Z

Merge branch 'aevri/affect' into 'master'

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

See merge request BuildStream/buildstream!885

28e96b18

by Angelos Evripiotis at 2018-10-20T10:21:26Z

BREAK: manual.yaml: don't set any default env vars

Remove the 'MAKEFLAGS' and 'V' defaults.

Now that there is a specialised 'make' element, these make-specific
defaults don't make sense here. This element is meant to be a blank
slate for folks to build on.

Note that this is a breaking change, that will require some users to
make changes to their .bst files if they are expecting these environment
variable to be set.

_versions.py: BST_FORMAT_VERSION bumped to 18.

24 changed files:

NEWS
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/_versions.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:

NEWS

@@ -2,6 +2,12 @@
  buildstream 1.3.1
  =================
 +  o BREAKING CHANGE: The 'manual' element lost its default 'MAKEFLAGS' and 'V'
 +    environment variables. There is already a 'make' element with the same
 +    variables. Note that this is a breaking change, it will require users to
 +    make changes to their .bst files if they are expecting these environment
 +    variable to be set.
++
    o Failed builds are included in the cache as well.
      `bst checkout` will provide anything in `%{install-root}`.
      A build including cached fails will cause any dependant elements

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/_versions.py

@@ -23,7 +23,7 @@
  # This version is bumped whenever enhancements are made
  # to the `project.conf` format or the core element format.
+ #
 -BST_FORMAT_VERSION = 17
 +BST_FORMAT_VERSION = 18
  # The base BuildStream artifact version

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

 -# No variables added for the manual element by default, set
 -# this if you plan to use make, and the sources cannot handle
 -# parallelization.
 -#
 -# variables:
 -#
 -#   notparallel: True
+-
  # Manual build element does not provide any default
  # build commands
  config:
@@ -28,14 +20,3 @@ config:
    strip-commands:
    - |
      %{strip-binaries}
+-
 -# Use max-jobs CPUs for building and enable verbosity
 -environment:
 -  MAKEFLAGS: -j%{max-jobs}
 -  V: 1
+-
 -# And dont consider MAKEFLAGS or V as something which may
 -# effect 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/empty_manual] 5 commits: Fix python warnings: Use collections.abc instead collections

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

Commits:

24 changed files:

Changes: