On Wed, Jun 15, 2011 at 8:28 PM, James Cameron <qu...@laptop.org> wrote:
> On Wed, Jun 15, 2011 at 12:01:55PM -0300, Esteban Bord?n wrote: > > This patch show the build number in the name page for first boot (see > > attached file). It's very useful to test the correct massive flashing > > for technical team. > > I like it. > > One small technical change though ... > > > @@ -86,7 +86,20 @@ class _NamePage(_Page): > > orientation=hippo.ORIENTATION_HORIZONTAL,) > > > > self._intro = intro > > + > > + if os.path.exists('/boot/olpc_build'): > > + build_no_path = '/boot/olpc_build' > > + elif os.path.exists('/etc/fedora-release'): > > + build_no_path = '/etc/fedora-release' > > + else: > > + build_no_path = None > > > > + if build_no_path: > > + fd = open('/etc/fedora-release', 'r') > > The line above should open build_no_path rather than > '/etc/fedora-release' > > -- > James Cameron > http://quozl.linux.org.au/ > 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. --Fred
_______________________________________________ Sugar-devel mailing list Sugar-devel@lists.sugarlabs.org http://lists.sugarlabs.org/listinfo/sugar-devel