On Sat, 2007-03-10 at 10:57 -0600, Tom 'spot' Callaway wrote: > On Fri, 2007-03-09 at 15:54 -0500, Robin Norwood wrote: > > "Tom 'spot' Callaway" <tcallawa@xxxxxxxxxx> writes: > > > > > On Fri, 2007-03-09 at 11:28 -0500, Robin Norwood wrote: > > > > > >> Looks pretty good to me at first glace - I'm not totally convinced we > > >> want to do this right now though...I'd like to get some more discussion > > >> on the list first. > > >> > > >> We'll probably want to re-enable 'make test' at some point, too. ;-) > > >> > > >> Since this isn't quite done yet, how about: > > >> > > >> o I build a perl with all the changes except this one, including a > > >> 'Requires: perl-devel' for the main perl package. This would render the core purpose of this split ad absurdum. > > >> o We let people chime in on your changes over the weekend. > > >> o If people like these changes, we'll get them into rawhide next week. > > > > > > It seems reasonable to me. There are some minor cosmetic changes I'd > > > like to make (perl_version as opposed to perl_vers, use perl_version > > > everywhere, macroize the other versions so they're easily fixable with > > > new perl releases). A matter of personal preference, but no problem with me. > > > I was initially concerned about circular deps (perl-devel will require > > > all of these new subpackages), Hmm? Where do you see circular deps? Could you point them out? At least I don't see any. > > > but perl won't need them as > > > BuildRequires, so it should be ok. > > Ok - I've got a -15 run through the build system. It seems to work for > > me. > > One other item: > > We probably should subpackage ExtUtils-Embed too. I had considered doing this, but had abandoned this thought, because there doesn't seem to exist a current standalone ExtUtils::Embed perl dist (AFAIS, current EU::Embed's are only available through the main perl-tarball. The EU::Embed dist CPAN lists seems to refer to an abandoned version). Ralf