lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Uwe Schindler" <>
Subject RE: ant precommit; annoyed with "checkout is dirty"
Date Mon, 26 Nov 2012 18:33:38 GMT
Hi David,

there are actually 2 reasons for this check:
- It prevents that you forget to add files before committing. We had this quite often, causing
unbuildable sources and Jenkins failures.
- Prevent test cases from leaving files around at wrong places. This was the original idea
behind the task, now obsoleted by the SecurityManager installed by the test-framework. Lots
of Solr tests did not set the working directory correctly and started to write files into
the src/ (!!!) folder. The check was added, so Jenkins could complain about those problems.
This is now solved in a better way, but the check is still good for non-tests. The installed
SecurityManager with a very strict security profile prevents wrong-behaving test - a test
that writes files outside the build directory will fail with a SecurityException (raised on
fileopen with write/append).


Uwe Schindler
H.-H.-Meier-Allee 63, D-28213 Bremen

> -----Original Message-----
> From: David Smiley ( []
> Sent: Monday, November 26, 2012 3:46 PM
> To:
> Subject: ant precommit; annoyed with "checkout is dirty"
> "ant precommit" will check if the source tree is "dirty" (i.e. contains files not
> in source control) and stop with a failure if so.  I find that rather annoying
> since I've usually got a variety of .patch files and IDE config changes.  What is
> the rationale behind this check?  How do people usually deal with it?
> Perhaps if I do my real development on another checkout (git based), I could
> then patch on the svn one for the commit.  Pretty annoying though, and the
> 4x port is yet another step.  There sure is a lot of burden to getting commits
> in.  I no longer care to improve little javadoc and typo stuff.
> ~ David
> -----
>  Author:
> server/book
> --
> View this message in context:
> precommit-annoyed-with-checkout-is-dirty-tp4022362.html
> Sent from the Lucene - Java Developer mailing list archive at
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: For additional
> commands, e-mail:

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message