Code review comment for lp:~mkhu/compiz/compiz.fix_1067246

Revision history for this message
Sam Spilsbury (smspillaz) wrote :

Just before this can be merged in, the install () command which installs whatever is in PUBLIC_HEADERS needs to be removed to (and that would effectively make the SET_TARGET_PROPERTIES call setting PUBLIC_HEADERS redundant in that directory too).

Right now, there are two install () statements that cover the same file.

(Sorry for the late review on that one, today's been kind of hectic, and the next week is going to be a bit all over the place too).

review: Needs Fixing

« Back to merge proposal