MeteoGroup / jbrotli

The repo has been migrated to Bitbucket https://bitbucket.org/dtnse/jbrotli/src/master/ and archived
Apache License 2.0
93 stars 21 forks source link

Create a multi-architecture jar #10

Open DavyLandman opened 7 years ago

DavyLandman commented 7 years ago

Have you thought about creating a multi-architecture jar and at init time look at the current architecture and load that native library?

zstd-jni and lz4-java do this. This turns it into one big jar instead of several jars. This in turn simplifies releases of applications that use jbrotli.

For the furture, for unsupported platforms, we could use the java decoder implementation that has surfaced in the brotli repo. Would be a nice fallback.

xiaodongw commented 7 years ago

+1 for it. And also can we have brotli packaged as a static lib? Just like netty-tcnative-boringssl-static (http://netty.io/wiki/forked-tomcat-native.html), so we do not have to worry about platform library versions.