Code review comment for lp:~seb128/ubuntu-system-settings/security-trust-localized-names

Revision history for this message
Sebastien Bacher (seb128) wrote :

> Just a few comments, but otherwise it looks good!

thanks, addressed those

> By the way, do we have a dependency on libglib2.0-dev in debian/control?

yes we do, quite some of the backends use glib or glib based libraries

> Oh, and please remove the QSettings #include.

done as well

« Back to merge proposal