<div dir="ltr"><div>I see that your work is based, at least in part, on my previous patch and research paper. Very nice.</div><div><br></div>A small note:<div>I suggest to move the whole section of "construct VM ()" from "new VM wizard" and into Main Code.</div><div><br></div><div>Why ?</div><div><br></div><div>This way it will allow us to construct both normal VMs *and* unattended VMs from CLI, instead of GUI, and add all suggested hardware controllers and disks quickly.</div><div><br></div><div>I didn't do it my patch, because the change would be pretty invasive, and I wanted to keep things clam and conservative.<br></div><div><br></div><div>Nowadays, if you create a VM from CLI (or API), you will need to manually know which audio and which disk controller to use for a particular guest OS, and if you try to create an Unattended Install VM from CLI, it will break down awfully, . (even using IMachine::IUnattended Main API)</div><div><br></div><div>The whole "suggested hardware" mechanism for new VM needs to be moved from GUI into Main API, "IMachine".</div><div><br></div><div><div>===</div><div>Another question:</div><div><h2 id="gmail-file0" style="font-family:arial,helvetica,sans-serif;font-size:14px;margin:0px auto;padding:0.1em 0px 0.25em 0.5em;border-bottom:none;color:rgb(51,51,51)">In file <a href="https://www.virtualbox.org/changeset/64994/vbox/trunk/src/VBox/Main/idl/VirtualBox.xidl" title="Show the changeset 64994 restricted to trunk/src/VBox/Main/idl/VirtualBox.xidl" style="text-decoration:none;color:inherit;border:none">Main/idl/VirtualBox.xidl</a></h2></div><div><br></div><div>You seem to be using a new parameter:</div><div><div> <span class="gmail-Apple-tab-span" style="white-space:pre">        </span>3815<span class="gmail-Apple-tab-span" style="white-space:pre">  </span>    <attribute name="imageIndex" type="unsigned short"> </div><div> <span class="gmail-Apple-tab-span" style="white-space:pre">        </span>3816<span class="gmail-Apple-tab-span" style="white-space:pre">  </span>      <desc> </div><div> <span class="gmail-Apple-tab-span" style="white-space:pre">      </span>3817<span class="gmail-Apple-tab-span" style="white-space:pre">  </span>        The image index on installation CD\DVD used to install. </div><div> <span class="gmail-Apple-tab-span" style="white-space:pre">        </span>3818<span class="gmail-Apple-tab-span" style="white-space:pre">  </span>        Used only with Windows installation CD\DVD </div><div> <span class="gmail-Apple-tab-span" style="white-space:pre">     </span>3819<span class="gmail-Apple-tab-span" style="white-space:pre">  </span>      </desc> </div><div> <span class="gmail-Apple-tab-span" style="white-space:pre">     </span>3820<span class="gmail-Apple-tab-span" style="white-space:pre">  </span>    </attribute> </div></div><div><br></div><div>Why is it needed ?</div>-- <br><div class="gmail_signature">-Alexey Eromenko "Technologov"</div>
</div></div>