-
Notifications
You must be signed in to change notification settings - Fork 166
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Create an ansible task for downloading icu #268
Comments
This is already included for the Windows machines: https://github.com/janeasystems/build/blob/aded718cdd1b826d0dfe508cc47b8d20a09df58e/setup/windows/ansible-playbook.yaml#L69-L88 |
Here's a slightly improved version I have as part of my "ansible 2.0 refactor": Edit: updated with 57.1. ---
# Downloads ICU and verifies checksums
# @requires dest=/tmp
- name: Download ICU
get_url:
- url: https://ssl.icu-project.org/files/icu4c/{{ item.version }}/icu4c-{{ item.version|replace('.','_') }}-src.tgz
- dest: {{ dest }}
- checksum: md5:{{ item.md5 }}
with_items:
- { version: '54.1', md5: 'e844caed8f2ca24c088505b0d6271bc0' }
- { version: '55.1', md5: 'e2d523df79d6cb7855c2fbe284f4db29' }
- { version: '56.1', md5: 'c4a2d71ff56aec5ebfab2a3f059be99d' }
- { version: '57.1', md5: 'f797503ecaebf1d38920013dc7893066' } |
Is this still an issue? Should it remain open? |
I don't believe this is an issue ever since Node.js started bundling and compiling against a stripped down ICU (icu-small, nodejs/node#6088). Now that Node.js v4.x is EOL I believe all LTS and current releases build this way and do not require an external ICU. |
This should be run for all release machines.
refs: #190
The text was updated successfully, but these errors were encountered: