Danek Duvall wrote: > On Thu, Oct 23, 2008 at 05:00:26PM -0700, Brock Pytlik wrote: > > >> Well, except that there are incompatible versions (version 0) for example. >> What I think I'll do is add another variable EARLIEST_COMPATIBLE_VERSION >> next to CURRENT_API_VERSION. In this case, that will be set to 1, and I'll >> so set(xrange(EARLIEST_COMPATIBLE_VERSION, CURRENT_API_VERSION + 1). >> >> Does that sound reasonable? >> > > Eh. It sounds like more work than I'd do, particularly if you'd have to > wrap the line. Up to you. > > Well, xrange(4) spits out 0, 1, 2, 3 which is wrong both in that it includes 0 (an incompatible version) and doesn't include 4 (which is a compatible version). Maybe I misunderstood your original idea (unlike K, I haven't had my coffee today ;) ). >> Do you (or anyone else) want to see another webrev, or is this ready to >> go back? >> > > I don't need to review this again. > Great, I'll look to get this back in soon then. > Danek >
_______________________________________________ pkg-discuss mailing list [email protected] http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
