Add two different HostSystem impls

Author: nekevss

Cargo.toml

@@ -95,10 +95,11 @@ zoneinfo64 = { workspace = true }
 resb = "0.1.0"
 
 [features]
-default = ["sys"]
+default = ["sys-local"]
 log = ["dep:log"]
 compiled_data = ["tzdb"]
-sys = ["std", "compiled_data", "dep:web-time", "dep:iana-time-zone"]
+sys = ["std", "compiled_data", "dep:web-time"]
+sys-local = ["sys", "dep:iana-time-zone"]
 tzdb = [
     "std",
     "timezone_provider/tzif",

src/builtins/core/now.rs

@@ -196,7 +196,7 @@ mod tests {
         assert_eq!(duration.milliseconds(), 0);
     }
 
-    #[cfg(all(feature = "tzdb", feature = "sys", feature = "compiled_data"))]
+    #[cfg(all(feature = "tzdb", feature = "sys-local", feature = "compiled_data"))]
     #[test]
     fn now_datetime_test() {
         use crate::Temporal;
@@ -205,9 +205,9 @@ mod tests {
 
         let sleep = 2;
 
-        let before = Temporal::now().plain_date_time_iso(None).unwrap();
+        let before = Temporal::local_now().plain_date_time_iso(None).unwrap();
         thread::sleep(StdDuration::from_secs(sleep));
-        let after = Temporal::now().plain_date_time_iso(None).unwrap();
+        let after = Temporal::local_now().plain_date_time_iso(None).unwrap();
 
         let diff = after.since(&before, DifferenceSettings::default()).unwrap();
 

src/sys.rs

@@ -25,32 +25,51 @@ use web_time::{SystemTime, UNIX_EPOCH};
 #[cfg(feature = "sys")]
 pub struct Temporal;
 
-#[cfg(feature = "sys")]
 impl Temporal {
     /// Get a `Now` object for the default host system.
-    pub fn now() -> Now<DefaultHostSystem> {
-        Now::new(DefaultHostSystem)
+    #[cfg(feature = "sys-local")]
+    #[deprecated(
+        since = "0.1.0",
+        note = "`now` deprecated was not clear about the host system implementation, please use `local_now`"
+    )]
+    pub fn now() -> Now<LocalHostSystem> {
+        Now::new(LocalHostSystem)
+    }
+
+    /// Get a `Now` object with a [`LocalHostSystem`], which
+    /// will use the host system's time zone as a fallback.
+    #[cfg(feature = "sys-local")]
+    pub fn local_now() -> Now<LocalHostSystem> {
+        Now::new(LocalHostSystem)
+    }
+
+    /// Get a `Now` object with a [`UtcHostSystem`], which
+    /// will use a UTC time zone as a fallback.
+    #[cfg(feature = "sys")]
+    pub fn utc_now() -> Now<UtcHostSystem> {
+        Now::new(UtcHostSystem)
     }
 }
 
-/// A default host system implementation
+/// A UTC host system implementation that will return the current time
+/// with the a UTC time zone as fallback.
 ///
-/// This implementation is backed by [`SystemTime`] and [`iana_time_zone`]
+/// This implementation is backed by [`std::time::SystemTime`].
 #[cfg(feature = "sys")]
-pub struct DefaultHostSystem;
+pub struct UtcHostSystem;
 
 #[cfg(feature = "sys")]
-impl HostHooks for DefaultHostSystem {}
+impl HostHooks for UtcHostSystem {}
 
 #[cfg(feature = "sys")]
-impl HostClock for DefaultHostSystem {
+impl HostClock for UtcHostSystem {
     fn get_host_epoch_nanoseconds(&self) -> TemporalResult<EpochNanoseconds> {
         get_system_nanoseconds()
     }
 }
 
 #[cfg(feature = "sys")]
-impl HostTimeZone for DefaultHostSystem {
+impl HostTimeZone for UtcHostSystem {
     fn get_host_time_zone(
         &self,
         provider: &impl timezone_provider::provider::TimeZoneProvider,
@@ -59,7 +78,34 @@ impl HostTimeZone for DefaultHostSystem {
     }
 }
 
-#[cfg(feature = "sys")]
+/// A local host system implementation that will return the current time
+/// with the system time zone as a fallback.
+///
+/// This implementation is backed by [`std::time::SystemTime`] and [`iana_time_zone`]
+#[cfg(feature = "sys-local")]
+pub struct LocalHostSystem;
+
+#[cfg(feature = "sys-local")]
+impl HostHooks for LocalHostSystem {}
+
+#[cfg(feature = "sys-local")]
+impl HostClock for LocalHostSystem {
+    fn get_host_epoch_nanoseconds(&self) -> TemporalResult<EpochNanoseconds> {
+        get_system_nanoseconds()
+    }
+}
+
+#[cfg(feature = "sys-local")]
+impl HostTimeZone for LocalHostSystem {
+    fn get_host_time_zone(
+        &self,
+        provider: &impl timezone_provider::provider::TimeZoneProvider,
+    ) -> TemporalResult<TimeZone> {
+        get_system_timezone(provider)
+    }
+}
+
+#[cfg(feature = "sys-local")]
 #[inline]
 pub(crate) fn get_system_timezone(provider: &impl TimeZoneProvider) -> TemporalResult<TimeZone> {
     iana_time_zone::get_timezone()