Code review comment for lp://staging/~vrruiz/gnome-control-center-signon/coverage-fixes

Revision history for this message
David King (amigadave) wrote :

I do not see the Makefile.am.coverage file being added, and when I run autogen.sh from a clean checkout I get:

automake-1.11: cannot open < Makefile.am.coverage: No such file or directory

Did you test this? Additionally, you have provided no justification for this change in the description. Is it to match other webapps or Product Strategy projects, and if so, is the coverage Makefile.am snippet in a common repository anywhere?

review: Needs Fixing

« Back to merge proposal