<font face="Default Sans Serif,Verdana,Arial,Helvetica,sans-serif" size="2"><br>On Date: 06/22/2016 around 10:50PM, Andrew Jeffery wrote:<br>...<br>>> <br>>> <a target="_blank" href="https://github.com/openbmc/openbmc/pull/297">https://github.com/openbmc/openbmc/pull/297</a><br>>> <br>>> Brad Bishop (3):<br>>>   Add pflash parallel build fix patch<br>>>   Remove unnecessary skeleton dependencies<br>>>   Removed unused host-ipmid-bt application<br><br>These should not have been included.<br><br>>> <br>>> Milton D. Miller II (9):<br>>...<br>>> Milton Miller (6):<br>><br>>Nit pick: Probably worth reconfiguring git to use a consistent<br>>user.name.<br><br>I did that last week.  I just hadn't reset the author on the older commits.<br>Now done.<br><br>Other accepted comments in the series:<br><br><font face="Default Monospace,Courier New,Courier,monospace" size="2">[PATCH openbmc v6 04/18] image_types_uboot: Don't truncate when assembling flash<br>> Would be good to keep the option ordering consistent here (conv=notrunc<br></font><font face="Default Monospace,Courier New,Courier,monospace" size="2"><font face="Default Monospace,Courier New,Courier,monospace" size="2">> </font>is after bs=1k on the remaining dd invocations). That's a minor issue<br></font><font face="Default Monospace,Courier New,Courier,monospace" size="2"><font face="Default Monospace,Courier New,Courier,monospace" size="2">> </font>though and this is a good catch, so:<br></font><br>Good catch, fixed<br><br>[PATCH openbmc v6 07/18] initfs: update: Don't exec sh or sulogin on error just exit 1<br>> hindrance<br><br>... not noticed by checkpatch , so ran logs though gedit and also found<br><br>decompreses -> decompresses, Sanitze -> Sanitize, penality -> penalty.<br><br><br><br><font face="Default Monospace,Courier New,Courier,monospace" size="2">[PATCH openbmc v6 09/18] initfs: update: Consistently add ERROR: and print to stderr<br>> Maybe we should lift the calls out to a shell function</font><br><br>Ok, although I called it echoerr and quoted  "$@".<br><br><br>Re: [PATCH openbmc v6 17/18] initfs: update: Sanitze whitelist directory entries<br>> quote path bits<br>[in changelog] done.<br><br><br>I have separate replies to 7, 10, 11, 16, and 17 with discussion.<br><br>milton<br></font><BR>