Closed Bug 1029870 Opened 6 years ago Closed 6 years ago
_incremental _update .sh to allow passing Product Version and Channel ID
Currently there is no way to pass the ChannelID and ProductVersion to the MAR binary at run time. See: http://hg.mozilla.org/mozilla-central/file/tip/tools/update-packaging/make_incremental_update.sh#l278 Adding provisions in the script to check $MOZ_CHANNEL_ID and $MOZ_PRODUCT_VERSION environment variables to allow for this at run time.
Adding patch. Note: The strange order of appending flags to the `$mar_command` is because the -c <output_file> flag needs to be at the end, else everything after the -c flag is treated as a filename Probably because of the way args are handled here: http://hg.mozilla.org/mozilla-central/file/a19e0434ea52/modules/libmar/tool/mar.c#l155
Assignee: nobody → ffledgling
Status: NEW → ASSIGNED
Attachment #8445637 - Flags: review?(nthomas)
Comment on attachment 8445637 [details] [diff] [review] Bug-1029870.patch Tested fine on our win32/mac/linux64 systems. If this works in the service too we can go ahead and land it.
Attachment #8445637 - Flags: review?(nthomas) → review+
How do we make sure it works in service as well?
Comment on attachment 8445637 [details] [diff] [review] Bug-1029870.patch https://hg.mozilla.org/integration/mozilla-inbound/rev/3fa5aac520f6 Expect the sheriffs to merge this to m-c within a day, closing the bug when they do.
Attachment #8445637 - Flags: checked-in+
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.