Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add implementation of sg_get_user_stats from C library libstatgrab latest version #18

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

bieli
Copy link

@bieli bieli commented Jan 29, 2024

I implemented statitics for sg_get_user_stats from libstatgrab library.
Original specification:
https://libstatgrab.org/docs/sg_get_user_stats.3.html

Added go modules, what is today standard and CI in github action, what is must have, too.

All modules from this library works well for me a long time on two platforms Linux and Darwin/MacOSX, so I send respect to author ;-)

@akhenakh akhenakh self-requested a review February 1, 2024 01:25
Copy link
Owner

@akhenakh akhenakh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is great, thanks, some cosmetic changes in the README

Comment on lines +3 to +5
![CI status](https://github.com/bieli/statgo/actions/workflows/ci.yaml/badge.svg)

![go report](https://goreportcard.com/badge/github.com/bieli/statgo)
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please change the url for the status back to this repo

@@ -27,6 +31,11 @@ Note: On OSX you need to install gcc to access cgo.

go get github.com/akhenakh/statgo

Note: On OSX you need to install brew and install library in OS with this cmd:
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note: On OSX, libstatgrab can be installed using brew:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants