[buildstream] branch Qinusty/470-bst-track-yaml-indent created (now 9e68388)

[no...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

not-in-ldap pushed a change to branch Qinusty/470-bst-track-yaml-indent
in repository https://gitbox.apache.org/repos/asf/buildstream.git.


      at 9e68388  Fixes #470: tracking modifies elements

This branch includes the following new commits:

     new 9e68388  Fixes #470: tracking modifies elements

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.

[buildstream] 01/01: Fixes #470: tracking modifies elements

[no...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

not-in-ldap pushed a commit to branch Qinusty/470-bst-track-yaml-indent
in repository https://gitbox.apache.org/repos/asf/buildstream.git

commit 9e6838882cecfe8a3e5247a4905018e76db9fa44
Author: Josh Smith <jo...@codethink.co.uk>
AuthorDate: Tue Aug 7 16:44:14 2018 +0100

    Fixes #470: tracking modifies elements
    
    This fixes the issue by preventing the nodes being sorted within the
    toplevel node prior to being dumped to file.
    
    HOWEVER: This breaks the interaction with include functionality and storing
    refs inline.
---
 buildstream/source.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/buildstream/source.py b/buildstream/source.py
index d58bfe2..24acb68 100644
--- a/buildstream/source.py
+++ b/buildstream/source.py
@@ -752,7 +752,7 @@ class Source(Plugin):
                 # Save the ref in the originating file
                 #
                 try:
-                    _yaml.dump(_yaml.node_sanitize(provenance.toplevel), provenance.filename.name)
+                    _yaml.dump(provenance.toplevel, provenance.filename.name)
                 except OSError as e:
                     raise SourceError("{}: Error saving source reference to '{}': {}"
                                       .format(self, provenance.filename.name, e),