Fix datagen with --segmenter-lstm-root

Manishearth · Nov 13, 2023 · 9c90987 · 9c90987
1 parent 4a6df7b
commit 9c90987
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -25,6 +25,7 @@
         - New CLI option `--format=blob2` (https://github.com/unicode-org/icu4x/pull/4207)
         - CLDR 44 compatibility fixes (https://github.com/unicode-org/icu4x/pull/4134, https://github.com/unicode-org/icu4x/pull/4156, https://github.com/unicode-org/icu4x/pull/4158)
         - Fix `supported_locales` for collator keys (https://github.com/unicode-org/icu4x/pull/4169)
+        - CLI: Fix behavior of `--segmenter-lstm-root` such that it does not override `icuexportdata-root` (https://github.com/unicode-org/icu4x/pull/4277)
 - Utilities
     - `databake`
         - Improvements `databake::test_bake!()` (https://github.com/unicode-org/icu4x/pull/4182)

diff --git a/provider/datagen/src/bin/datagen/mod.rs b/provider/datagen/src/bin/datagen/mod.rs
@@ -67,7 +67,7 @@ fn main() -> eyre::Result<()> {
     };
 
     provider = match config.segmenter_lstm {
-        config::PathOrTag::Path(path) => provider.with_icuexport(path)?,
+        config::PathOrTag::Path(path) => provider.with_segmenter_lstm(path)?,
         #[cfg(feature = "networking")]
         config::PathOrTag::Latest => {
             provider.with_segmenter_lstm_for_tag(DatagenProvider::LATEST_TESTED_SEGMENTER_LSTM_TAG)