<div dir="ltr">Dear Martin,<div><br></div><div>Oh, I see. Thank you for your information. I should have seen how Cargo.toml is implemented in nbdkit!</div><div><br></div><div>Regards,</div><div>Hiroyuki</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">2019年7月6日(土) 20:03 Martin Kletzander <<a href="mailto:mkletzan@redhat.com">mkletzan@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 Fri, Jul 05, 2019 at 08:37:20AM +0100, Richard W.M. Jones wrote:<br>
>On Fri, Jul 05, 2019 at 01:28:12PM +0900, Hiroyuki Katsura wrote:<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>
>> I'm sorry I forgot adding Cargo.lock to .gitignore. I'll add it to<br>
>> .gitignore. However, I think Cargo.toml should be staged. This is because<br>
>> this is a file managed by hands. It contains dependencies, versions,<br>
>> editions, crate name. It can be generated by shellscript which dumps such<br>
>> information. However, I think it may be not preferable.<br>
><br>
>OK, no problem.<br>
><br>
<br>
Just one thing, the Cargo.toml includes a version under which the crate would be<br>
published.  I presume the version would be the same as the one of the project<br>
itself, i.e. when releasing libguestfs-x.y.z, we publish guestfs-rs-x.y.z to<br>
<a href="http://crates.io" rel="noreferrer" target="_blank">crates.io</a>.  To make them in sync, it might be better to do it the same way<br>
nbdkit does it, having <a href="http://Cargo.toml.in" rel="noreferrer" target="_blank">Cargo.toml.in</a> which is taken and the version placeholder<br>
is replaced by the actual version.<br>
<br>
>Rich.<br>
><br>
>> Regards,<br>
>> Hiroyuki<br>
>><br>
>> 2019年7月4日(木) 19:39 Richard W.M. Jones <<a href="mailto:rjones@redhat.com" target="_blank">rjones@redhat.com</a>>:<br>
>><br>
>> > 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<br>
>> > 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<br>
>> > <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>
>> ><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>
>Fedora Windows cross-compiler. Compile Windows programs, test, and<br>
>build Windows installers. Over 100 libraries supported.<br>
><a href="http://fedoraproject.org/wiki/MinGW" rel="noreferrer" target="_blank">http://fedoraproject.org/wiki/MinGW</a><br>
</blockquote></div>