011/6/18 Sascha Silbe <span dir="ltr"><<a href="mailto:sascha-ml-reply-to-2011-3@silbe.org">sascha-ml-reply-to-2011-3@silbe.org</a>></span><br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

<br>
Nice idea. As for the patch, I wonder why you use different code than<br>
what's in extensions/cpsection/aboutcomputer/model.py. I'm not sure<br>
whether importing or copying is the better option here, but different<br>
code altogether definitely isn't. We need to show the same value in both<br>
the Intro and the About my Computer CP section.<br>
<br>
Importing extensions/cpsection/aboutcomputer/model.py has the obvious<br>
advantage that there's no code duplication and thus no risk of diverging<br>
code paths. The disadvantage is that the Intro would break if someone<br>
chose not to install the About my Computer CP section. Copying the code<br>
and adding a note to both copies that they should be kept in sync would<br>
allow the Intro screen to be used even in absence of the About my<br>
Computer CP section.<br></blockquote><div><br>I don't know why someone would chose not install "About my computer" CP Section. IMHO the best option is importing extensions/cpsection/aboutcomputer/model.py if the extension is installed, otherwise intro screen will not display build number.<br>

<br><br>regards,<br>Esteban.<br></div></div><br>