Closed Bug 617188 Opened 9 years ago Closed 9 years ago

updater parameters in update_verify should be changed according to the new syntax

Categories

(Release Engineering :: General, defect, P1)

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: rail, Assigned: rail)

References

Details

Attachments

(3 files, 3 obsolete files)

bug 600777 changed command line parameters of updater(.exe) and added a mandatory apply-to-dir parameter. Full syntax is:

Usage: updater update-dir apply-to-dir [wait-pid [callback-working-dir callback-path args...]]

At the moment we call updater as
../../update/updater ../../update 0

It tries to chdir to "0" and fails.
Attachment #495705 - Flags: review?(nrthomas)
Attached patch update UpdateVerifyFactory (obsolete) — Splinter Review
Attachment #495706 - Flags: review?(nrthomas)
Attached patch configs (obsolete) — Splinter Review
Attachment #495707 - Flags: review?(nrthomas)
Testing the patches in staging. So far verify.sh runs good.
I think I would have set it up as use_old_updater so that you can set the global 'use_new_updater': True, with False on 1.9.1 and 1.9.2, and we don't have to pass --new-updater for future releases. Do we have time to do that ?
(In reply to comment #5)
> I think I would have set it up as use_old_updater so that you can set the
> global 'use_new_updater': True, with False on 1.9.1 and 1.9.2, and we don't
> have to pass --new-updater for future releases. Do we have time to do that ?

Sure. I'll update the patches today.
Attachment #495705 - Attachment is obsolete: true
Attachment #495808 - Flags: review?(nrthomas)
Attachment #495705 - Flags: review?(nrthomas)
Attachment #495706 - Attachment is obsolete: true
Attachment #495809 - Flags: review?(nrthomas)
Attachment #495706 - Flags: review?(nrthomas)
Attached patch configsSplinter Review
Attachment #495707 - Attachment is obsolete: true
Attachment #495810 - Flags: review?(nrthomas)
Attachment #495707 - Flags: review?(nrthomas)
Attachment #495808 - Flags: review?(nrthomas) → review+
Attachment #495809 - Flags: review?(nrthomas) → review+
Comment on attachment 495810 [details] [diff] [review]
configs

>diff --git a/mozilla/config.py b/mozilla/config.py
>+    # Bug 617188
>+    'use_old_updater': False,

Nit: The comment isn't necessary. 

Please let the Thunderbird and SeaMonkey guys know about this change in case they're using the 0.8.0 automation and/or it will also be landing on the 0.7 branch.
Attachment #495810 - Flags: review?(nrthomas) → review+
Comment on attachment 495808 [details] [diff] [review]
verify.sh: add --old-updater parameter

http://hg.mozilla.org/build/tools/rev/89131c300c1c
Attachment #495808 - Flags: checked-in+
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Blocks: 627271
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.