Skip to content

[fuchsia] Update ICU timezone data directory path to /config/tzdata/icu/44/le - #194241

Draft
zijiehe-google-com wants to merge 1 commit into
flutter:masterfrom
zijiehe-google-com:fuchsia-tzdata-path
Draft

zijiehe-google-com wants to merge 1 commit into
flutter:masterfrom
zijiehe-google-com:fuchsia-tzdata-path

Conversation

@zijiehe-google-com

@zijiehe-google-com zijiehe-google-com commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor

On Fuchsia, the ICU timezone data capability tzdata-icu was factored out of config-data and mounted at /config/tzdata/icu (see meta/common.shard.cml and flutter-team-archive/engine#48907).

However, runner.cc still pointed to the legacy /config/data/tzdata/icu/44/le path under config-data. On modern Fuchsia builds where config-data no longer contains timezone files, flutter_runner fails to open the directory (Could not open: '/config/data/tzdata/icu/44/le') and falls back to bundled timezone data in icudtl.dat, ignoring the system timezone database.

This change updates kICUTZDataDir to /config/tzdata/icu/44/le to match common.shard.cml and the Dart VM's Fuchsia OS layer (runtime/vm/os_fuchsia.cc).

See:

Pre-launch Checklist

If you need help, consider asking for advice on the #hackers-new channel on Discord.

If this change needs to override an active code freeze, provide a comment explaining why. The code freeze workflow can be overridden by code reviewers. See pinned issues for any active code freezes with guidance.

Note: The Flutter team is currently trialing the use of Gemini Code Assist for GitHub. Comments from the gemini-code-assist bot should not be taken as authoritative feedback from the Flutter team. If you find its comments useful you can update your code accordingly, but if you are unsure or disagree with the feedback, please feel free to wait for a Flutter team member's review for guidance on which automated comments should be addressed.

@github-actions github-actions Bot added engine flutter/engine related. See also e: labels. platform-fuchsia Fuchsia code specifically labels Oct 12, 2026
…cu/44/le

Updates kICUTZDataDir from /config/data/tzdata/icu/44/le to /config/tzdata/icu/44/le
to match the tzdata-icu directory capability mounted in common.shard.cml and
the Dart VM's Fuchsia OS layer.
@Hzj-jie
Hzj-jie force-pushed the fuchsia-tzdata-path branch from 6f9bb74 to b33725d Compare October 12, 2026 19:48
@zijiehe-google-com zijiehe-google-com changed the title Fuchsia tzdata path [fuchsia] Update ICU timezone data directory path to /config/tzdata/icu/44/le Oct 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

engine flutter/engine related. See also e: labels. platform-fuchsia Fuchsia code specifically

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant