Hi Sean, I think I saw another potential problem, lines 233 to 237 should be changed from
if (tmpDir.exists()) { recursiveDelete(tmpDir); } else { tmpDir.mkdirs(); } to if (tmpDir.exists()) { recursiveDelete(tmpDir); } tmpDir.mkdirs(); to always make sure the temporary path exists. Sebastian Sean Owen schrieb: > That must be it. I had removed the '/' earlier since on OS X the temp > dir path ends with '/', and at the time I believed it was the cause of > some other failures (which I'm guessing I was wrong about). I can > easily make the logic account for both cases. > > Sean > > On Tue, Apr 6, 2010 at 11:24 AM, Sebastian Schelter > <sebastian.schel...@zalando.de> wrote: > >> Hi Sean, >> >> I can only do I guess why the test fails: >> >> Line 225 in ItemSimilarityTest is missing a / when constructing the path >> to the temporary directory: >> >> String tmpDirPath = System.getProperty("java.io.tmpdir") + >> ItemSimilarityTest.class.getCanonicalName(); >> >> which makes it >> >> /tmporg.apache.mahout.cf.taste.hadoop.similarity.item.ItemSimilarityTest >> >> on my system for example. Maybe the hudson user on the CI server is not >> allowed to create this path. >> >> Sebastian >> >>