scripts/mlist-sync.py should use LaunchpadScript as a base
Bug #652007 reported by
Curtis Hovey
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
High
|
Curtis Hovey | ||
Bug Description
scripts/
LaunchpadCronSc
would be a bug - it should either be in cronscripts, or not inherit
from LaunchpadScript.
If your scripts shouldn't be generating OOPS reports, we need to
discuss why they are inheriting from LaunchpadCronScript instead of
LaunchpadScript.
Related branches
lp:~sinzui/launchpad/mlist-sync-0
- Brad Crittenden (community): Approve (code)
-
Diff: 547 lines (+169/-320)5 files modifiedlib/lp/services/mailman/doc/staging.txt (+0/-159)
lib/lp/services/mailman/testing/__init__.py (+11/-7)
lib/lp/services/mailman/testing/sync.py (+0/-144)
lib/lp/services/mailman/tests/test_mlist_sync.py (+156/-0)
scripts/mlist-sync.py (+2/-10)
| Changed in launchpad-registry: | |
| status: | Triaged → In Progress |
| assignee: | nobody → Curtis Hovey (sinzui) |
| tags: | added: mailing-lists |
| Changed in launchpad-registry: | |
| status: | In Progress → Fix Committed |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote Bug fixed by a commit | #1 |
| tags: | added: qa-needstesting |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in launchpad-registry: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Fixed in stable r11691 <http://