-
Notifications
You must be signed in to change notification settings - Fork 31
Conversation
@jewelnewman's shell history from running a manual install:
|
…develop-fix-debian9-setup
1). Update debia:9 installation. Run installation and pass validation in temporary Dockerfile under Mac OSX. Since current Windows wsl/debian only support debian:11. Therefore, It didn't run installation and validation under Windows10 wsl/debian:9. 2). Fix #1087 missing default timezone issue
Set default timezone to Los_Angeles.
|
You should be able to determine the local timezone with the python geocoder and timezonefinder, e.g.,
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The call to https://ipapi.co/timezone is potentially rate limited. We should catch this error and fall back on another method if it occurs.
….., enforcing user to set local time zone
In windows WSL/Debain:11, the default timezone is Etc/GMT+X
|
If http code is 200, set the timezone from the response of URL. Otherwise , set the default timezone as UTC/GMT
|
This PR fixes the Debian 9 installer needed for WSL support.
Current issues
Code changes
build-aux/setup-linux-debian9.sh
Documentation changes
None
Test and Validation Notes
None