mirror of
https://github.com/nodejs/node.git
synced 2024-11-21 10:59:27 +00:00
tools: fix timezone update tool
The spawnSync call was previously silently failing with this error: ```sh icupkg: unable to open input file "icudt*.dat" ``` because spawnSync doesn't support globbing. This change replaces the spawnSync call with execSync because that supports globbing. I have tested this workflow with some minor modifications in my fork and I can confirm that it works as expected now. The bot opened this PR - https://github.com/RaisinTen/node/pull/2 which updates deps/icu-small/source/data/in/icudt71l.dat.bz2. Fixes: https://github.com/nodejs/node/issues/44865 Signed-off-by: Darshan Sen <raisinten@gmail.com> PR-URL: https://github.com/nodejs/node/pull/44870 Reviewed-By: Richard Lau <rlau@redhat.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
This commit is contained in:
parent
22c39b1ddd
commit
71e9bc1108
@ -1,6 +1,6 @@
|
||||
#!/usr/bin/env node
|
||||
// Usage: tools/update-timezone.mjs
|
||||
import { execSync, spawnSync } from 'node:child_process';
|
||||
import { execSync } from 'node:child_process';
|
||||
import { renameSync, readdirSync, rmSync } from 'node:fs';
|
||||
import { exit } from 'node:process';
|
||||
|
||||
@ -26,13 +26,7 @@ if (latestVersion === currentVersion) {
|
||||
execSync('bzip2 -d deps/icu-small/source/data/in/icudt*.dat.bz2');
|
||||
fileNames.forEach((file) => {
|
||||
renameSync(`icu-data/tzdata/icunew/${latestVersion}/44/le/${file}`, `deps/icu-small/source/data/in/${file}`);
|
||||
spawnSync(
|
||||
'icupkg', [
|
||||
'-a',
|
||||
file,
|
||||
'icudt*.dat',
|
||||
], { cwd: 'deps/icu-small/source/data/in/' }
|
||||
);
|
||||
execSync(`icupkg -a ${file} icudt*.dat`, { cwd: 'deps/icu-small/source/data/in/' });
|
||||
rmSync(`deps/icu-small/source/data/in/${file}`);
|
||||
});
|
||||
execSync('bzip2 -z deps/icu-small/source/data/in/icudt*.dat');
|
||||
|
Loading…
Reference in New Issue
Block a user