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

Contra documentation, now() requires std #535

Closed
dfoxfranke opened this issue Feb 9, 2021 · 2 comments
Closed

Contra documentation, now() requires std #535

dfoxfranke opened this issue Feb 9, 2021 · 2 comments

Comments

@dfoxfranke
Copy link

The chrono documentation describes the clock feature as follows:

clock: enables reading the system time (now), independent of whether std::time::SystemTime is present, depends on having a libc.

But this is not the case: clock pulls in std, and the implementation of Utc::now() is just a wrapper around std::time::SystemTime::now().

@pitdicker
Copy link
Collaborator

The current readme describes the clock feature as:

  • clock: Enables reading the system time (now) and local timezone (Local).

And the main documentation in lib.rs:

  • clock: Enables reading the system time (now) that depends on the standard library for UNIX-like operating systems and the Windows API (winapi) for Windows.

The documentation in lib.rs still needs some updating, but at least we no longer it is "independent of whether std::time::SystemTime is present".

@pitdicker
Copy link
Collaborator

The documentation in lib.rs is updated in #1260.

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

No branches or pull requests

2 participants