I double checked (also 3.6.1, 3.6.2, and 4.2.1 releases): hossman's commit is correct actually.
This is the one used for back compat (its tested with the version comparator), but its just marking which version of lucene wrote the segment. The variable name should really be changed, "MAIN" says nothing. This one is the more important one though: another reason why its not triggered by the version sysprop from build.xml: our comparator cannot deal with any -SNAPSHOT or any maven suffixes or any of that horseshit. it needs real version numbers. On Wed, May 15, 2013 at 7:57 AM, Robert Muir <[email protected]> wrote: > On Wed, May 15, 2013 at 4:23 AM, Uwe Schindler <[email protected]> wrote: >> Are we sure that this is the right thing? The LUCENE_MAIN_VERSION is used >> for index compatibility and should always be only in X.Y format. >> >> Please revert this! >> > > Uwe is correct: please only adjust build.xml here or whatever, but > don't change this! --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
