fix license verification on windows?

elastic / elasticsearch

Free and Open, Distributed, RESTful Search Engine

https://www.elastic.co/products/elasticsearch

Other

69.42k stars 24.57k forks source link

fix license verification on windows? #12118

Closed rmuir closed 9 years ago

rmuir commented 9 years ago

I had to disable license verification on windows so that mvn verify would work in #12117

I installed strawberry perl and unzip.exe in my path, but the script will consistently compute the wrong sha1 checksum for lucene-analysis-kuromoji jar (i was testing with plugins/analysis-kuromoji)... i've verified (with windows fciv) that the checksum is correct, so there is something not right there.

Not sure its critical to fix this, its just omitted on windows right now.

s1monw commented 9 years ago

for now I think it's fine to just run this on unix though. @clintongormley what do you think?

rmuir commented 9 years ago

we turned this on for windows in #12528