<font face="Default Sans Serif,Verdana,Arial,Helvetica,sans-serif" size="2"><br>About 02/23/2016 04:55PM in some time zone, Joel Stanley wrote:<br>>On Wed, Feb 24, 2016 at 1:14 AM, Andrew Jeffery <<a target="_blank" href="mailto:andrew@aj.id.au">andrew@aj.id.au</a>><br>>wrote:<br>>> It was inconsistent with the rest of the file, which seems to use 7<br>>> spaces. In particular, vim-sleuth was detecting the indentation<br>>style<br>>> from the first indent(s) in the file, which were in the oe_mkimage<br>>> function.<br>><br>>7 spaces seems kind of unusual.<br>><br>>Upstream bbclass files are a mixture; I'd go with 8 spaces as I<br>>suspect that was the intent in this file.<br>><br><br>Looks like yocto-poky/meta/classes/image_types_uboot.bbclass<br>upon which this is based used 4 spaces for the function that<br>was copied.<br><br>Going to 8 spaces puts one \ over 80 columns in do_generate_flash<br>but its after a || so it can just be dropped and the shell will parse the <br>next line implicitly.<br><br>I don't care if we leave it at 7 for now or go to 8.<br><br><br>>><br>>> Signed-off-by: Andrew Jeffery <<a target="_blank" href="mailto:andrew@aj.id.au">andrew@aj.id.au</a>><br>>> ---<br>>>  meta-phosphor/classes/obmc-phosphor-image_types_uboot.bbclass | 6<br>>+++---<br>>>  1 file changed, 3 insertions(+), 3 deletions(-)<br>><br><br>milton<br></font><BR>