<div class="gmail_quote"><font face="'courier new', monospace">On Mon, Mar 29, 2010 at 4:16 PM, Peter Robinson </font><span dir="ltr"><font face="'courier new', monospace"><<a href="mailto:pbrobinson@gmail.com" target="_blank">pbrobinson@gmail.com</a>></font></span><font face="'courier new', monospace"> wrote:<br>
</font><blockquote class="gmail_quote" style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0.8ex;border-left-width:1px;border-left-color:rgb(204, 204, 204);border-left-style:solid;padding-left:1ex">
<font face="'courier new', monospace"><br></font></blockquote><blockquote class="gmail_quote" style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0.8ex;border-left-width:1px;border-left-color:rgb(204, 204, 204);border-left-style:solid;padding-left:1ex">
<font face="'courier new', monospace">...</font></blockquote><blockquote class="gmail_quote" style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0.8ex;border-left-width:1px;border-left-color:rgb(204, 204, 204);border-left-style:solid;padding-left:1ex">
<font face="'courier new', monospace"> </font></blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><font face="'courier new', monospace">Do you think the patch is either useful for<br>
upstream or self contained enough it won't affect it? If so could you<br>
possibly file a bug [1] so we can can get it upstream?<br><br>
Cheers,<br>
Peter<br><br>
[1]
</font><a href="https://bugzilla.redhat.com/enter_bug.cgi?product=Fedora&version=rawhide&component=livecd-tools" target="_blank"><font face="'courier new', monospace">https://bugzilla.redhat.com/enter_bug.cgi?product=Fedora&version=rawhide&component=livecd-tools</font></a><font face="'courier new', monospace"><br>
</font></blockquote></div><font face="'courier new', monospace"><br></font><div><font face="'courier new', monospace">See </font><a href="http://lists.sugarlabs.org/archive/soas/2010-April/000998.html" target="_blank"><font face="'courier new', monospace">http://lists.sugarlabs.org/archive/soas/2010-April/000998.html</font></a><font face="'courier new', monospace">.</font></div>
<div><font face="'courier new', monospace"><br></font></div><div><font face="'courier new', monospace">Also </font><a href="https://bugzilla.redhat.com/show_bug.cgi?id=448030" target="_blank"><font face="'courier new', monospace">https://bugzilla.redhat.com/show_bug.cgi?id=448030</font></a></div>
<div><font face="'courier new', monospace"><br></font></div><div><font face="'courier new', monospace">There are several changes:</font></div><div><font face="'courier new', monospace"><br>
</font></div><div><font face="'courier new', monospace">1. I couldn't help but adjust the whitespace in the livecd-iso-to-disk</font></div><div><font face="'courier new', monospace"> script to work with it effectively.</font></div>
<div><font face="'courier new', monospace">2. Updated livecd-iso-to-disk.pod (attached)</font></div><div><font face="'courier new', monospace">3. Added a --help option to show usage.</font></div>
<div><font face="'courier new', monospace">4. Included extended help in usage as the script is often distributed</font></div><div><font face="'courier new', monospace"> as a solitary file.</font></div>
<div><font face="'courier new', monospace">5. Reordered the options to semi group related items</font></div><div><font face="'courier new', monospace">6. Moved the verification test down after some common input errors</font></div>
<div><font face="'courier new', monospace"> (to save time for users on slower processors).</font></div><div><font face="'courier new', monospace">7. Included a syslinux-config-file block to resolve cross-filesystem</font></div>
<div><font face="'courier new', monospace"> installs. (All installations will pass through this code.)</font></div><div><font face="'courier new', monospace"><br>
</font></div><div><font face="'courier new', monospace">I have not tested all the many code paths. And the code reordering</font></div><div><font face="'courier new', monospace">could have some subtle effects.</font></div>
<div><font face="'courier new', monospace"><br></font></div><div><font face="'courier new', monospace">So with all those changes, it will need extra scrutiny.</font></div>
<div><font face="'courier new', monospace"><br></font></div><div><span style="font-family:'courier new', monospace">The capability that this version provides is significant, I believe.</span></div>
<div><font face="'courier new', monospace">Vendors, Fedora Ambassadors, and many others wishing to easily share</font></div><div><font face="'courier new', monospace">customized images will have a fairly simple method.</font></div>
<div><font face="'courier new', monospace"><br></font></div><div><font face="'courier new', monospace"><span style="font-family:arial"><div>
<font face="'courier new', monospace">Thanks for reviewing!</font></div><div><font face="'courier new', monospace"><br></font></div><div><font face="'courier new', monospace"> --Fred</font></div>
<div><font face="'courier new', monospace"><br></font></div></span></font></div>