Code review comment for lp://staging/~osomon/notes-app/fix-swipe-to-delete-tests
Revision history for this message
| Olivier Tilloy (osomon) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
513534c
demo site
(Get the code!)
> > Thanks Francis for the detailed investigation.
> > Is the daily-build PPA disabled from the CI config for notes-app now? If
> not,
> > can we remove it?
>
> I will do that as a short term solution. In general we want to avoid this as
> it makes resolving build dependencies harder and forces issues to just move
> from upstream merger to daily release integration testing. In other words, we
> may be able to get your change in trunk, but it might be blocked by the same
> problem in daily release testing.
I’m fine with keeping the PPA enabled, but then we need to understand why building with it fails, and we need to fix the problem. Any idea which dependencies are breaking those tests? Or can you point me to the relevant logs so I can look into it myself?