Closed Bug 1071107 Opened 10 years ago Closed 10 years ago

Sisyphus - replace get-extension-uiid.pl, work around missing changeset in build txt files, workaround extension installation crash, work around kill firefox on windows 7

Categories

(Testing Graveyard :: Sisyphus, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: bc, Assigned: bc)

References

Details

Attachments

(4 files)

various fixes:

replace perl version of get-extension-uiid.pl with python version get-extension-uuid.py since the perl version depends on various cpan modules which are problematic.
Attachment #8493210 - Flags: review?(cbook)
work around missing changeset in build txt files when building the build database for bisecting. See
http://stage.mozilla.org/pub/mozilla.org/firefox/nightly/2014-09-21-mozilla-central-debug/firefox-35.0a1.en-US.debug-linux-x86_64.txt
Attachment #8493211 - Flags: review?(cbook)
bug 1069776 introduced a crash on shutdown when installing extensions. Workaround it.
Attachment #8493212 - Flags: review?(cbook)
Attached patch force-kill.patchSplinter Review
kill firefox has been unreliable on windows 7 32bit due to incorrect detection of process ids.
Attachment #8493213 - Flags: review?(cbook)
Comment on attachment 8493210 [details] [diff] [review]
get-extension-uuid.patch

looks great thanks Bob
Attachment #8493210 - Flags: review?(cbook) → review+
Attachment #8493211 - Flags: review?(cbook) → review+
Comment on attachment 8493212 [details] [diff] [review]
install-extension-crash.patch

Review of attachment 8493212 [details] [diff] [review]:
-----------------------------------------------------------------

thanks bob, maybe we need to review this again when the regression is fixed
Attachment #8493212 - Flags: review?(cbook) → review+
Attachment #8493213 - Flags: review?(cbook) → review+
Product: Testing → Testing Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: