Skip to content

Drop time dependency from bootstrap#93685

Merged
bors merged 1 commit intorust-lang:masterfrom
Mark-Simulacrum:drop-time
Feb 13, 2022
Merged

Drop time dependency from bootstrap#93685
bors merged 1 commit intorust-lang:masterfrom
Mark-Simulacrum:drop-time

Conversation

@Mark-Simulacrum
Copy link
Copy Markdown
Member

This was only used for the inclusion of 'current' dates into our manpages, but
it is not clear that this is practically necessary. The manpage is essentially
never updated, and so we can likely afford to keep a manual date in these files.
It also seems possible to just omit it, but that may cause other tools trouble,
so avoid doing that for now.

This is largely done to reduce bootstrap complexity; the time crate is not particularly
small and in #92480 would have started pulling in num-threads, which does runtime
thread count detection. I would prefer to avoid that, so filing this to just drop the nearly
unused dependency entirely.

r? @pietroalbini

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merged-by-bors This PR was explicitly merged by bors. S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.

Projects

None yet

Development

Successfully merging this pull request may close these issues.