Releases/Firefox 17.0b1/BuildNotes: Difference between revisions

no edit summary
No edit summary
No edit summary
Line 96: Line 96:
</pre>
</pre>


=== Problem with win32 'km' locale ===
=== Update generation failed because of missing 'km' build ===
The win32 km locale failed to build with the following error:
Update generation failed with:
<pre>
<pre>2012-10-11 08:22:03,160 - DEBUG - requests.packages.urllib3.connectionpool._make_request#267: "GET /pub/mozilla.org/firefox/nightly/17.0b1-candidates/build2/win32/km/firefox-17.0b1.checksums HTTP/1.1" 404 288
Traceback (most recent call last):
Traceback (most recent call last):
   File "e:/builds/moz2_slave/rel-m-beta-w32-rpk-4/mozilla-beta/toolkit/mozapps/installer/windows/nsis/preprocess-locale.py", line 321, in <module>
   File "tools/scripts/updates/create-snippets.py", line 90, in <module>
     preprocess_locale_files(config_dir, l10n_dirs)
    info = getChecksum(platform, locale, checksumsFile)
   File "e:/builds/moz2_slave/rel-m-beta-w32-rpk-4/mozilla-beta/toolkit/mozapps/installer/windows/nsis/preprocess-locale.py", line 110, in preprocess_locale_files
  File "tools/scripts/updates/create-snippets.py", line 72, in getChecksum
     True)
    contents = requests.get(checksumsUrl, config={'danger_mode': True}).content
   File "e:/builds/moz2_slave/rel-m-beta-w32-rpk-4/mozilla-beta/toolkit/mozapps/installer/windows/nsis/preprocess-locale.py", line 48, in get_locale_strings
  File "/builds/slave/rel-m-beta-updates/tools/lib/python/vendor/requests-0.10.8/requests/api.py", line 51, in get
     if value[-1] == "\"" and value[0] == "\"":
     return request('get', url, **kwargs)
IndexError: string index out of range
   File "/builds/slave/rel-m-beta-updates/tools/lib/python/vendor/requests-0.10.8/requests/api.py", line 39, in request
make[2]: *** [l10ngen/setup.exe] Error 1
     return s.request(method=method, url=url, **kwargs)
make[1]: *** [repackage-win32-installer] Error 2
   File "/builds/slave/rel-m-beta-updates/tools/lib/python/vendor/requests-0.10.8/requests/sessions.py", line 203, in request
make: *** [repackage-win32-installer-km] Error 2
     r.send(prefetch=prefetch)
  File "/builds/slave/rel-m-beta-updates/tools/lib/python/vendor/requests-0.10.8/requests/models.py", line 585, in send
    self.response.raise_for_status()
  File "/builds/slave/rel-m-beta-updates/tools/lib/python/vendor/requests-0.10.8/requests/models.py", line 810, in raise_for_status
    raise http_error
requests.exceptions.HTTPError: 404 Client Error
</pre>
</pre>


Filed {{bug|800329}}. Release Management said we can ship without km, so we're not going to try to fix it for this release. Used "force build" to start the "win32 repack complete" builder to get downstream jobs firing.
To fix, had to manually change the patcher config to exclude 'km' from win32 for 17.0b1: https://hg.mozilla.org/build/tools/rev/f85e15d49942. Retagged tools with FIREFOX_17_0b1_{RELEASE,BUILD2} and used "force build" to start updates again.
 
Failed again because "updates" regenerated the patcher config from shipped locales. Manually changed shipped-locales this time, on the relbranch: https://hg.mozilla.org/releases/mozilla-beta/rev/a3cbcfe120bd. Retagged tools with FIREFOX_17_0b1_{RELEASE,BUILD2} and used "force build" to start updates again.
 
Failed again because it still tried to download checksums file. FIXME there's a bug in the new patcher config code that means we don't support removed locales. Next around attempt: faking out the checksums and removing the snippets later:
# ffxbld@stage
cd /pub/mozilla.org/firefox/candidates/17.0b1-candidates/build2/win32
mkdir km
cd km
cp ../ach/*.checksums .
 
Then forced the builder again.
 
After it completed, had to do some cleanup on stage:
# ffxbld@stage
cd /pub/mozilla.org/firefox/candidates/17.0b1-candidates/build2/win32
rm -rf km
 
And in the snippets:
# ffxbld@aus3-staging
cd /opt/aus2/snippets/staging
find Firefox-17.0b1-build2 -maxdepth 5 -type d -wholename '*WINNT*km*' -exec rm -rf {} \;
 
Didn't bother deleting km test snippets from the test snippets. They'll get overwritten in 17.0b2.
canmove, Confirmed users
6,441

edits