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.

Sign up for GitHub

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

Jump to bottom

remove chrono dependency? #34

Closed
Roguelazer opened this issue Jun 6, 2022 · 2 comments · Fixed by #35
Closed

remove chrono dependency? #34

Roguelazer opened this issue Jun 6, 2022 · 2 comments · Fixed by #35

Comments

@Roguelazer
Copy link
Contributor

Chrono is caught up in the "reading env vars from C code is unsafe" mess and thus anything that uses dbase-rs will have issues from cargo-audit. What do you think about directly replacing it with time v0.3?
@Roguelazer Roguelazer mentioned this issue Jun 6, 2022
Merged
@tmontaigu
Copy link
Owner

Chrono is caught up in the "reading env vars from C code is unsafe" mess

Do you have links that explain this issue ? Ì'd like to learn about it

@Roguelazer
Copy link
Contributor Author

Roguelazer commented Jun 6, 2022 •
edited

time-rs/time#293 and chronotope/chrono#499

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

replace chrono with time; fix some things clippy was upset about EasyPost/dbase-rs
2 participants
@Roguelazer @tmontaigu