<br><br><div class="gmail_quote">On Wed, Jun 1, 2011 at 11:48 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;">

<div class="im">On Thu, Jun 02, 2011 at 09:25:46AM +0500, Sebastian Silva wrote:<br>
> From: Sebastian Silva <<a href="mailto:icarito@sugarlabs.org">icarito@sugarlabs.org</a>><br>
><br>
> We don't ship rainbow everywhere so it's worth a try.<br>
<br>
</div>Heh.<br>
<div class="im"><br>
> ---<br>
>  sugarcommander.py |   36 ++++++++++++++++++++++++++++++++++--<br>
>  1 files changed, 34 insertions(+), 2 deletions(-)<br>
><br>
> diff --git a/sugarcommander.py b/sugarcommander.py<br>
> index 781bce9..f9cdc3f 100755<br>
> --- a/sugarcommander.py<br>
> +++ b/sugarcommander.py<br>
> @@ -16,7 +16,7 @@<br>
>  # along with this program; if not, write to the Free Software<br>
>  # Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA<br>
>  import logging<br>
> -import os<br>
> +import os, shutil<br>
<br>
</div>Separate lines.<br>
<div class="im"><br>
>  import time<br>
>  import gtk<br>
>  import pango<br>
> @@ -233,8 +233,16 @@ class SugarCommander(activity.Activity):<br>
>          self.resize_width_entry.hide()<br>
><br>
>          vbox = gtk.VBox(homogeneous=True,  spacing=5)<br>
> +        self.export_button = gtk.Button(_("Export Item To The Filesystem"))<br>
> +        self.export_button.connect('clicked',  self.export_journal_entry)<br>
> +        self.export_button.show()<br>
<br>
</div>Seems verbose.  I suggest just "Export".<br>
<div class="im"><br>
> +        target_path = os.path.join (target_dir, target_file)<br>
> +        try:<br>
> +            shutil.copyfile(file_path, target_path)<br>
> +        except IOError:<br>
> +            self.alert(_('Failure'), _('could not write %s to %s') % (target_file,target_dir))<br>
> +        else:<br>
> +            self.alert(_('Success'),  _('%s was copied to %s.')<br>
> +                    % (target_file, target_dir ))<br>
> +<br>
<br>
</div>A few minor additional or missing spaces.<br>
<br>
I like it.  A partial but adequate solution to a significant problem<br>
that will help in finding the ultimate solution.<br>
<br>
Reviewed-by: James Cameron <<a href="mailto:quozl@laptop.org">quozl@laptop.org</a>><br>
<font color="#888888"><br>
--<br>
James Cameron<br>
<a href="http://quozl.linux.org.au/" target="_blank">http://quozl.linux.org.au/</a><br>
</font><div><div></div><div class="h5">_______________________________________________<br>
Sugar-devel mailing list<br>
<a href="mailto:Sugar-devel@lists.sugarlabs.org">Sugar-devel@lists.sugarlabs.org</a><br>
<a href="http://lists.sugarlabs.org/listinfo/sugar-devel" target="_blank">http://lists.sugarlabs.org/listinfo/sugar-devel</a><br>
</div></div></blockquote></div><div><br></div>Same comments as James,  otherwise seems o.k ;)<div><br></div><div>Thanks and cheers. </div><div> <div><br></div><div><br></div></div>