<div style="line-height:1.7;color:#000000;font-size:14px;font-family:arial"><br><pre>At 2014-01-27 23:10:04,"Cole Robinson" <crobinso@redhat.com> wrote:
>On 01/27/2014 02:48 AM, Chen Hanxiao wrote:
>> From: Chen Hanxiao <chenhanxiao@cn.fujitsu.com>
>> 
>> 
>> Chen Hanxiao (2):
>>   virt-install: add support for option "--cpu host-model-basic"
>>   man: update virt-intall man page for option "--cpu host-model-basic"
>> 
>>  man/virt-install.pod | 5 +++++
>>  virtinst/cli.py      | 3 +++
>>  2 files changed, 8 insertions(+)
>> 
>
>Idea and code look fine, but I don't like the name 'host-model-basic'. How
>about 'host-model-only' or 'host-model-name' ?</pre><pre>I think <span style="line-height: 1.7;">'host-model-only'  would be better.</span></pre><pre>>
>Also, please squash the two patches together, for small option additions we
>should just commit tests, docs, and code all together in one patch.
>
<span style="line-height: 1.7;">I'll squash them together with v2 patch.</span></pre><pre>Thanks for your advice. </pre><pre><br></pre><pre><br></pre><pre><br></pre></div>