|
| 1 | +# Libs Meeting 2021-02-03 |
| 2 | + |
| 3 | +###### tags: `Libs Meetings` `Minutes` |
| 4 | + |
| 5 | +**Attendees**: Amanieu, sfackler, dtolnay, Yaahc, JoshTriplett, Mara |
| 6 | + |
| 7 | +## Agenda |
| 8 | + |
| 9 | +- [Open action items](https://hackmd.io/ovrbJj6CRduRgSA0Wzg2zg) |
| 10 | +- Triage |
| 11 | +- Anything else? |
| 12 | + |
| 13 | +## Triage |
| 14 | + |
| 15 | +### FCPs |
| 16 | + |
| 17 | +9 open T-libs FCPs: |
| 18 | +<details><summary><a href="https://github.com/rust-lang/rfcs/issues?q=is%3Aopen+label%3AT-libs+label%3Aproposed-final-comment-period">3 <code>rust-lang/rfcs</code> FCPs</a></summary> |
| 19 | + |
| 20 | + - [[close 2944](https://github.com/rust-lang/rfcs/issues/2944#issuecomment-714795934)] *RFC: add the Freeze trait to libcore/libstd* - (2 checkboxes left) |
| 21 | + - [[close 2708](https://github.com/rust-lang/rfcs/issues/2708#issuecomment-724941427)] *Generic Pointer to Field* - (2 checkboxes left) |
| 22 | + - [[merge 2996](https://github.com/rust-lang/rfcs/issues/2996#issuecomment-747664240)] *adds async stream rfc* - (3 checkboxes left) |
| 23 | +</details> |
| 24 | +<details><summary><a href="https://github.com/rust-lang/rust/issues?q=is%3Aopen+label%3AT-libs+label%3Aproposed-final-comment-period">6 <code>rust-lang/rust</code> FCPs</a></summary> |
| 25 | + |
| 26 | + - [[close 80595](https://github.com/rust-lang/rust/issues/80595#issuecomment-753374897)] *\`impl PartialEq<Punct> for char\`; symmetry for #78636* - (3 checkboxes left) |
| 27 | + - [[merge 78880](https://github.com/rust-lang/rust/issues/78880#issuecomment-759882257)] *Add \`NotSupported\` to \`std::io::ErrorKind\`* - (3 checkboxes left) |
| 28 | + - [[merge 81050](https://github.com/rust-lang/rust/issues/81050#issuecomment-765341798)] *Stabilize core::task::ready!* - (3 checkboxes left) |
| 29 | + - [[merge 77704](https://github.com/rust-lang/rust/issues/77704#issuecomment-760056634)] *Implement indexing slices with pairs of core::ops::Bound<usize>* - (3 checkboxes left) |
| 30 | + - [[close 75793](https://github.com/rust-lang/rust/issues/75793#issuecomment-769762122)] *Impl task::Wake for Fn* - (3 checkboxes left) |
| 31 | + - [[merge 70516](https://github.com/rust-lang/rust/issues/70516#issuecomment-771646381)] *Tracking Issue for ASCII methods on OsStr* - (4 checkboxes left) |
| 32 | +</details> |
| 33 | +<p></p> |
| 34 | + |
| 35 | +[Amanieu (1)](https://rfcbot.rs/fcp/Amanieu), [BurntSushi (8)](https://rfcbot.rs/fcp/BurntSushi), [KodrAus (1)](https://rfcbot.rs/fcp/KodrAus), [dtolnay (1)](https://rfcbot.rs/fcp/dtolnay), [m-ou-se (5)](https://rfcbot.rs/fcp/m-ou-se), [sfackler (2)](https://rfcbot.rs/fcp/sfackler), [withoutboats (8)](https://rfcbot.rs/fcp/withoutboats) |
| 36 | + |
| 37 | +### Nominated |
| 38 | + |
| 39 | +- [1 `rust-lang/rfcs` items](https://github.com/rust-lang/rfcs/issues?q=is%3Aopen+label%3AT-libs+label%3AI-nominated) |
| 40 | + - [[2979](https://github.com/rust-lang/rfcs/pull/2979)] *\[RFC\]: Libs Team Governance* |
| 41 | +- [7 `rust-lang/rust` items](https://github.com/rust-lang/rust/issues?q=is%3Aopen+label%3AT-libs+label%3AI-nominated) |
| 42 | + - [[66481](https://github.com/rust-lang/rust/issues/66481)] *Context and Waker might be accidentally \`Sync\`* |
| 43 | + - Moved to wg-async-foundations last meeting. |
| 44 | + - [[72981](https://github.com/rust-lang/rust/pull/72981)] *Stabilize the backtrace feature.* |
| 45 | + FCP finished. Should be merged? |
| 46 | + - Type in core, all methods as an extension trait in std, imported `as _` into prelude |
| 47 | + - Alternatively: wait on std+core unification? |
| 48 | + - Check if we can add `impl Type` to std for types of core. This might be okay for non-trait stuff. |
| 49 | + - [[74304](https://github.com/rust-lang/rust/pull/74304)] *Stabilize the Wake trait* |
| 50 | + FCP finished. Should be merged? |
| 51 | + - Looks ready to be merged. Fine without the blanket impl. |
| 52 | + - [[79245](https://github.com/rust-lang/rust/pull/79245)] *BTreeMap: remove Ord bound where it will never be needed* |
| 53 | + FCP finished. Should be merged? |
| 54 | + - Still moving. |
| 55 | + - usize::BITS causing breakage - https://github.com/rust-lang/rust/issues/76904 |
| 56 | + - Investigate: Stabilize for a future release? Warnings/errors about those should not be supressed by cap-lints. |
| 57 | + - Revert for now |
| 58 | + - lexical-core looks unmaintained. owner inactive on github, last commit long ago. |
| 59 | + - Maybe it can be fixed in `nom`, as the biggest consumer of this crate? |
| 60 | + - [[80962](https://github.com/rust-lang/rust/pull/80962)] *Stabilize remaining integer methods as \`const fn\`* |
| 61 | + - FCP already started. nothing to discuss. |
| 62 | + - [[81382](https://github.com/rust-lang/rust/pull/81382)] *Make \`is\_sorted\[\_by\_key\]\` require \`Ord\` instead of \`PartialOrd\` and remove \`Iterator::is\_sorted\_by\_key\`* |
| 63 | + - PartialOrd is fine, as that matches `<=`. |
| 64 | + - [[81460](https://github.com/rust-lang/rust/issues/81460)] *regression 1.51: lifetime may not live long enough with \`&impl Error\`* |
| 65 | + - This was expected. Wontfix. |
| 66 | + |
| 67 | +### Waiting on team |
| 68 | + |
| 69 | +- [0 `rust-lang/rfcs` items](https://github.com/rust-lang/rfcs/issues?q=is%3Aopen+label%3AT-libs+label%3AS-waiting-on-team) |
| 70 | +- [5 `rust-lang/rust` items](https://github.com/rust-lang/rust/issues?q=is%3Aopen+label%3AT-libs+label%3AS-waiting-on-team) |
| 71 | + - [[71780](https://github.com/rust-lang/rust/pull/71780)] *Implement String::remove\_matches* |
| 72 | + - [[76901](https://github.com/rust-lang/rust/pull/76901)] *Implement RFC 2500 Needle API (Part 1)* |
| 73 | + - [[77326](https://github.com/rust-lang/rust/pull/77326)] *Stabilize \`Option::unwrap\_none\` and \`Option::expect\_none\`* |
| 74 | + - [[77728](https://github.com/rust-lang/rust/pull/77728)] *Expose force\_quotes,* |
| 75 | + - [[80572](https://github.com/rust-lang/rust/pull/80572)] *Add a \`Result::ok\_or\_err\` method to extract a \`T\` from \`Result\<T, T\>\`* |
| 76 | + |
| 77 | +### Needs decision |
| 78 | + |
| 79 | +- [15 `rust-lang/rust` items](https://github.com/rust-lang/rust/issues?q=is%3Aopen+label%3AT-libs+label%3AI-needs-decision) |
| 80 | + - [[21319](https://github.com/rust-lang/rust/issues/21319)] *Consider making std::thread::Builder reusable* |
| 81 | + - [[25053](https://github.com/rust-lang/rust/issues/25053)] *UnsafeCell should implement the Copy trait* |
| 82 | + - [[26951](https://github.com/rust-lang/rust/issues/26951)] *Abort on some large allocation requests, Panic on other* |
| 83 | + - [[29494](https://github.com/rust-lang/rust/issues/29494)] *Command does not escape arguments as expected on windows* |
| 84 | + - [[37868](https://github.com/rust-lang/rust/issues/37868)] *std::process::Command's current\_dir behaves differently on Unix and Windows, with respect to relative exe paths* |
| 85 | + - [[39081](https://github.com/rust-lang/rust/issues/39081)] *\`std::env::temp\_dir\` should return env var XDG\_RUNTIME\_DIR under Linux if declared* |
| 86 | + - [[39186](https://github.com/rust-lang/rust/issues/39186)] *Document that std::os::unix::process::CommandExt.uid() triggers setgroups(0,0)* |
| 87 | + - [[42412](https://github.com/rust-lang/rust/issues/42412)] *Incorporate @raphlinus' container cheat sheet into the std::collections module docs* |
| 88 | + - [[42951](https://github.com/rust-lang/rust/issues/42951)] *Spec request. Hasher: is write\_u32 eqivalent to 4 calls of write\_u8?* |
| 89 | + - [[56889](https://github.com/rust-lang/rust/issues/56889)] *Write::write\_all erroring when encountering Ok(0) interacts poorly with the contract of Write::write* |
| 90 | + - [[59878](https://github.com/rust-lang/rust/issues/59878)] *Box\<\[T\]\> should have an IntoIter implementation.* |
| 91 | + - [[60880](https://github.com/rust-lang/rust/issues/60880)] *Should Thumb bugs get increased visibility, e.g. Tier 1 status?* |
| 92 | + - [[62726](https://github.com/rust-lang/rust/issues/62726)] *Tracking issue for io\_slice\_advance* |
| 93 | + - [[79490](https://github.com/rust-lang/rust/issues/79490)] *Whether floating point value Display should adhere to IEEE standard?* |
| 94 | + - [[80823](https://github.com/rust-lang/rust/pull/80823)] *Add Cow::disown\_if() and Cow::disown\_if\_eq()* |
| 95 | + |
| 96 | +## Actions |
| 97 | + |
| 98 | +- [x] Reply to all issues/PRs discussed in this meeting, or add them to the [open action items](https://hackmd.io/ovrbJj6CRduRgSA0Wzg2zg). |
0 commit comments