estin / geosuggest

suggest by name or find nearest by coordinates cities
MIT License
9 stars 1 forks source link

Replace/remove unmaintained procinfo crate? #2

Closed jqnatividad closed 1 year ago

jqnatividad commented 1 year ago

Hi @estin ,

It seems the crate is no longer maintained

https://github.com/danburkert/procinfo-rs/issues/46

Perhaps, sysinfo can be used instead? https://docs.rs/sysinfo/latest/sysinfo/

It's well-maintained and cross-platform.

Alternatively, as procinfo is only used for arguably edge-case memory tests, maybe even remove it?

estin commented 1 year ago

remove it - it's best choice. memory test it's edge case. cover case of often reloading geoip2 db in runtime. in my production usage of geosuggest it's not needed at all