<html><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class=""><blockquote type="cite" class="">I'll measure it but I suspect it's on the order of msecs. It has to read<br class="">about 80K of flash to find the version string.</blockquote><div class=""><br class=""></div>ok<div class=""><br class=""></div><div class=""><blockquote type="cite" class="">Seems like the version finder code should live in main/ovms_version.cpp?</blockquote><div><br class=""></div><div>Yep.</div><div><br class=""></div><div><blockquote type="cite" class="">Maybe add this after the "Boot partition" line?</blockquote><br class=""></div><div>I think the top part of the report is showing what you have, and the bottom part is what is available. Perhaps order like this:</div><div><br class=""></div></div><blockquote style="margin: 0 0 0 40px; border: none; padding: 0px;" class=""><div><blockquote type="cite" class="">OVMS# ota status<br class=""> Firmware: 3.1.011-16-gb8dc60e-dirty/factory/main (build idf<br class="">v3.1-dev-1586-g65355d28 Nov 11 2018 16:39:30)<br class=""></blockquote><br class=""><blockquote type="cite" class=""> Running partition: factory<br class=""> Boot partition: factory<br class=""></blockquote><blockquote type="cite" class=""> Factory image: 3.1.011-16-gb8dc60e-dirty<br class=""> OTA_0 image: 3.1.011<br class=""> OTA_1 image: 3.1.011-16-gb8dc60e</blockquote><br class=""><blockquote type="cite" class=""> Server Available: 3.1.011 (no update required)<br class=""> Sun Nov 4 01:53:00 UTC 2018 OTA release<br class=""> [...]<br class=""></blockquote></div></blockquote><div class=""><div><br class=""></div><div>Better?</div><div><br class=""></div><div>Regards, Mark.</div><div><br class=""><blockquote type="cite" class=""><div class="">On 12 Nov 2018, at 11:50 AM, Craig Leres <<a href="mailto:leres@xse.com" class="">leres@xse.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div class="">On 11/11/2018 6:53 PM, Mark Webb-Johnson wrote:<br class=""><blockquote type="cite" class="">Looks cool. Those prefix and postfix were put there for that reason, but we never got round to doing it.<br class=""><br class="">How is performance? If acceptable, please send a pull request and we’ll get it merged (unless anyone else has any comments / suggestions)?<br class=""></blockquote><br class="">I'll measure it but I suspect it's on the order of msecs. It has to read<br class="">about 80K of flash to find the version string.<br class=""><br class="">Seems like the version finder code should live in main/ovms_version.cpp?<br class=""><br class=""><blockquote type="cite" class="">Could you extend the ‘ota status’ command to show the same?<br class=""></blockquote><br class="">Currently it shows:<br class=""><br class=""> OVMS# ota status<br class=""> Running partition: factory<br class=""> Boot partition: factory<br class=""> Firmware: 3.1.011-16-gb8dc60e-dirty/factory/main (build idf<br class="">v3.1-dev-1586-g65355d28 Nov 11 2018 16:39:30)<br class=""> Server Available: 3.1.011 (no update required)<br class=""><br class=""> Sun Nov 4 01:53:00 UTC 2018 OTA release<br class=""> [...]<br class=""><br class="">Maybe add this after the "Boot partition" line?<br class=""><br class=""> Factory image: 3.1.011-16-gb8dc60e-dirty<br class=""> OTA_0 image: 3.1.011<br class=""> OTA_1 image: 3.1.011-16-gb8dc60e<br class=""><br class="">I'll try for a pull request next weekend.<br class=""><br class=""><span class="Apple-tab-span" style="white-space:pre"> </span><span class="Apple-tab-span" style="white-space:pre"> </span>Craig<br class=""></div></div></blockquote></div><br class=""></div></body></html>