<font color="#000099"><font size="2"><font face="trebuchet ms,sans-serif"><br></font></font></font><br><div class="gmail_quote">On Wed, Jun 15, 2011 at 10:56 PM, Frederick Grose <span dir="ltr"><<a href="mailto:fgrose@gmail.com">fgrose@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div class="gmail_quote"><div class="im">On Wed, Jun 15, 2011 at 8:28 PM, James Cameron <span dir="ltr"><<a href="mailto:quozl@laptop.org" target="_blank">quozl@laptop.org</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Wed, Jun 15, 2011 at 12:01:55PM -0300, Esteban Bord?n wrote:<br>
> This patch show the build number in the name page for first boot (see<br>
> attached file). It's very useful to test the correct massive flashing<br>
> for technical team.<br>
<br>
I like it.<br>
<br>
One small technical change though ...<br>
<br>
> @@ -86,7 +86,20 @@ class _NamePage(_Page):<br>
>                         orientation=hippo.ORIENTATION_HORIZONTAL,)<br>
><br>
>          self._intro = intro<br>
> +<br>
> +        if os.path.exists('/boot/olpc_build'):<br>
> +            build_no_path = '/boot/olpc_build'<br>
> +        elif os.path.exists('/etc/fedora-release'):<br>
> +            build_no_path = '/etc/fedora-release'<br>
> +        else:<br>
> +            build_no_path = None<br>
><br>
> +        if build_no_path:<br>
> +            fd = open('/etc/fedora-release', 'r')<br>
<br>
The line above should open build_no_path rather than<br>
'/etc/fedora-release'<br>
<font color="#888888"><br>
--<br>
James Cameron<br>
<a href="http://quozl.linux.org.au/" target="_blank">http://quozl.linux.org.au/<br></a></font></blockquote><div><br></div></div><div>To keep with Sugar's design, it would be good to put the text in gray, and in the lower left corner, so as to de-emphasize it in relation to the Learner's primary activity on that screen, and to balance the visual weights with the '> Next' and '< Back' buttons.</div>


<div><br></div><div>           --Fred</div></div></blockquote><div><br></div><div>Once the particular "massive flashing" protocol has been tested and debugged, is the information really needed at that point?</div>

</div><br>