<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Aug 30, 2016 at 4:07 AM, Saverio Proto <span dir="ltr"><<a href="mailto:zioproto@gmail.com" target="_blank">zioproto@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><span class="">> Most of the topics that were covered in the Ubuntu packaging session are<br>
> summarized in our wiki, to which I've updated based on our discussions:<br>
> <a href="https://wiki.ubuntu.com/OpenStack" rel="noreferrer" target="_blank">https://wiki.ubuntu.com/<wbr>OpenStack</a><br>
<br>
</span>Hello Corey,<br>
<br></blockquote><div><br></div><div>Hello Saverio,</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">
thanks for updating the wiki so quickly.<br></blockquote><div><br></div><div>Np, thanks for the input.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">
I am trying to rebuild stable/liberty cinder adding<br>
<a href="https://review.openstack.org/#/c/306610/" rel="noreferrer" target="_blank">https://review.openstack.org/#<wbr>/c/306610/</a><br>
<br>
I am building for ubuntu trusty at the moment.<br>
<br>
First of all this line makes no sense to people not familiar with launchpad:<br>
<br>
git clone lp:~ubuntu-server-dev/ubuntu/+<wbr>source/nova<br>
<br>
I would write:<br>
<br>
git clone <a href="https://git.launchpad.net/~ubuntu-server-dev/ubuntu/+source/nova" rel="noreferrer" target="_blank">https://git.launchpad.net/~<wbr>ubuntu-server-dev/ubuntu/+<wbr>source/nova</a></blockquote><div><br></div><div>I added this.  Note there's a section at the top of that page called 'Git Configuration' linking to lp configuration.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><br>
<br>
I had to look up my shell history from NYC to understand what to<br>
replace in 'lp' :)<br>
Or we could link this page <a href="https://help.launchpad.net/Code/Git" rel="noreferrer" target="_blank">https://help.launchpad.net/<wbr>Code/Git</a><br>
where it explains how to hack the gitconfig.<br>
<br>
Also, there is a part: "if you have added an appropriate changelog<br>
comment then: debcommit"<br>
We can improve saying that we can edit the changelog with the command 'dch -i'<br></blockquote><div><br></div><div>Good point, I've made that update.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">
<br>
I tried to submit a merge request for the cinder package.<br>
<a href="https://code.launchpad.net/~zioproto/ubuntu/+source/cinder/+git/cinder/+merge/304341" rel="noreferrer" target="_blank">https://code.launchpad.net/~<wbr>zioproto/ubuntu/+source/<wbr>cinder/+git/cinder/+merge/<wbr>304341</a><br>
<br>
I already spotted an error in my patch in the debian changelog, a<br>
malformed email was added probably by debcommit.<br>
<br>
What is the review workflow ? I should commit amend or just adding<br>
commits on top of this branch ?<br>
<br></blockquote><div><br></div><div>First, thanks for contributing!</div><div><br></div><div>You can just fix that in a follow on commit.  For the most part though I want to see a clean git history.  When I merge I use --ff-only, so your git history will get merged into the main branch in tact.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">
thanks !<br>
<span class=""><font color="#888888"><br>
Saverio<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">Regards,<br>Corey<br></div></div>
</div></div>