[Merge] lp:~indicator-applet-developers/dbus-test-runner/no-werror into lp:dbus-test-runner

Sebastien Bacher seb128 at ubuntu.com
Tue Jan 15 09:07:07 UTC 2019


Review: Needs Information

Thanks for the work, the 'removing -Werror' part looks good to me but I don't understand the code change/new symbol, why do we need that to be able to remove Werror? It would be nice if those changes who be at least described in the changelog (or split in another commit if that's orthogonal to the dropping of Werror)
-- 
https://code.launchpad.net/~indicator-applet-developers/dbus-test-runner/no-werror/+merge/361756
Your team Indicator Applet Developers is subscribed to branch lp:dbus-test-runner.



More information about the Ubuntu-reviews mailing list