New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
SNTP improvements #13
Conversation
rfocosi
commented
Sep 15, 2019
- Added retry get time server information, to fix issue on SM-X boot;
- Timezone is a parameter now (/z);
- Change on command parameters when a TIMESERVER environment variable is set. Dot(.) is not necessary anymore;
Hi @rfocosi, sorry I hadn't got (or I had missed) notifications for this pull request. I'll take a look. |
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.
Looks good, I've just requested a small change in the presentation string.
Also, I did a lot of other changes and bugfix in another repository and I will send you another PR after approving this one :) |
This change will be added as part of a separate .gitignore file.
Thanks for your work. I have updated the SNTP tool in my web site with the new v1.2. |