Hi,
kindly see the attached patch and the commit message. I'm fine with merging the patch after we branch 1-12 and master but I didn't want to forget to remove this obsolete code while I'm looking at the resolver subtree.
On 12/15/2014 03:10 AM, Jakub Hrozek wrote:
Hi,
kindly see the attached patch and the commit message. I'm fine with merging the patch after we branch 1-12 and master but I didn't want to forget to remove this obsolete code while I'm looking at the resolver subtree.
Ack to this one. Please note that it depends on: "RESOLV: Add an internal function to read TTL from a DNS packet"
On Tue, Jan 06, 2015 at 12:50:26PM +0100, Pavel Březina wrote:
On 12/15/2014 03:10 AM, Jakub Hrozek wrote:
Hi,
kindly see the attached patch and the commit message. I'm fine with merging the patch after we branch 1-12 and master but I didn't want to forget to remove this obsolete code while I'm looking at the resolver subtree.
Ack to this one. Please note that it depends on: "RESOLV: Add an internal function to read TTL from a DNS packet"
Thanks for the review, attached is a patch rebased on the current master.
On 01/14/2015 04:22 PM, Jakub Hrozek wrote:
On Tue, Jan 06, 2015 at 12:50:26PM +0100, Pavel Březina wrote:
On 12/15/2014 03:10 AM, Jakub Hrozek wrote:
Hi,
kindly see the attached patch and the commit message. I'm fine with merging the patch after we branch 1-12 and master but I didn't want to forget to remove this obsolete code while I'm looking at the resolver subtree.
Ack to this one. Please note that it depends on: "RESOLV: Add an internal function to read TTL from a DNS packet"
Thanks for the review, attached is a patch rebased on the current master.
Still ack.
On Thu, Jan 15, 2015 at 12:02:46PM +0100, Pavel Březina wrote:
On 01/14/2015 04:22 PM, Jakub Hrozek wrote:
On Tue, Jan 06, 2015 at 12:50:26PM +0100, Pavel Březina wrote:
On 12/15/2014 03:10 AM, Jakub Hrozek wrote:
Hi,
kindly see the attached patch and the commit message. I'm fine with merging the patch after we branch 1-12 and master but I didn't want to forget to remove this obsolete code while I'm looking at the resolver subtree.
Ack to this one. Please note that it depends on: "RESOLV: Add an internal function to read TTL from a DNS packet"
Thanks for the review, attached is a patch rebased on the current master.
Still ack.
* master: 5594736ea2618bb3e487f47fd199e1d2cf4c58fd
sssd-devel@lists.fedorahosted.org