Age | Commit message (Collapse) | Author |
|
am: 416012905c am: 82d8ad561d
Original change: https://android-review.googlesource.com/c/platform/external/timezone-boundary-builder/+/1287532
Change-Id: If7974414a9ec64c02181defa0f6ff3d261d00e2c
|
|
am: 416012905c
Original change: https://android-review.googlesource.com/c/platform/external/timezone-boundary-builder/+/1287532
Change-Id: Ie22be7be668f92590aa077503e8afd6b2078c594
|
|
Original change: https://android-review.googlesource.com/c/platform/external/timezone-boundary-builder/+/1287532
Change-Id: I776e11be18dcad224ffe38be3991742635500a7a
|
|
Original change: https://android-review.googlesource.com/c/platform/external/timezone-boundary-builder/+/1287532
Change-Id: Iebfb4e039f83968de65ed0c640c1dd807d789185
|
|
Android additions to timezone-boundary-builder to assist with the data
generation process execution.
Addition of some Docker infra to improve repeatability of execution
(not improve security). Also a README.md to explain the process.
Note: Use --skip_analyze_diffs for run_tzdb.sh as the analysis doesn't
work well with --dist_dir. Upstream fix to follow.
Bug: 190733142
Test: Manual testing
Change-Id: I7b6d54e8dc56a03444d4b71fe3891384a87ce2d6
|
|
98f13a6355 am: ec66c3d5b4 am: 473e4fddaf
Original change: https://android-review.googlesource.com/c/platform/external/timezone-boundary-builder/+/1733814
Change-Id: Id2680d68aef514806c2b9176041023d042f80df3
|
|
98f13a6355 am: ec66c3d5b4
Original change: https://android-review.googlesource.com/c/platform/external/timezone-boundary-builder/+/1733814
Change-Id: Ie08bb7edf5624f40c26a3d14671a8e37b0a0d979
|
|
98f13a6355
Original change: https://android-review.googlesource.com/c/platform/external/timezone-boundary-builder/+/1733814
Change-Id: I95d6c07a867501a7de27dc3a12328b6833012d8b
|
|
Original change: https://android-review.googlesource.com/c/platform/external/timezone-boundary-builder/+/1733814
Change-Id: I1289fa7f263c909b08aa45aefb92a3a52f76c7b7
|
|
Taking all upstream files as they are, i.e.
git checkout aosp/upstream-master -- <file>
Bug: 190733142
Test: Manual execution, etc.
Change-Id: Ifdb4f9cd29d379b67381f1d1e66ef767041edd66
|
|
README: Add Ruby library
|
|
|
|
|
|
|
|
|
|
|
|
Fixes #89
|
|
Fixes #83
|
|
|
|
|
|
|
|
Fixes #82
|
|
Fixes #90.
|
|
|
|
|
|
Add yargs for flags, and several follow-up commits for new flags
|
|
This commit fixes lint errors and updates package-lock.json /
package.json.
package-lock.json / package were generated by reverting:
git checkout 5ce291934eba5a2d114b756f33ff62f9ac0db632 -- \
package-lock.json package.json
... then running ...
npm install yargs
Note: timezone-boundary-builder ran without adding yargs explicitly
because it is a transitive dependency of some other packages already
used, but recording the dependency seems like good practice.
"npm audit fix" was _not_ run.
Tested with:
$ node --max-old-space-size=8192 index.js \
--downloads_dir=mydownloads \
--dist_dir=mydist \
--included_zones America/New_York America/Chicago
|
|
More fixes related to review feedback:
1) Added yargs to package.json, reran npm update (also updates
package-lock.json)
2) Added global variables for downloads_dir / dist_dir, which holds the
absolute path for argv.downloads_dir / argv.dist_dir after a call to
path.resolve(). Switched the code to use them.
3) Fixed hardcoding of "dist/" and "download/" in the zipInputData step.
Tested with:
$ node --max-old-space-size=8192 index.js \
--downloads_dir=mydownloads \
--dist_dir=mydist \
--included_zones America/New_York America/Chicago
|
|
|
|
Updated README.md and index.js
Tested by running:
$ node --max-old-space-size=8192 index.js \
--included_zones America/New_York America/Chicago \
--excluded_zones America/New_York \
--dist_dir ./dist3 \
--downloads_dir ./downloads2 \
--skip_zip \
--skip_shapefile
$ node --max-old-space-size=8192 index.js \
--included_zones America/New_York America/Chicago \
--excluded_zones America/New_York \
--dist_dir ./dist2 \
--downloads_dir ./downloads2
$ node --max-old-space-size=8192 index.js \
--included_zones America/New_York America/Chicago \
--excluded_zones America/New_York
[No failures, assumed output is as it was before changes]
|
|
Original change: https://android-review.googlesource.com/c/platform/external/timezone-boundary-builder/+/1288772
Change-Id: I0841dacf8dc8cd19ae762335cc012d2fd5add45c
|
|
Original change: https://android-review.googlesource.com/c/platform/external/timezone-boundary-builder/+/1283946
Change-Id: I7f69fe1bde9d2ecf2cb2d6e0c7ce319c63fb8991
|
|
persistent-https://android.git.corp.google.com/platform/external/timezone-boundary-builder into Merge2 am: 8457c14d36
Original change: https://android-review.googlesource.com/c/platform/external/timezone-boundary-builder/+/1283945
Change-Id: Ice7d30ed90a88c8de72091bc23e28d789ae2665a
|
|
Add flags to turn off of the later steps when not required.
Tested with:
$ node --max-old-space-size=8192 index.js \
--included_zones America/New_York America/Chicago \
--excluded_zones America/New_York \
--dist_dir ./dist3 \
--downloads_dir ./downloads2 \
--skip_zip \
--skip_shapefile
|
|
Tested with:
$ node --max-old-space-size=8192 index.js \
--included_zones America/New_York America/Chicago \
--excluded_zones America/New_York \
--dist_dir ./dist2 \
--downloads_dir ./downloads2
|
|
Tested with:
$ node --max-old-space-size=8192 index.js \
--included_zones America/New_York America/Chicago \
--excluded_zones America/New_York
|
|
Add yargs integration for command line flags in preparation for adding a
more. yargs doesn't appear to support the existing flags containing
minus ('-') characters in the middle, i.e. --no-validation and
--filtered-zones so these have been renamed to --no_validation and
--included_zones respectively.
--included_zones now takes a list without quotes or commas.
This change also updates the README.md example to demonstrate the
--included_zones flag.
|
|
Refs #82
Refs #83
|
|
|
|
|
|
Fixes #78
|
|
|
|
|
|
|
|
|
|
|
|
timezone
Refs #76
|
|
|
|
|
|
Refs #76
|