search

valyala / gozstd

go wrapper for zstd
MIT License
432 stars 60 forks source link

Enable building and use of this package on illumos-based operating systems #46

Closed szaydel closed 1 year ago

szaydel commented 2 years ago

Thank you for considering this PR. I built the zstd library on a recent version of the OmniOS, which is an illumos-based operating system and included it with the tweak modeling already existing examples. I went ahead and confirmed that the package builds correctly, and I am able to leverage it in VictoriaMetrics, which depends on it.

szaydel commented 2 years ago

I just wanted to bump this request and see if there is anything more I can do to help move this forward. Thanks!

szaydel commented 1 year ago

Good day, I wanted to follow-up and see why there seems to be acknowledgement or feedback. If this is inappropriate for any reason, please do let me know. If you would like this updated since the PR is now stale, I am happy to do that. Any feedback would be better than nothing.

Thank you, Sam.

valyala commented 1 year ago

@szaydel , thanks for the pull request!

valyala commented 1 year ago

Sorry for the delay with the merge.

szaydel commented 1 year ago

@valyala , no problem at all with this. I am sure there is a lot on your plates. I appreciate it!