Re: Build issue with last dirichlet change

2010-01-20 Thread Jeff Eastman
Whew! doing a clean build resolved the phantom test class. All tests pass, even mine :) Jeff Eastman wrote: The build compiles but org.apache.mahout.math.TestVectorWritable fails for some reason and it does not get to my test. Jeff Eastman wrote: I will run the build before I commit. I wil

Re: Build issue with last dirichlet change

2010-01-20 Thread Jake Mannix
TestVectorWritable fails? What is it saying in the surefire report? -jake On Wed, Jan 20, 2010 at 5:42 PM, Jeff Eastman wrote: > The build compiles but org.apache.mahout.math.TestVectorWritable fails for > some reason and it does not get to my test. > > > > Jeff Eastman wrote: > >> I will ru

Re: Build issue with last dirichlet change

2010-01-20 Thread Jeff Eastman
The build compiles but org.apache.mahout.math.TestVectorWritable fails for some reason and it does not get to my test. Jeff Eastman wrote: I will run the build before I commit. I will run the build before I commit. ... I will run the build before I commit. my bad Ted Dunning wrote: Our mod

Re: Build issue with last dirichlet change

2010-01-20 Thread Jeff Eastman
I will run the build before I commit. I will run the build before I commit. ... I will run the build before I commit. my bad Ted Dunning wrote: Our modules aren't working out as well as expected. On Wed, Jan 20, 2010 at 4:56 PM, Jeff Eastman wrote: Sean Owen wrote: That last commi

Re: Build issue with last dirichlet change

2010-01-20 Thread Jeff Eastman
I can easily move it into the /utils/test subtree. Tried that and it now builds ok. I actually like it fine there with the Lucene/text stuff. Sean Owen wrote: Actually, I suppose it's not a big deal if core *tests* depend on utils. I can make that adjustment if that sounds right. On Wed, Jan

Re: Build issue with last dirichlet change

2010-01-20 Thread Sean Owen
I don't think there's any practical or conceptual problem with core *tests* depending on utils. On Wed, Jan 20, 2010 at 5:13 PM, Benson Margulies wrote: > You may have hit a classic maven annoyance which is shared test code. > Generally, people seem to end up with more modules than they expected

Re: Build issue with last dirichlet change

2010-01-20 Thread Benson Margulies
You may have hit a classic maven annoyance which is shared test code. Generally, people seem to end up with more modules than they expected to. On Wed, Jan 20, 2010 at 8:06 PM, Ted Dunning wrote: > Our modules aren't working out as well as expected. > > On Wed, Jan 20, 2010 at 4:56 PM, Jeff Eastm

Re: Build issue with last dirichlet change

2010-01-20 Thread Ted Dunning
Our modules aren't working out as well as expected. On Wed, Jan 20, 2010 at 4:56 PM, Jeff Eastman wrote: > Sean Owen wrote: > >> That last commit concerning the dirichlet code and models seems to >> cause the build to fail -- or else I'm the victim of another >> environment-specific issue. >> >>

Re: Build issue with last dirichlet change

2010-01-20 Thread Sean Owen
Actually, I suppose it's not a big deal if core *tests* depend on utils. I can make that adjustment if that sounds right. On Wed, Jan 20, 2010 at 4:56 PM, Jeff Eastman wrote: > Sean Owen wrote: >> >> That last commit concerning the dirichlet code and models seems to >> cause the build to fail --

Re: Build issue with last dirichlet change

2010-01-20 Thread Jeff Eastman
Sean Owen wrote: That last commit concerning the dirichlet code and models seems to cause the build to fail -- or else I'm the victim of another environment-specific issue. I note it only because the fix raises a question. It causes core/ to depend utils/, and I had thought that was not the idea