Closed
Bug 454115
Opened 16 years ago
Closed 15 years ago
make_incremental_updates.py doesn't use patch-if for extensions/ & searchplugins/ on Mac
Categories
(Release Engineering :: General, defect, P2)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: nthomas, Assigned: coop)
Details
Attachments
(1 file)
(deleted),
patch
|
nthomas
:
review+
coop
:
checked-in+
|
Details | Diff | Splinter Review |
I noticed while doing some testing for bug 414560, that the update manifest for a partial update is different when generated by make_incremental_updates.py, only on the mac though. Here's a diff between what patcher2 generates (slow mode), and what make_incremental_updates.py (fast) gives:
< patch-if "Contents/MacOS/extensions/{972ce4c6-7e08-4474-a285-3208198ce6fd}" "Contents/MacOS/extensions/{972ce4c6-7e08-4474-a285-3208198ce6fd}/install.rdf.patch" "Contents
/MacOS/extensions/{972ce4c6-7e08-4474-a285-3208198ce6fd}/install.rdf"
> patch "Contents/MacOS/extensions/{972ce4c6-7e08-4474-a285-3208198ce6fd}/install.rdf.patch" "Contents/MacOS/extensions/{972ce4c6-7e08-4474-a285-3208198ce6fd}/install.rdf"
51d51
ie slow mode is uses patch-if and fast isn't. (historical note: we use patch-if for stuff in extensions/ because DOMi and Talkback used to be optional installs on Windows, so the updater had to have some concept of "patch this if it's present, otherwise carry on").
This is a mac only "problem" because the fast mode using
if filename.startswith("extensions/"):
while on Mac it begins Contents/MacOS/extensions/...
Reporter | ||
Comment 1•16 years ago
|
||
Same for searchplugins, the code is at
http://mxr.mozilla.org/seamonkey/source/tools/update-packaging/make_incremental_updates.py#57
Summary: make_incremental_updates.py doesn't use patch-if for extensions/ on Mac → make_incremental_updates.py doesn't use patch-if for extensions/ & searchplugins/ on Mac
Updated•16 years ago
|
Component: Release Engineering → Release Engineering: Future
Priority: -- → P3
Assignee | ||
Updated•15 years ago
|
Assignee: nobody → ccooper
Assignee | ||
Updated•15 years ago
|
Status: NEW → ASSIGNED
Component: Release Engineering: Future → Release Engineering
Priority: P3 → P2
Assignee | ||
Comment 2•15 years ago
|
||
It looks like the add-if was susceptible to this too, so I fixed it up as well.
Attachment #416978 -
Flags: review?(nrthomas)
Reporter | ||
Updated•15 years ago
|
Attachment #416978 -
Flags: review?(nrthomas) → review+
Assignee | ||
Comment 3•15 years ago
|
||
Comment on attachment 416978 [details] [diff] [review]
Use patch-if on Mac extensions/ and searchplugins/
http://hg.mozilla.org/mozilla-central/rev/b45c4ef383b2
Attachment #416978 -
Flags: checked-in+
Assignee | ||
Comment 4•15 years ago
|
||
Checking in make_incremental_updates.py;
/cvsroot/mozilla/tools/update-packaging/make_incremental_updates.py,v <-- make_incremental_updates.py
new revision: 1.12; previous revision: 1.11
done
Holding off on landing to 1.9.0 and 1.9.1 pending kick-off of 3.0.17 and 3.5.7.
Reporter | ||
Comment 5•15 years ago
|
||
That's OK, this won't get picked up in releases until we do a new UPDATE_PACKAING_Rn tag in CVS.
Assignee | ||
Comment 6•15 years ago
|
||
http://hg.mozilla.org/releases/mozilla-1.9.1/rev/5c9730741212
http://hg.mozilla.org/releases/mozilla-1.9.2/rev/3887e9ab7690
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
Product: mozilla.org → Release Engineering
You need to log in
before you can comment on or make changes to this bug.
Description
•