<div class="gmail_quote">On Wed, Jun 15, 2011 at 8:28 PM, James Cameron <span dir="ltr"><<a href="mailto:quozl@laptop.org">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>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><br>