RE: analytics.adoc validation does not pass

2020-01-26 Thread Uwe Schindler
I fixed it, It was a bug in the checker script. Somebody defined Windows Line endings in a backwards way: diff --git a/lucene/tools/src/groovy/check-source-patterns.groovy b/lucene/tools/src/groovy/check-source-patterns.groovy index 62565e6..4688584 100644 --- a/lucene/tools/src/groovy/chec

RE: analytics.adoc validation does not pass

2020-01-26 Thread Uwe Schindler
I found the bug, it's a stupid one: The line splitter is: def singleLineSplitter = ~$/\n\r?/$; But should be (as it's \r\n on windows not the other way round): def singleLineSplitter = ~$/\r?\n/$; I'll commit the fix. It's just plain wrong! Uwe - Uwe Schindler Achterdiek 19, D-28357 Bremen

Re: analytics.adoc validation does not pass

2020-01-26 Thread Mikhail Khludnev
Looking into. On Sun, Jan 26, 2020 at 12:56 PM Dawid Weiss wrote: > Hi Cassandra! > > I get this when running precommit on gradle build (gradlew precommit): > > > Task :validateSourcePatterns > [ant:groovy] Unescaped symbol "->" on line #43: > solr/solr-ref-guide/src/analytics.adoc > [ant:groovy

Re: analytics.adoc validation does not pass

2020-01-26 Thread Dawid Weiss
Maybe. But I truly hate when git tries to alter those line endings for me... this hasn't been the first time I spent an hour looking at what the hell is wrong between Windows/ Linux on the same checkout... D. On Sun, Jan 26, 2020 at 11:34 AM Uwe Schindler wrote: > > But we should nevertheless ch

Re: analytics.adoc validation does not pass

2020-01-26 Thread Uwe Schindler
But we should nevertheless change the validator script to not fail on that. I think the split by lines regex is problematic. Uwe Am January 26, 2020 10:31:11 AM UTC schrieb Dawid Weiss : >Sigh... It's because of the recent change to solr/.gitattributes >(SOLR-10713)... End of lines differ for ad

Re: analytics.adoc validation does not pass

2020-01-26 Thread Dawid Weiss
Sigh... It's because of the recent change to solr/.gitattributes (SOLR-10713)... End of lines differ for adoc files. Jason -- please change this one from auto to "don't touch line endings on any of other files", whatever the magic is. # Handles all files not treated particularly below * t

Re: analytics.adoc validation does not pass

2020-01-26 Thread Uwe Schindler
On Jenkins (Ant build) it only seems to happen on Windows. Which is strange. Uwe Am January 26, 2020 10:13:44 AM UTC schrieb Dawid Weiss : >Ok, I think it's the validation code at fault here. Mike -- would you >take a look at why the validation of solr-ref-guide doesn't pass? >There has to be a d

Re: analytics.adoc validation does not pass

2020-01-26 Thread Dawid Weiss
Ok, I think it's the validation code at fault here. Mike -- would you take a look at why the validation of solr-ref-guide doesn't pass? There has to be a difference in infrastructure somewhere because we use the same validation script... Dawid On Sun, Jan 26, 2020 at 10:55 AM Dawid Weiss wrote:

analytics.adoc validation does not pass

2020-01-26 Thread Dawid Weiss
Hi Cassandra! I get this when running precommit on gradle build (gradlew precommit): > Task :validateSourcePatterns [ant:groovy] Unescaped symbol "->" on line #43: solr/solr-ref-guide/src/analytics.adoc [ant:groovy] Unescaped symbol "->" on line #52: solr/solr-ref-guide/src/analytics.adoc I don'