Code review comment for lp:~kalikiana/midori/screensaver

Revision history for this message
Paweł Forysiuk (tuxator) wrote :

In WIN32 case I would add explicit HAVE_LIBNOTIFY=0 to match waf behaviour. When it is missing it defaults to 0 so it still builds and works fine but i think it is better to be explicit here. Otherwise looks sane.

review: Needs Fixing

« Back to merge proposal