Merge lp://qastaging/~mvo/software-center/track-axi-changes into lp://qastaging/software-center

Proposed by Michael Vogt
Status: Merged
Merged at revision: 2960
Proposed branch: lp://qastaging/~mvo/software-center/track-axi-changes
Merge into: lp://qastaging/software-center
Diff against target: 151 lines (+63/-4)
3 files modified
softwarecenter/db/database.py (+28/-2)
softwarecenter/paths.py (+4/-0)
test/test_database.py (+31/-2)
To merge this branch: bzr merge lp://qastaging/~mvo/software-center/track-axi-changes
Reviewer Review Type Date Requested Status
Gary Lasker (community) Approve
Review via email: mp+101494@code.qastaging.launchpad.net

Description of the change

This branch adds a monitor for changes in the apt-xapian-index to ensure the DB is reopened
if the cron job changes the apt-xapian-index database.

To post a comment you must log in.
2924. By Michael Vogt

softwarecenter/db/database.py: add comment about approach

2925. By Michael Vogt

add proper test and disconnect the signal to avoid multiple tracking of the changed signal

Revision history for this message
Michael Vogt (mvo) wrote :

This should fix bug #507836

2926. By Michael Vogt

merged from trunk

Revision history for this message
Gary Lasker (gary-lasker) wrote :

Very nice! And thanks for the unit test!!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches