Mike Kupfer <mike.kupfer at sun.com> writes: > I just finished going through the P1-P3 bugs and doing some simple risk > analysis (likelihood of introducing breakage, possibility of being > harder than the initial estimate). > > I added 456 ("DbLookups nits") to the blocker list for the internal > review. This was less about risk and more about fixing known problems > that we'd just flag again in our internal review. > > The only other bugs that caught my eye were all Cadmium things: > > 356 cdm recommit needs to cope with the tags it may be trashing > 418 git formatted hg diffs can lose copies/renames > 464 ActiveList build doesn't consider one half of a working copy merge. > > Rich has a code review request out for 356, and he's asked me to review > 464. > > Does 418 still need to be a stopper? Or can we just document it as a > restriction?
I think it could be documented as a restriction, but other opinion is desired. > Also, I'd been keeping Cadmium separate from the rest of the tools in > terms of planning for reviews. Do we want to keep doing that? The list > of open issues for Cadmium is small enough that maybe we should plan on > reviewing it at the same time as everything else...? Sounds good to me, the more people who look at it, the better. -- Rich