<div dir="ltr">> Have you decided whether we need to commit the generated files<br>> (Cargo.toml, Cargo.lock)?  It looks like in this series those files<br>> are still included and not added to .gitignore.<span class="gmail-im" style="color:rgb(80,0,80)"><br></span><div><br></div><div>I'm sorry I forgot adding Cargo.lock to .gitignore. I'll add it to .gitignore. However, I think Cargo.toml should be staged. This is because this is a file managed by hands. It contains dependencies, versions, editions, crate name. It can be generated by shellscript which dumps such information. However, I think it may be not preferable.</div><div><br></div><div>Regards,</div><div>Hiroyuki</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">2019年7月4日(木) 19:39 Richard W.M. Jones <<a href="mailto:rjones@redhat.com">rjones@redhat.com</a>>:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Tue, Jul 02, 2019 at 10:09:00PM +0900, Hiroyuki Katsura wrote:<br>
> I fixed the patch I submitted before based on comments, and there are some<br>
> commits which are merged or divided. So, I will re-send all the patches.<br>
<br>
I looked at the v2 patches and I think they're in reasonable<br>
shape.<br>
<br>
There's been a lot of discussion of using ’git rebase --interactive’.<br>
I think that's a good thing, and also good practice if you're going to<br>
become a more frequent open source contributor.  It helps people when<br>
they're reviewing patches.<br>
<br>
For this particular case I think we'd probably squash all the patches<br>
into a single commit when pushing them upstream.<br>
<br>
Have you decided whether we need to commit the generated files<br>
(Cargo.toml, Cargo.lock)?  It looks like in this series those files<br>
are still included and not added to .gitignore.<br>
<br>
Rich.<br>
<br>
-- <br>
Richard Jones, Virtualization Group, Red Hat <a href="http://people.redhat.com/~rjones" rel="noreferrer" target="_blank">http://people.redhat.com/~rjones</a><br>
Read my programming and virtualization blog: <a href="http://rwmj.wordpress.com" rel="noreferrer" target="_blank">http://rwmj.wordpress.com</a><br>
virt-builder quickly builds VMs from scratch<br>
<a href="http://libguestfs.org/virt-builder.1.html" rel="noreferrer" target="_blank">http://libguestfs.org/virt-builder.1.html</a><br>
</blockquote></div>