Skip to content
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

fixed a time zone bug of BEFORE, AFTER value #49

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fixed a time zone bug of BEFORE, AFTER value #49

wants to merge 1 commit into from

Conversation

jihoonparkk
Copy link

Please review and merge it.

@pferrel
Copy link
Collaborator

pferrel commented Mar 22, 2018

what versions of macOS has this been tested on?

@jihoonparkk
Copy link
Author

macOS High Sierra (version 10.13.3)

@pferrel
Copy link
Collaborator

pferrel commented Mar 23, 2018

Odd, I get no errors on macOS High Sierra. Does this have to do with the your timezone vs mine?

@jihoonparkk
Copy link
Author

jihoonparkk commented Mar 25, 2018

My timezone. I'm in South Korea using +09:00 UTC offset.

For example, my local time is assumed to be "2018-03-26 01:00:00."

  • Previous Code
    date -v +0d +"%Y-%m-%dT%H:%M:%SZ"
    => The result is "2018-03-26T01:00:00Z" in ISO 8601 format. Same time is "2018-03-25T16:00:00+09:00." It's different to my local time.

  • Fixed Code
    date -v +0d -u +%FT%TZ
    => The result is "2018-03-25T16:00:00Z" in ISO 8601 format. Same time is "2018-03-26T01:00:00+09:00." It's same to my local time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants