BIND 10 #1978: communication to b10-auth from b10-ddns
BIND 10 Development
do-not-reply at isc.org
Wed Jun 6 05:34:51 UTC 2012
#1978: communication to b10-auth from b10-ddns
-------------------------------------+-------------------------------------
Reporter: | Owner: kevin_tes
jinmei | Status: accepted
Type: task | Milestone:
Priority: | Sprint-20120612
medium | Resolution:
Component: DDNS | Sensitive: 0
Keywords: | Sub-Project: DNS
Defect Severity: N/A | Estimated Difficulty: 5
Feature Depending on Ticket: | Total Hours: 0
Add Hours to Ticket: 0 |
Internal?: 0 |
-------------------------------------+-------------------------------------
Comment (by kevin_tes):
Replying to [comment:6 jinmei]:
> Replying to [comment:5 kevin_tes]:
>
> > > I've done mostly the same thing in #1513. Depending on your
progress
> > > in this ticket, you may want to start from a snapshot of trac1513.
> > > Or, if you've not actually started your work, I can take over it
> > > and complete it within trac1513 (so you can work on other things
> > > rather than first understanding trac1513).
>
> > This ticket should wait for the trac1513 to be merged.
>
> I suggest you branch from the latest snapshot of trac1513, make your
> own change on it and then get it reviewed, as we are already in the
> latter half of the sprint (and running out of time) and according to
> the initial review trac1513 should be pretty ready for merge already.
>
> Also, from a quick look you seem to mostly copy the corresponding
> xfrin implementation. Such duplicate code wouldn't be good. It's
> better to share it by both xfrin and ddns somehow.
>
> (And of course, I assume you already have tests as you seem to have
> the main code:-)
I am not sure what you mean by "(and running out of time)",if it must be
done today, I am sorry I can finish it on Thursday, if this disturbs the
plan of this sprint,you guys can take this ticket and I'll review it.
--
Ticket URL: <http://bind10.isc.org/ticket/1978#comment:7>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development
More information about the bind10-tickets
mailing list