search

checkpoint-restore / checkpointctl

A tool for in-depth analysis of container checkpoints
Apache License 2.0
87 stars 15 forks source link

make: Add install and uninstall targets #41

Closed rst0git closed 1 year ago

rst0git commented 1 year ago

This pull request adds install and uninstall Makefile targets for the checkpointctl tool.

adrianreber commented 1 year ago

Not sure, but would a make install and make uninstall during test make sense?

adrianreber commented 1 year ago

Maybe to the default location and some random location.

rst0git commented 1 year ago

I've updated the pull request with a test for the install/uninstall make targets and documentation for the steps to install checkpointctl from source code.

codecov-commenter commented 1 year ago

Codecov Report

Patch and project coverage have no change.

Comparison is base (0560541) 63.77% compared to head (6a9b0ee) 63.77%.

:exclamation: Current head 6a9b0ee differs from pull request most recent head 61233d6. Consider uploading reports for the commit 61233d6 to get more accurate results

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #41 +/- ## ======================================= Coverage 63.77% 63.77% ======================================= Files 3 3 Lines 254 254 ======================================= Hits 162 162 Misses 69 69 Partials 23 23 ``` Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.