<div class="gmail_quote">On Fri, Jul 8, 2011 at 4:35 AM, Stefan Tauner <span dir="ltr"><<a href="mailto:stefan.tauner@student.tuwien.ac.at">stefan.tauner@student.tuwien.ac.at</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

i had the idea to implement an expiration date for flashrom binaries<br>
(i.e. a time bomb). in the case distributions ship ancient binaries<br>
(in the future) this may save a bunch of hardware.<br>
<br>
i noted two reactions back then when i proposed this idea on IRC in my<br>
todo.txt:<br>
 <twice11> If I validated software to work on some system, I don't want<br>
 it to suddenly break one year later.<br>
 <agaran> twice11: but kind notify that version you use is %d days old<br>
 (and repeated every multiple of time or 100days or so) could be nice<br></blockquote><div><br></div><div>Interesting idea, but I have to agree with twice11 here. I do not want a version of flashrom that works fine on the systems I'm supporting to suddenly start breaking scripts or throwing superfluous warnings to users and testers. This will definitely cause headaches for ODMs/OEMs and commercial Linux system vendors :-(</div>

<div><br></div><div>IMHO it would be more useful to try and get generic distros to a) not include a flashrom binary by default and b) when a user installs it manually, fetch and compile sources from SVN.</div><div><br></div>

<div>As for the patch itself, if this idea moves forward then it would be nice to isolate the code a bit more so it can be easily removed. Rather than placing it in print_version(), I recommend creating another function which would be called from cli_classic(). Something like:</div>

<div><div>--- cli_classic.c       (revision 1367)</div><div>+++ cli_classic.c       (working copy)</div><div>@@ -146,6 +146,7 @@</div><div> </div><div>        print_version();</div><div>        print_banner();</div><div>
+       print_expiration();</div>
<div> </div><div>        if (selfcheck())</div><div>                exit(1);</div></div><div><br></div><div>where print_expiration() contains the code that prints warning.</div><div><br></div><div><br></div></div>-- <br>
David Hendricks (dhendrix)<br>
Systems Software Engineer, Google Inc.<br>