Age | Commit message (Collapse) | Author |
|
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
|
|
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]
|
|
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
|
|
|
|
|
|
Fixes #76
|
|
Fixes #77
|
|
|
|
|
|
|
|
Bump acorn from 6.1.1 to 6.4.1
|
|
Bump lodash from 4.17.11 to 4.17.15
|