Drop time dependency from bootstrap#93685
Conversation
This comment has been minimized.
This comment has been minimized.
027356e to
2f23624
Compare
|
Sounds good. @bors r+ |
|
📌 Commit 2f23624 has been approved by |
|
@bors rollup |
…albini Drop time dependency from bootstrap 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 rust-lang#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`
|
Looks like #92480 (comment) hitted here too |
|
While I support this change (I didn't even know |
…albini Drop time dependency from bootstrap 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 rust-lang#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`
|
Rollup #93736 failed on this PR with: Looks like the |
|
@bors r- |
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.
2f23624 to
2a8c750
Compare
|
@bors r+ rollup=iffy Should be fixed. |
|
📌 Commit 2a8c750 has been approved by |
|
☀️ Test successful - checks-actions |
|
Finished benchmarking commit (05d1652): comparison url. Summary: This benchmark run did not return any relevant results. If you disagree with this performance assessment, please file an issue in rust-lang/rustc-perf. @rustbot label: -perf-regression |
Fix manpage version replacement and use verbose version - Fix a bug where `t!(fs::copy(&page_src, &page_dst))` was called after`fs::write`, silently overwriting the substituted content with the original template (restoring the `<INSERT VERSION HERE>` placeholder verbatim) (Related rust-lang#93685) - Use `rust_info().version()` instead of the bare version number, so the man page now includes the full version string with commit hash and date
Fix manpage version replacement and use verbose version - Fix a bug where `t!(fs::copy(&page_src, &page_dst))` was called after`fs::write`, silently overwriting the substituted content with the original template (restoring the `<INSERT VERSION HERE>` placeholder verbatim) (Related rust-lang#93685) - Use `rust_info().version()` instead of the bare version number, so the man page now includes the full version string with commit hash and date
Rollup merge of #155194 - davidkna:man-version, r=clubby789 Fix manpage version replacement and use verbose version - Fix a bug where `t!(fs::copy(&page_src, &page_dst))` was called after`fs::write`, silently overwriting the substituted content with the original template (restoring the `<INSERT VERSION HERE>` placeholder verbatim) (Related #93685) - Use `rust_info().version()` instead of the bare version number, so the man page now includes the full version string with commit hash and date
Fix manpage version replacement and use verbose version - Fix a bug where `t!(fs::copy(&page_src, &page_dst))` was called after`fs::write`, silently overwriting the substituted content with the original template (restoring the `<INSERT VERSION HERE>` placeholder verbatim) (Related rust-lang/rust#93685) - Use `rust_info().version()` instead of the bare version number, so the man page now includes the full version string with commit hash and date
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