Looks like you put it in packages/theme-selva, correct?
I think it's fine to have it in CVS so we can all work on it together.
There's an open question about where we should be putting these themes (this is the first instance I know of a dotlrn theme landing in its own package), and how to make sure that we don't unnecessarily duplicate code in them, but these are secondary IMO.