jackrabbit-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Dominique Pfister <dominique.pfis...@day.com>
Subject Re: Build failed in Hudson: Jackrabbit-trunk » Jackrabbit Core #636
Date Thu, 09 Jul 2009 10:25:38 GMT

I was able to find out how those "repository" and "repository.xml" are
created; it is actually a combination of tests:

instantiates a TransientRepository with home=null/config=null and
o.a.j.core.RepositoryFactoryImpl registers this repository in its
static REPOSITORY_INSTANCES map, with key=null.

sets system properties with
home=target/repository/config=target/repository.xml and asks
o.a.j.core.RepositoryFactoryImpl to get or create a repository with
key=null. Unfortunately, the transient repository registered above is
still in the map and therefore this test inadvertently uses the wrong
repository, which will create those file system entries.

This can be reproduced by creating a read-only file "repository", and
running the two tests in sequence. I then get failures with stack

javax.jcr.RepositoryException: Invalid repository configuration file:
	at o.a.j.core.TransientRepository$2.getRepository(TransientRepository.java:230)
Caused by: o.a.j.ConfigurationException: Repository directory
repository does not exist
	at o.a.j.core.config.RepositoryConfig.create(RepositoryConfig.java:174)

I don't know exactly where this should be fixed: use system properties
in RepositoryFactoryTest as well or provide some clean up in
RepositoryFactoryImpl's REPOSITORY_INSTANCES map. Marcel, can you (or
Julian) take a look at this?


On Tue, Jun 30, 2009 at 11:28 PM, Tobias Bocanegra<tripod@day.com> wrote:
> hi,
>> For some reason the Hudson build creates an unexpected ./repository
>> directory (not ./target/repository) that gets flagged by RAT. I'm not
>> yet sure what causes that, but for now I simply added an extra RAT
>> exclude rule that should make the Hudson build succeed again.
> it's not just the hudson build - i have those "repository" and
> "repository.xml", too.
> i guess it's one of the tests that does not use the proper repository home.
> regards, toby

View raw message