Skip Menu |

This queue is for tickets about the Tree-Binary CPAN distribution.

Report information
The Basics
Id: 86822
Status: rejected
Priority: 0/
Queue: Tree-Binary

People
Owner: Nobody in particular
Requestors: pgoetz [...] jcvi.org
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: 1.05



Subject: new Tree::Binary not backwards compatible... again
Date: Tue, 9 Jul 2013 18:34:25 -0400
To: "bug-Tree-Binary [...] rt.cpan.org" <bug-Tree-Binary [...] rt.cpan.org>
From: "Goetz, Phil" <pgoetz [...] jcvi.org>
I just spent a day last year going through code and changing it to work with last year's new API for Tree:Binary. Now we upgraded the module and the API has changed again. It would be nice if it were backwards compatible with previous versions. For example, if you could provide a new() function that takes no arguments, like the old one, that would help. - Phil
Subject: Re: [rt.cpan.org #86822] new Tree::Binary not backwards compatible... again
Date: Wed, 10 Jul 2013 10:03:06 +1000
To: bug-Tree-Binary [...] rt.cpan.org
From: Ron Savage <ron [...] savage.net.au>
Hi Phil On 10/07/13 08:35, Goetz, Phil via RT wrote: Show quoted text
> Tue Jul 09 18:35:00 2013: Request 86822 was acted upon. > Transaction: Ticket created by pgoetz@jcvi.org > Queue: Tree-Binary > Subject: new Tree::Binary not backwards compatible... again > Broken in: (no value) > Severity: (no value) > Owner: Nobody > Requestors: pgoetz@jcvi.org > Status: new > Ticket<URL: https://rt.cpan.org/Ticket/Display.html?id=86822> > > > I just spent a day last year going through code and changing it to work with last year's new API for Tree:Binary. > Now we upgraded the module and the API has changed again. > It would be nice if it were backwards compatible with previous versions. > For example, if you could provide a new() function that takes no arguments, like the old one, that would help.
I am sorry to hear this. Now, just to be absolutely sure, you do realize I hope that there used to be 2 Tree::Binary modules on CPAN: One standalone and one within Tree. The latter is now Tree::Binary2. Please double-check which one you're using 'v' which one you think you're using :-). Cheers -- Ron Savage http://savage.net.au/ Ph: 0421 920 622
Subject: RE: [rt.cpan.org #86822] new Tree::Binary not backwards compatible... again
Date: Mon, 15 Jul 2013 17:55:04 -0400
To: "bug-Tree-Binary [...] rt.cpan.org" <bug-Tree-Binary [...] rt.cpan.org>
From: "Goetz, Phil" <pgoetz [...] jcvi.org>
Thanks for explaining! My code was expecting the module formerly known as Tree::Binary. Show quoted text
________________________________________ From: ron@savage.net.au via RT [bug-Tree-Binary@rt.cpan.org] Sent: Tuesday, July 09, 2013 8:03 PM To: Goetz, Phil Subject: Re: [rt.cpan.org #86822] new Tree::Binary not backwards compatible... again <URL: https://rt.cpan.org/Ticket/Display.html?id=86822 > Hi Phil On 10/07/13 08:35, Goetz, Phil via RT wrote:
> Tue Jul 09 18:35:00 2013: Request 86822 was acted upon. > Transaction: Ticket created by pgoetz@jcvi.org > Queue: Tree-Binary > Subject: new Tree::Binary not backwards compatible... again > Broken in: (no value) > Severity: (no value) > Owner: Nobody > Requestors: pgoetz@jcvi.org > Status: new > Ticket<URL: https://rt.cpan.org/Ticket/Display.html?id=86822> > > > I just spent a day last year going through code and changing it to work with last year's new API for Tree:Binary. > Now we upgraded the module and the API has changed again. > It would be nice if it were backwards compatible with previous versions. > For example, if you could provide a new() function that takes no arguments, like the old one, that would help.
I am sorry to hear this. Now, just to be absolutely sure, you do realize I hope that there used to be 2 Tree::Binary modules on CPAN: One standalone and one within Tree. The latter is now Tree::Binary2. Please double-check which one you're using 'v' which one you think you're using :-). Cheers -- Ron Savage http://savage.net.au/ Ph: 0421 920 622
The problem is confusion over 2 different modules called Tree::Binary.