Revert datagen fs feature

CHANGELOG.md

@@ -1,16 +1,13 @@
 # Changelog
 
-
-## icu4x 1.2.1 & 1.2.2 (Apr 17, 2023)
+## icu4x 1.2.1 (Apr 17, 2023)
 
 Note: A subset of crates received patch releases in the 1.2 stream.
 
 - `icu_capi`
   - Fixed [#3344](https://github.com/unicode-org/icu4x/pull/3344), `buffer_provider` feature accidentally pulling in extra crates
-- `icu_datagen` 1.2.1
+- `icu_datagen`
   - Fixed [#3339](https://github.com/unicode-org/icu4x/pull/3339), incorrect Cargo features
-- `icu_datagen` 1.2.2
-  - Fixed [#3354](https://github.com/unicode-org/icu4x/pull/3354), ability to run datagen without support for `icu_provider_fs::FsDataProvider`
 - `icu_locid_transform`
   - Fixed [#3332](https://github.com/unicode-org/icu4x/issues/3332), missing `+?Sized` bound
 - `icu_segmenter`

provider/datagen/Cargo.toml

@@ -51,7 +51,7 @@ icu_timezone = { version = "1.2.0", path = "../../components/timezone", features
 icu_provider = { version = "1.2.0", path = "../core", features = ["std", "log_error_context", "datagen"]}
 icu_provider_adapters = { version = "1.2.0", path = "../adapters", features = ["datagen"] }
 icu_provider_blob = { version = "1.2.0", path = "../blob", features = ["export"] }
-icu_provider_fs = { version = "1.2.0", path = "../fs", features = ["export"], optional = true }
+icu_provider_fs = { version = "1.2.0", path = "../fs", features = ["export"] }
 
 # Other
 cached-path = { version = ">=0.5, <0.7", optional = true }
@@ -106,7 +106,6 @@ use_wasm = ["icu_codepointtrie_builder/wasm"]
 # rule based segmenter data will not be generated.
 use_icu4c = ["icu_codepointtrie_builder/icu4c"]
 networking = ["dep:cached-path"]
-fs = ["dep:icu_provider_fs"]
 
 [[bin]]
 name = "icu4x-datagen"

provider/datagen/src/bin/datagen.rs

@@ -13,7 +13,6 @@ mod cli {
 
     #[derive(Copy, Clone, PartialEq, Eq, PartialOrd, Ord, ValueEnum, Debug)]
     pub(crate) enum Format {
-        #[cfg_attr(feature = "fs", value(hide = true))]
         Dir,
         Blob,
         Mod,
@@ -85,7 +84,6 @@ struct Cli {
 
     #[arg(short, long, value_enum, default_value_t = cli::Syntax::Json)]
     #[arg(help = "--format=dir only: serde serialization format.")]
-    #[cfg_attr(feature = "fs", arg(hide = true))]
     syntax: cli::Syntax,
 
     #[arg(short, long)]
@@ -335,9 +333,6 @@ fn main() -> eyre::Result<()> {
             if v == cli::Format::DeprecatedDefault {
                 log::warn!("Defaulting to --format=dir. This will become a required parameter in the future.");
             }
-            #[cfg(not(feature = "fs"))]
-            eyre::bail!("--format=dir only available with the `fs` Cargo feature");
-            #[cfg(feature = "fs")]
             icu_datagen::Out::Fs {
                 output_path: matches
                     .output

provider/datagen/src/lib.rs

@@ -99,13 +99,10 @@ impl CldrLocaleSubset {
 /// [Out::Fs] serialization formats.
 pub mod syntax {
     #[doc(no_inline)]
-    #[cfg(feature = "fs")]
     pub use icu_provider_fs::export::serializers::bincode::Serializer as Bincode;
     #[doc(no_inline)]
-    #[cfg(feature = "fs")]
     pub use icu_provider_fs::export::serializers::json::Serializer as Json;
     #[doc(no_inline)]
-    #[cfg(feature = "fs")]
     pub use icu_provider_fs::export::serializers::postcard::Serializer as Postcard;
 }
 
@@ -125,7 +122,6 @@ use icu_provider::datagen::*;
 use icu_provider::prelude::*;
 use icu_provider_adapters::empty::EmptyDataProvider;
 use icu_provider_adapters::filter::Filterable;
-#[cfg(feature = "fs")]
 use icu_provider_fs::export::serializers::AbstractSerializer;
 use prelude::*;
 use rayon::prelude::*;
@@ -319,9 +315,6 @@ impl Default for BakedOptions {
 #[non_exhaustive]
 pub enum Out {
     /// Output to a file system tree
-    ///
-    /// **Enabled with the `fs` Cargo feature.**
-    #[cfg(feature = "fs")]
     Fs {
         /// The root path.
         output_path: PathBuf,
@@ -355,7 +348,6 @@ pub enum Out {
 impl core::fmt::Debug for Out {
     fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
         match self {
-            #[cfg(feature = "fs")]
             Self::Fs {
                 output_path,
                 serializer,
@@ -415,7 +407,6 @@ pub fn datagen(
         .into_iter()
         .map(|out| -> Result<Box<dyn DataExporter>, DataError> {
             Ok(match out {
-                #[cfg(feature = "fs")]
                 Out::Fs {
                     output_path,
                     serializer,

provider/datagen/src/source.rs

@@ -329,9 +329,7 @@ impl SerdeCache {
         for<'de> S: serde::Deserialize<'de> + 'static + Send + Sync,
     {
         self.read_and_parse(path, |bytes| {
-            serde_json::from_slice(bytes)
-                .map_err(std::io::Error::from)
-                .map_err(DataError::from)
+            serde_json::from_slice(bytes).map_err(DataError::from)
         })
     }
 

tools/testdata-scripts/Cargo.toml

@@ -11,7 +11,7 @@ edition = "2021"
 [dependencies]
 crlify = { path = "../../utils/crlify" }
 databake = { path = "../../utils/databake" }
-icu_datagen = { path = "../../provider/datagen", default-features = false, features = ["fs"] }
+icu_datagen = { path = "../../provider/datagen", default-features = false }
 icu_locid = { path = "../../components/locid", features = ["databake"] }
 icu_provider = { path = "../../provider/core" }
 repodata = { path = "../../provider/repodata" }