[Merge] lp:~nskaggs/notes-app/ap-setup-update into lp:notes-app

Martin Pitt martin.pitt at ubuntu.com
Mon Jul 21 16:08:52 UTC 2014


Review: Needs Fixing

Mostly working now; the AP tests succeed, but autopkgtest fails on having a ton of stderr, due to several exceptions and the fake web server log: http://paste.ubuntu.com/7830998/

I think these are all just from the fake web server, so probably its output should be directed to /dev/null, or perhaps to a log file which you print on a test failure, or just generally put into $ADT_ARTIFACTS/web-server.log?

Alternatively, add the "allow-stderr" restriction.

Thanks!
-- 
https://code.launchpad.net/~nskaggs/notes-app/ap-setup-update/+merge/225992
Your team Ubuntu Phablet Team is subscribed to branch lp:notes-app.



More information about the Ubuntu-reviews mailing list