search

clearlinux / micro-config-drive

An alternative and small cloud-init implementation in C
Other
46 stars 17 forks source link

Various cleanups #23

Closed ikeydoherty closed 8 years ago

ikeydoherty commented 8 years ago

Various cleanups/fix-ups needed in the codebase.

Note that I originally intended to include a new make coverage target, but the way in which the make check target is actually handled as a SUBDIR breaks distcheck by leaving stray *.gcno files. This should ideally be changes to a common .a to be shared among the test suite and the main binary in future, with a top-level check, coverage, and check-valgrind (Out of scope for this PR): https://github.com/ikeydoherty/micro-config-drive/commit/62f215cd23ec4ee6f723eceedf661d04633d686c

sofar commented 8 years ago

:+1: these all look great, thanks.

devimc commented 8 years ago

:+1:

sofar commented 8 years ago

I'm sorry, I should have closed this one since I already merged it. @devimc.