search

FlexSearch / FlexLucene

IKVM Build scripts
Apache License 2.0
16 stars 5 forks source link

Platform is x64 #1

Closed ltrzesniewski closed 10 years ago

ltrzesniewski commented 10 years ago

Hi, This build script and the NuGet package target x64. Is there a specific reason for that? Lucene on IKVM should theoretically work just fine on AnyCPU. BTW, thanks for the convenient package.

seemantr commented 10 years ago

@ltrzesniewski Thanks for the kind words. Actually you are right, Lucene on IKVM will work with any target. This was mainly written to support FlexSearch which is a 64 bit only product. So, I didn't think about compiling it for any platform.

Now a days I have moved away from using Powershell and relying on F# based scripts. My new compile script uses java ivy behind the scene to get all necessary dependency and then compile it. This helps me in avoiding hard coding.

ltrzesniewski commented 10 years ago

Then please consider building FlexLucene for the AnyCPU target for your next NuGet release, as I'm sure many users would benefit from that.

For instance, I had to reconfigure my CI to explicitly run tests in x64 mode, which was somewhat inconvenient because of a library... but it was still more convenient than maintaining a port of Lucene on IKVM myself ;-) Too bad Lucene.NET lags so much behind the java version, but that's another story...

seemantr commented 10 years ago

@ltrzesniewski Sorry for the late reply. I was waiting for the new Lucene 4.9 to land. Now the newly generated dlls don't have the x64 tag. So hopefully your problem will be resolved.

ltrzesniewski commented 10 years ago

@seemantr Awesome, thanks!