yjh0502 / erl-brotli

Brotli encoder NIF for Erlang
Other
29 stars 17 forks source link

Update to brotli 1.0.1 #1

Closed jamesvl closed 7 years ago

jamesvl commented 7 years ago

Intent

This takes the c source code from the upstream Google brotli project and includes it here.

There are a few minor changes needed to the Makefile and _nif to get it to compile given their code layout changes.

Other Changes

Also updated our library name to match the name of the .so being created.

Minor changes to the README to show Elixir-folks how to use the library.

yjh0502 commented 7 years ago

@jamesvl Thanks!

jamesvl commented 7 years ago

No problem!

Although I just noticed some interesting behavior in the Makefile that caused a small issue - I'll open an Issue to describe it and you can decide how to best proceed.