<div dir="ltr">+1 for Robie's work<div><br></div><div>One thing I'd changed was the usage of echo "..${<span style="color:rgb(221,34,0);font-size:13.3333px;background-color:rgb(255,240,240)">$rootpw}..</span>" >> /some/file as I am not sure if this will be shown in the processlist and therefor would cause an security issue?<br></div></div><div class="gmail_extra"><br><div class="gmail_quote">2016-03-17 16:25 GMT+01:00 Robie Basak <span dir="ltr"><<a href="mailto:robie.basak@ubuntu.com" target="_blank">robie.basak@ubuntu.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">OK, 5.7 packaging is ready now, and pushed to the 5.7 repository. A<br>
couple of major changes to what was there before:<br>
<br>
1) It has 5.6 in its history, so figuring out the past is much easier.<br>
<br>
2) There are a series of logical and small commits that can be<br>
individually reviewed to get from 5.6 to 5.7. This way when reviewing I<br>
can be sure that we have not regressed anything.<br>
<br>
I cannot replace the master branch, so I pushed to a branch called<br>
"rbasak" and also copied the current master branch from Bjoern to one<br>
called "jesusch" (his username). Can a DD please delete the current<br>
master branch and rename "rbasak" to "master"?<br>
<br>
I found the lack of history to be a big pain during 5.6. As none of us<br>
were around many years ago, we need to be able to ask git questions like<br>
"when was line X added and with what changelog message". Not preserving<br>
this history in git makes life much harder so I think it's worth doing<br>
this properly from the start.<br>
<br>
I also ask (again) that people only push to the master branch high<br>
quality commits of this form: one logical change per commit, with a<br>
sensible commit message explaining the change.<br>
<br>
With that, I think my branch is ready for upload to Debian experimental,<br>
and I'd appreciate any review. One thing that still needs to be done is<br>
translation updates. We have various updates in the BTS and also need to<br>
unfuzz the latest s/5.6/5.7/ changes.<br>
<br>
Bjoern, is there anything that you want to add to this branch? I looked<br>
through your previous commits, but as they weren't broken down I<br>
couldn't really see what changes I could pick out.<br>
<br>
I have also uploaded this work to Ubuntu. A couple of Ubuntu-specific<br>
changes were needed that are not applicable to Debian. You can track<br>
this in the ubuntu/devel branch that I have pushed.<br>
<span class="HOEnZb"><font color="#888888"><br>
Robie<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature">Mit freundlich Grüßen / Kind regards<br><br>Björn Boschman</div>
</div>