Code review comment for lp:~smspillaz/compiz-core/compiz-core.dist-buildsystem

Revision history for this message
Aurélien Gâteau (agateau) wrote :

Looks nicer, just two minor nitpicks:
1. The "if" cmake command allows you to simplify the tests from "if (${IS_GIT_REPO})" to "if (IS_GIT_REPO)".
2. There is some commented code, not sure if it is intentional or not.

« Back to merge proposal