Merge "web: console -- clarify package: comment"

1 files changed, 7 insertions, 7 deletions

diff --git a/web/src/containers/build/Console.jsx b/web/src/containers/build/Console.jsx
index 94c5f6f07..35a7b0a04 100644
--- a/web/src/containers/build/Console.jsx
+++ b/web/src/containers/build/Console.jsx
@@ -64,13 +64,13 @@ class TaskOutput extends React.Component {
   renderResults(value) {
     const interesting_results = []
 
-    // This was written to assume "value" is an array of
-    // key/value mappings to output.  This seems to be a
-    // good assumption for the most part, but "package:" for
-    // whatever reason outputs a result that is just an array of
-    // strings with what packages were installed.  So, if we
-    // see an array of strings as the value, we just swizzle
-    // that into a key/value so it displays usefully.
+    // This was written to assume "value" is an array of key/value
+    // mappings to output.  This seems to be a good assumption for the
+    // most part, but "package:" on at least some distros --
+    // RedHat/yum/dnf we've found -- outputs a result that is just an
+    // array of strings with what packages were installed.  So, if we
+    // see an array of strings as the value, we just swizzle that into
+    // a key/value so it displays usefully.
     const isAllStrings = value.every(i => typeof i === 'string')
     if (isAllStrings) {
       value = [ {output: [...value]} ]