Opened on 09/22/2016 at 08:18:56 AM
Closed on 09/22/2016 at 05:14:54 PM
#4458 closed change (fixed)
Update dependencies for updateUpdateManifests script
Reported by: | trev | Assignee: | trev |
---|---|---|---|
Priority: | P1 | Milestone: | |
Module: | Infrastructure | Keywords: | |
Cc: | matze | Blocked By: | #4340, #4452 |
Blocking: | Platform: | Unknown / Cross platform | |
Ready: | yes | Confidential: | no |
Tester: | Unknown | Verified working: | no |
Review URL(s): |
https://codereview.adblockplus.org/29354719/ |
Description
Background
With #4340 and #4452 the updateUpdateManifests script now uses PyCrypto rather than M2Crypto. The update server configuration hasn't been updated yet.
What to change
Make sure to install python-crypto package.
Attachments (0)
Change History (8)
comment:1 Changed on 09/22/2016 at 08:20:09 AM by trev
- Review URL(s) modified (diff)
- Status changed from new to reviewing
comment:2 Changed on 09/22/2016 at 08:20:32 AM by trev
- Blocked By 4340, 4452 added
comment:3 Changed on 09/22/2016 at 08:30:12 AM by abpbot
comment:4 Changed on 09/22/2016 at 08:39:16 AM by trev
- Resolution set to fixed
- Status changed from reviewing to closed
comment:5 Changed on 09/22/2016 at 01:17:26 PM by trev
- Resolution fixed deleted
- Status changed from closed to reopened
Reopening - the installed version of PyCrypto is too old.
comment:6 Changed on 09/22/2016 at 01:18:00 PM by trev
- Review URL(s) modified (diff)
- Status changed from reopened to reviewing
comment:7 Changed on 09/22/2016 at 05:10:50 PM by abpbot
A commit referencing this issue has landed:
Issue 4458 - Use pip to install current versions of dependencies for updateUpdateManifests script
comment:8 Changed on 09/22/2016 at 05:14:54 PM by trev
- Resolution set to fixed
- Status changed from reviewing to closed
Deployed, this will hopefully work now.
Note: See
TracTickets for help on using
tickets.
A commit referencing this issue has landed:
Issue 4458 - Update dependencies for updateUpdateManifests script