[scm-migration-dev] collapsing Monaco and BooBug in to one object

2008-10-01 Thread Mark J. Nelson
John Sonnenschein wrote: > How's this look: http://cr.opensolaris.org/~error404/monaboo/ > > a list given to __init__ that ranks the priority. default is to only > check bugster. > > When it loops through all the bugs from whatever's in results[0] it > removes them from crs and then tries the n

[scm-migration-dev] collapsing Monaco and BooBug in to one object

2008-09-18 Thread John Sonnenschein
How's this look: http://cr.opensolaris.org/~error404/monaboo/ a list given to __init__ that ranks the priority. default is to only check bugster. When it loops through all the bugs from whatever's in results[0] it removes them from crs and then tries the next bugdb down the list ( with the

[scm-migration-dev] collapsing Monaco and BooBug in to one object

2008-09-18 Thread Mark J. Nelson
John Sonnenschein wrote: > On 17-Sep-08, at 11:37 AM, Mark J. Nelson wrote: >> - I'm not thrilled with the explicit fallback from Monaco and/or >> BOO to DOO, especially not in the BugDB lookup method. I think the >> caller should be able to specify where to look for bugs, and how to >> pri

[scm-migration-dev] collapsing Monaco and BooBug in to one object

2008-09-17 Thread John Sonnenschein
On 17-Sep-08, at 11:37 AM, Mark J. Nelson wrote: > > - I'm not thrilled with the explicit fallback from Monaco and/or > BOO to DOO, especially not in the BugDB lookup method. I think the > caller should be able to specify where to look for bugs, and how to > prioritize those sources of inf

[scm-migration-dev] collapsing Monaco and BooBug in to one object

2008-09-17 Thread Mark J. Nelson
John Sonnenschein wrote: > Hey again, > > I need another code review ( on top of the one I need for > http://cr.opensolaris.org/~error404/140/ ) I replied to that one from the original request. > new code: > http://cr.opensolaris.org/~error404/122/ - Aren't you missing an "else" and some inden

[scm-migration-dev] collapsing Monaco and BooBug in to one object

2008-09-17 Thread Richard Lowe
John Sonnenschein writes: > Hey again, > > I need another code review ( on top of the one I need for > http://cr.opensolaris.org/~error404/140/ ) > > new code: > http://cr.opensolaris.org/~error404/122/ > > and the tests fail, but I'm blaming the tests, they try to touch boobug > and monaco dir

[scm-migration-dev] collapsing Monaco and BooBug in to one object

2008-09-17 Thread John Sonnenschein
Hey again, I need another code review ( on top of the one I need for http://cr.opensolaris.org/~error404/140/ ) new code: http://cr.opensolaris.org/~error404/122/ and the tests fail, but I'm blaming the tests, they try to touch boobug and monaco directly, which have ceased to exist. New tests: