X-RT-Original-Encoding: utf-8 X-RT-Interface: Web Content-Disposition: inline References: <20090304083739.GC13465@mandriva.com> Message-ID: X-Mailer: MIME-tools 5.508 (Entity 5.508) In-Reply-To: Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: binary RT-Send-CC: Content-Length: 2223 On Mon Jun 19 19:47:24 2017, tmark wrote: > The bug fixes look fine. Ideally, it would be better to do them > separately. I have moved the code relevant to 18183 to rt18183 branch. There is also rt19430a branch that contains only the fixes related to 19430 (+ 2 minor nits), but no test framework. The test framework will be handled in 45434. In summary: please review rt19430a and ignore (but not delete) rt19430. Test related commits from it will be cherry-picked later. > Did you do any manual testing the client under linux? Yes, but my tests were not extensive. I disabled ubuntu interface management and started the compiled dhclient with update linux script. It got IPv4 address properly and configured /etc/resolv.conf. I also ran the unit-tests that are available on rt19430. > Release notes: > > 1. When you add new entries, you append them to the end of the Changes > section, not add them to the beginning, > It's opposite of the way we do it in Kea's ChangeLog. > > 2. You did thank Mr.Pollock, but normally we reply to the requester > explaining that we've addressed their issue and then ask them if they > want to be cited, and if so how do they want to appear. > > (BTW, both of these are discussed in the developer's survival guide ;) Where is the latest version of it available? I have mine from the last vodoo sessi^H^H^H^H^H dhcpd overview training you did couple months ago. > The rest of the comments have to do with the test framework. First, > thanks for doing this, it should come in handy. > > 1. How much work would it be to move the test framework under it's own > ticket and branch? Might be easier to merge into 4.3.6/4.1-ESV-R15 on > it's own and I am also undecided as to whether those branches should > get it. Not that much, as long as we maintain proper order of things. I have split the code. So far 18183 (branch rt18183) and 19430 (branch rt19430a) are ready for review. Once both are merged in, we could branch rt45434 and cherry-pick changes from the old rt19430. I have the list of commits ready, so should be easy as long as both fixes from 18183 and 19430 are on master. Rest of the comments ignored here as no longer relevant. They'll be addressed in 45434.