Opened on 04/29/2014 at 12:18:32 PM
Closed on 04/30/2014 at 12:54:45 PM
Last modified on 10/22/2014 at 11:32:51 AM
#399 closed change (fixed)
Generate update manifests for Safari releases
| Reported by: | trev | Assignee: | sebastian |
|---|---|---|---|
| Priority: | P1 | Milestone: | |
| Module: | Infrastructure | Keywords: | |
| Cc: | Blocked By: | ||
| Blocking: | Platform: | ||
| Ready: | yes | Confidential: | no |
| Tester: | Verified working: | no | |
| Review URL(s): | |||
Description
Background
sitescripts.extensions.bin.updateDownloadLinks script is currently responsible for both retrieving download links for the most current release and generating update manifests. However, it can currently only generate update manifests for Gecko and Android extensions.
What to change
Add support for Safari update manifests so that these manifests are also generated.
Attachments (0)
Change History (7)
comment:1 Changed on 04/29/2014 at 03:36:42 PM by sebastian
- Owner set to sebastian
- Status changed from new to assigned
comment:2 Changed on 04/30/2014 at 09:50:33 AM by philll
- Status changed from assigned to new
comment:3 Changed on 04/30/2014 at 11:44:24 AM by sebastian
- Review URL(s) modified (diff)
- Status changed from new to reviewing
comment:4 Changed on 04/30/2014 at 12:26:24 PM by sebastian
- Resolution set to fixed
- Status changed from reviewing to closed
comment:5 Changed on 04/30/2014 at 12:47:05 PM by trev
- Resolution fixed deleted
- Status changed from closed to reopened
The script errored out on the server:
subprocess.CalledProcessError: Command '['hg', '-R', '.../hg/adblockplusie', 'cat', '-r', 'tip', '.../hg/adblockplusie/metadata']' returned non-zero exit status 1
Sounds like a check is missing to skip unknown repository types. The unknown repository types exists because the same configuration is used for devbuilds.
comment:6 Changed on 04/30/2014 at 12:54:45 PM by sebastian
- Resolution set to fixed
- Status changed from reopened to closed
comment:7 Changed on 10/22/2014 at 11:32:51 AM by sebastian
- Cc sebastian removed
The assigned state will be dropped by #403