Consider implementing tzset workaround for Android < O |
||||
Issue descriptionSee https://android-review.googlesource.com/#/c/platform/bionic/+/321708/ where localtime_r is made to call tzset. See internal bug 62563020, where taruhi@ recommends adding this workaround to v8/src/base/platform/platform-posix-time.cc on Android versions lacking the patch.
,
Aug 23 2017
,
Aug 23 2017
V8 currently doesn't seem to call tzset on Linux at all. It might require updating the Sandbox to make such a call possible in Chrome [1]. Probably also a good idea to make sure to do this tzset call only on the affected Android platforms, as going through the sandbox has some cost. [1] https://cs.chromium.org/chromium/src/content/zygote/zygote_main_linux.cc
,
Aug 23
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue. Sorry for the inconvenience if the bug really should have been left as Available. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Sep 4
rmcilroy@ can you please evaluate?
,
Sep 6
This has been untouched for a year and is not on my radar. Is this still a priority? Technically it seems a bit complex due to sandbox concerns, so we would have to budget the work.
,
Sep 6
I don't think the priority has changed, although it was never high priority. It'll become obsolete in maybe 5 years when Android >= O reaches saturation :P |
||||
►
Sign in to add a comment |
||||
Comment 1 by jgruber@chromium.org
, Aug 23 2017Labels: -Pri-3 Pri-2
Status: Available (was: Untriaged)