Opened 3 years ago

Last modified 7 weeks ago

#820 new task

Would like to contribute to making smartctl a static library

Reported by: Allan Liu Owned by:
Priority: minor Milestone: undecided
Component: all Version: 6.5
Keywords: library Cc:

Description

Hi,

We had a need to utilize smartctl as a library for one of our infrastructure management applications, but didn't seem like one will ever be on the road map, so I made one to get us what we need in the interim. This has proved to be rather useful for us, and we would love to contribute back to the project. However, I didn't really see any place to properly put up a PR so I put one up on the github mirror (https://github.com/mirror/smartmontools/pull/2). Would love to get some feedback from you guys (and obviously the opportunity to contribute back). Thanks!

  • Allan

Change History (4)

comment:1 in reply to:  description Changed 3 years ago by Christian Franke

Milestone: undecided

Thanks for your contribution.

..., but didn't seem like one will ever be on the road map, ...

This is longer true, as I'm working on a more generic approach for a smartmontools API/library. The first draft will be published sometimes after Release 6.6.

This has proved to be rather useful for us, and we would love to contribute back to the project.

Thanks - but sorry, your approach is rather incompatible to my current approach for an API. So it is rejected for now.

However, I didn't really see any place to properly put up a PR so I put one up on the github mirror (https://github.com/mirror/smartmontools/pull/2).

Please note that this mirror project is independent from smartmontools developers and probably will never accept your pull requests. We still use SVN as upstream repository.

Feel free to create a fork of this mirror and add your changes there. Then I would add this to our Links page.

comment:2 Changed 3 months ago by Allan Liu

Hi Christian,

Totally forgot to fill you in on this from a couple years ago, but I made a simple lib from the smartmontools fork here: https://github.com/allanliu/smartmontools

I'm going to begin work on adding NVMe support soon. Would be great if you could still add this to your links page.

Thanks,

Allan

comment:3 Changed 3 months ago by Christian Franke

Would be great if you could still add this to your links page.

Of course - see this change of Links page.

BTW: We now have an official GitHub mirror.

comment:4 Changed 7 weeks ago by Allan Liu

Awesome! Thanks Christian!

Note: See TracTickets for help on using tickets.