uses deprecated Branch.revision_history call
Bug #903703 reported by
Jelmer Vernooij
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
High
|
Jelmer Vernooij | ||
Bug Description
Launchpad uses the Branch.
Instead, Launchpad should be accessing only those parts of the branch history that it needs.
Related branches
lp:~jelmer/launchpad/no-revhistory-1
- Graham Binns (community): Approve (code)
-
Diff: 75 lines (+7/-12)2 files modifiedlib/lp/codehosting/codeimport/tests/test_worker.py (+6/-10)
lib/lp/codehosting/codeimport/tests/test_workermonitor.py (+1/-2)
lp:~jelmer/launchpad/no-revhistory-2
- Graham Binns (community): Approve
-
Diff: 218 lines (+62/-13)7 files modifiedlib/lp/code/bzr.py (+24/-0)
lib/lp/code/model/branchjob.py (+5/-2)
lib/lp/code/model/tests/test_branchjob.py (+6/-5)
lib/lp/code/tests/test_bzr.py (+21/-1)
lib/lp/codehosting/codeimport/tests/test_worker.py (+1/-2)
lib/lp/codehosting/scanner/bzrsync.py (+2/-1)
lib/lp/codehosting/scanner/tests/test_bzrsync.py (+3/-2)
lp:~jelmer/launchpad/no-revhistory-3
- Graham Binns (community): Approve (code)
-
Diff: 177 lines (+94/-8)3 files modifiedlib/lp/code/bzr.py (+36/-0)
lib/lp/code/tests/test_bzr.py (+53/-5)
lib/lp/codehosting/scanner/bzrsync.py (+5/-3)
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #1 |
| tags: | added: qa-needstesting |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in launchpad: | |
| status: | Fix Committed → Fix Released |
| tags: | removed: qa-ok |
| Changed in launchpad: | |
| status: | Fix Released → Triaged |
| status: | Triaged → In Progress |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #2 |
| tags: | added: qa-needstesting |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
| tags: |
added: bad-commit-14787 qa-bad removed: qa-needstesting |
| Changed in launchpad: | |
| status: | Fix Committed → In Progress |
| tags: | removed: bad-commit-14787 qa-bad |
| Changed in launchpad: | |
| assignee: | Jelmer Vernooij (jelmer) → nobody |
| status: | In Progress → Triaged |
| Changed in launchpad: | |
| assignee: | nobody → Jelmer Vernooij (jelmer) |
| status: | Triaged → In Progress |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #3 |
| tags: | added: qa-needstesting |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #4 |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #5 |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in launchpad: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Fixed in stable r14502 <http://