<div style="white-space:pre-wrap">Ignore this, Paul has a pending patch for it. </div><br><div class="gmail_quote"><div dir="ltr">On Tue, Nov 17, 2015 at 10:42 AM Christopher Larson &lt;<a href="mailto:kergoth@gmail.com">kergoth@gmail.com</a>&gt; wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">From: Christopher Larson &lt;<a href="mailto:chris_larson@mentor.com" target="_blank">chris_larson@mentor.com</a>&gt;<br>
<br>
We don&#39;t want the user to see errors from the git commands run by metadata_scm<br>
on their console, so we need to capture or suppress stderr as well as stdout.<br>
This was the case prior to the rewrite of the git hash logic, but the 2&gt;&amp;1 was<br>
lost when it was reworked. Bring it back to avoid messages like this in builds<br>
with non-git layers:<br>
<br>
    fatal: Not a git repository (or any of the parent directories): .git<br>
<br>
Cc: Ross Burton &lt;<a href="mailto:ross.burton@intel.com" target="_blank">ross.burton@intel.com</a>&gt;<br>
Signed-off-by: Christopher Larson &lt;<a href="mailto:chris_larson@mentor.com" target="_blank">chris_larson@mentor.com</a>&gt;<br>
---<br>
 meta/classes/metadata_scm.bbclass | 4 ++--<br>
 1 file changed, 2 insertions(+), 2 deletions(-)<br>
<br>
diff --git a/meta/classes/metadata_scm.bbclass b/meta/classes/metadata_scm.bbclass<br>
index 64465fa..a8152f7 100644<br>
--- a/meta/classes/metadata_scm.bbclass<br>
+++ b/meta/classes/metadata_scm.bbclass<br>
@@ -69,7 +69,7 @@ def base_get_metadata_git_branch(path, d):<br>
<br>
     try:<br>
         return subprocess.check_output([&quot;git&quot;, &quot;rev-parse&quot;, &quot;--abbrev-ref&quot;, &quot;HEAD&quot;],<br>
-                                       cwd=path).strip()<br>
+                                       cwd=path, stderr=subprocess.STDOUT).strip()<br>
     except:<br>
         return &quot;&lt;unknown&gt;&quot;<br>
<br>
@@ -78,6 +78,6 @@ def base_get_metadata_git_revision(path, d):<br>
<br>
     try:<br>
         return subprocess.check_output([&quot;git&quot;, &quot;rev-parse&quot;, &quot;HEAD&quot;],<br>
-                                       cwd=path).strip()<br>
+                                       cwd=path, stderr=subprocess.STDOUT).strip()<br>
     except:<br>
         return &quot;&lt;unknown&gt;&quot;<br>
--<br>
2.2.1<br>
<br>
</blockquote></div>