Code review comment for lp://qastaging/~snwh/gnome-control-center-signon/high-contrast-icons

Revision history for this message
Alberto Mardegan (mardy) wrote :

Thanks Sam for working on this. Your patch however doesn't work: the "HighContrast" icons are correctly added to the resulting debian package, but the "hicolor" ones don't get installed anymore.
I think this is due to changing the value of the "iconthemedir" variable on the fly; maybe using a variable with a different name could help.

review: Needs Fixing

« Back to merge proposal