Closed
Bug 772853
Opened 11 years ago
Closed 11 years ago
upgrade clang to r160176
Categories
(Release Engineering :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: espindola, Assigned: espindola)
Details
Attachments
(2 files, 3 obsolete files)
482 bytes,
patch
|
rail
:
review+
|
Details | Diff | Splinter Review |
2.80 KB,
patch
|
rail
:
review+
|
Details | Diff | Splinter Review |
Opening the bug in releng. Let me know it should use some other component now that we have tooltool.
Assignee | ||
Updated•11 years ago
|
Attachment #641056 -
Attachment is patch: true
Attachment #641056 -
Flags: review?(rail)
Updated•11 years ago
|
Attachment #641056 -
Flags: review?(rail) → review+
Assignee | ||
Comment 1•11 years ago
|
||
for the record, this is what I did (I will cleanup the scripts and put the in m-c at some point): -------------------------------- $ cat ~/mozilla/create-manifest.sh #!/bin/sh set -e rm -f *.manifest tooltool.py setup.sh curl -O https://raw.github.com/jhford/tooltool/master/tooltool.py curl -O https://hg.mozilla.org/mozilla-central/raw-file/tip/build/unix/build-clang/setup.sh python tooltool.py -m linux32.manifest add clang-linux32.tar.bz2 setup.sh python tooltool.py -m linux64.manifest add clang-linux64.tar.bz2 setup.sh python tooltool.py -m darwin.manifest add clang-darwin.tar.bz2 setup.sh sed -i 's/clang-.*.tar.bz2/clang.tar.bz2/g' *.manifest python ~/mozilla/create-manifest-aux.py $ cat ~/mozilla/create-manifest-aux.py #!/bin/python import os for platform in ['darwin', 'linux32', 'linux64']: old_name = 'clang-' + platform + '.tar.bz2' manifest = platform + '.manifest' new_name = eval(file(manifest).read())[1]['digest'] os.rename(old_name, new_name) -------------------------- Run the script, copy the manifest to an m-c tree, edit to add the clang version. And then copy the files to the server: $ ssh respindola@people.mozilla.org -L 1500:relengweb1.dmz.scl3.mozilla.com:22 and in another terminal scp -P 1500 * respindola@localhost:/var/www/html/runtime-binaries/tooltool/sha512/
Assignee | ||
Updated•11 years ago
|
Summary: upgrade clang to r160034 → upgrade clang to r160105
Assignee | ||
Comment 2•11 years ago
|
||
Testing on try found an issue with r160034, which I fixed in 160105.
Assignee: nobody → respindola
Attachment #641056 -
Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #641473 -
Flags: review?(rail)
Updated•11 years ago
|
Attachment #641473 -
Flags: review?(rail) → review+
Assignee | ||
Comment 3•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/b24995e163b5
Assignee | ||
Comment 4•11 years ago
|
||
To be committed after a try run. I updated the scripts a bit (almost good enough for code review :-) ), which is why the indentation changed a bit: cat ~/mozilla/create-manifest-aux.py #!/bin/python import os import simplejson import sys def key_sort(item): item = item[0] if item == 'size': return 0 if item == 'digest': return 1 if item == 'algorithm': return 3 return 4 rev = os.path.basename(os.getcwd()).split('-')[1] for platform in ['darwin', 'linux32', 'linux64']: old_name = 'clang-' + platform + '.tar.bz2' manifest = platform + '.manifest' data = eval(file(manifest).read()) new_name = data[1]['digest'] data[1]['filename'] = 'clang.tar.bz2' data = [{'clang_version' : 'r%s' % rev }] + data out = file(manifest,'w') simplejson.dump(data, out, indent=0, item_sort_key=key_sort) out.write('\n') os.rename(old_name, new_name) #!/bin/sh set -e rm -f *.manifest tooltool.py setup.sh curl -O https://raw.github.com/jhford/tooltool/master/tooltool.py curl -O https://hg.mozilla.org/mozilla-central/raw-file/tip/build/unix/build-clang/setup.sh python tooltool.py -m linux32.manifest add clang-linux32.tar.bz2 setup.sh python tooltool.py -m linux64.manifest add clang-linux64.tar.bz2 setup.sh python tooltool.py -m darwin.manifest add clang-darwin.tar.bz2 setup.sh python ~/mozilla/create-manifest-aux.py
Attachment #641533 -
Flags: review?(rail)
Updated•11 years ago
|
Attachment #641533 -
Flags: review?(rail) → review+
Assignee | ||
Updated•11 years ago
|
Summary: upgrade clang to r160105 → upgrade clang to r160176
Assignee | ||
Comment 5•11 years ago
|
||
Sorry, I should have known that bugs is handling visibility attributes are never alone. I have fixed a few more of them and try is looking good so far with 160176: https://tbpl.mozilla.org/?tree=Try&pusher=respindola@mozilla.com
Attachment #641473 -
Attachment is obsolete: true
Attachment #641533 -
Attachment is obsolete: true
Attachment #642169 -
Flags: review?(rail)
Assignee | ||
Comment 6•11 years ago
|
||
Attachment #642170 -
Flags: review?
Assignee | ||
Updated•11 years ago
|
Attachment #642170 -
Flags: review? → review?(rail)
Updated•11 years ago
|
Attachment #642169 -
Flags: review?(rail) → review+
Updated•11 years ago
|
Attachment #642170 -
Flags: review?(rail) → review+
Comment 7•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/bc145c1e0647 https://hg.mozilla.org/mozilla-central/rev/024c3fafd531
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•10 years ago
|
Product: mozilla.org → Release Engineering
You need to log in
before you can comment on or make changes to this bug.
Description
•