|
| 1 | +# Libs Meeting 2021-02-10/11 |
| 2 | + |
| 3 | +###### tags: `Libs Meetings` `Minutes` |
| 4 | + |
| 5 | +**Attendees**: KodrAus, Amanieu, dtolnay, Mara, joshtriplett |
| 6 | + |
| 7 | +## Agenda |
| 8 | + |
| 9 | +- [Open action items](https://hackmd.io/ovrbJj6CRduRgSA0Wzg2zg) |
| 10 | +- Edition 2021 |
| 11 | + - Range types |
| 12 | + - Prelude |
| 13 | + - Ask Dirkjan if he's still interested. |
| 14 | +- Triage |
| 15 | +- Anything else? |
| 16 | + |
| 17 | +## Triage |
| 18 | + |
| 19 | +### FCPs |
| 20 | + |
| 21 | +6 open T-libs FCPs: |
| 22 | +<details><summary><a href="https://github.com/rust-lang/rfcs/issues?q=is%3Aopen+label%3AT-libs+label%3Aproposed-final-comment-period">2 <code>rust-lang/rfcs</code> FCPs</a></summary> |
| 23 | + |
| 24 | + - [[close 2944](https://github.com/rust-lang/rfcs/issues/2944#issuecomment-714795934)] *RFC: add the Freeze trait to libcore/libstd* - (2 checkboxes left) |
| 25 | + - [[close 2708](https://github.com/rust-lang/rfcs/issues/2708#issuecomment-724941427)] *Generic Pointer to Field* - (2 checkboxes left) |
| 26 | +</details> |
| 27 | +<details><summary><a href="https://github.com/rust-lang/rust/issues?q=is%3Aopen+label%3AT-libs+label%3Aproposed-final-comment-period">4 <code>rust-lang/rust</code> FCPs</a></summary> |
| 28 | + |
| 29 | + - [[merge 81050](https://github.com/rust-lang/rust/issues/81050#issuecomment-765341798)] *Stabilize core::task::ready!* - (3 checkboxes left) |
| 30 | + - [[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) |
| 31 | + - [[merge 70516](https://github.com/rust-lang/rust/issues/70516#issuecomment-771646381)] *Tracking Issue for ASCII methods on OsStr* - (3 checkboxes left) |
| 32 | + - [[merge 74773](https://github.com/rust-lang/rust/issues/74773#issuecomment-776643142)] *Tracking Issue for str\_split\_once* - (5 checkboxes left) |
| 33 | +</details> |
| 34 | +<p></p> |
| 35 | + |
| 36 | +[Amanieu (2)](https://rfcbot.rs/fcp/Amanieu), [BurntSushi (6)](https://rfcbot.rs/fcp/BurntSushi), [KodrAus (2)](https://rfcbot.rs/fcp/KodrAus), [dtolnay (1)](https://rfcbot.rs/fcp/dtolnay), [m-ou-se (1)](https://rfcbot.rs/fcp/m-ou-se), [sfackler (2)](https://rfcbot.rs/fcp/sfackler), [withoutboats (4)](https://rfcbot.rs/fcp/withoutboats) |
| 37 | + |
| 38 | +### Nominated |
| 39 | + |
| 40 | +- [2 `rust-lang/rfcs` items](https://github.com/rust-lang/rfcs/issues?q=is%3Aopen+label%3AT-libs+label%3AI-nominated) |
| 41 | + - [[2979](https://github.com/rust-lang/rfcs/pull/2979)] *\[RFC\]: Libs Team Governance* |
| 42 | + - [[3058](https://github.com/rust-lang/rfcs/pull/3058)] *try\_trait\_v2: A new design for the ? desugaring* |
| 43 | + - Discuss next week, when scott joins us. |
| 44 | + - Read before the meeting. |
| 45 | +- [0 `rust-lang/rust` items](https://github.com/rust-lang/rust/issues?q=is%3Aopen+label%3AT-libs+label%3AI-nominated) |
| 46 | + |
| 47 | +### Waiting on team |
| 48 | + |
| 49 | +- [0 `rust-lang/rfcs` items](https://github.com/rust-lang/rfcs/issues?q=is%3Aopen+label%3AT-libs+label%3AS-waiting-on-team) |
| 50 | +- [5 `rust-lang/rust` items](https://github.com/rust-lang/rust/issues?q=is%3Aopen+label%3AT-libs+label%3AS-waiting-on-team) |
| 51 | + - [[71780](https://github.com/rust-lang/rust/pull/71780)] *Implement String::remove\_matches* |
| 52 | + - Unstable. Libs-impl? |
| 53 | + - [[76901](https://github.com/rust-lang/rust/pull/76901)] *Implement RFC 2500 Needle API (Part 1)* |
| 54 | + - Needs a reviewer. We don't have the reviewer bandwidth. :( |
| 55 | + - KodrAus to take a look: both burnsushi and withoutboats both suggested the current API might need a bit of a rethink. |
| 56 | + - [[77326](https://github.com/rust-lang/rust/pull/77326)] *Stabilize \`Option::unwrap\_none\` and \`Option::expect\_none\`* |
| 57 | + - assert on is_none() good enough? |
| 58 | + - could still be useful for nicer syntax, to have it as a suffix. |
| 59 | + - fcp close |
| 60 | + - [[77728](https://github.com/rust-lang/rust/pull/77728)] *Expose force\_quotes,* |
| 61 | + - Seal `CommandExt` after crater run. <- Amanieu |
| 62 | + - RFC for portability lint could help for similar situations, to not use extension traits for platform-specific stuff anymore. |
| 63 | + - dtolnay to write the idea down |
| 64 | + - [[80572](https://github.com/rust-lang/rust/pull/80572)] *Add a \`Result::ok\_or\_err\` method to extract a \`T\` from \`Result\<T, T\>\`* |
| 65 | + - Yes, we should have this method. |
| 66 | + - Names are hard. Proposed one seems okay. |
| 67 | + - Alternatively, `into_inner()`? |
| 68 | + - Merge as is, unstably. Bikeshed more later. :) |
| 69 | + |
| 70 | +### Needs decision |
| 71 | + |
| 72 | +- [14 `rust-lang/rust` items](https://github.com/rust-lang/rust/issues?q=is%3Aopen+label%3AT-libs+label%3AI-needs-decision) |
| 73 | + - [[21319](https://github.com/rust-lang/rust/issues/21319)] *Consider making std::thread::Builder reusable* |
| 74 | + - KodrAus to write reply |
| 75 | + - [[25053](https://github.com/rust-lang/rust/issues/25053)] *UnsafeCell should implement the Copy trait* |
| 76 | + - Language issue? |
| 77 | + - [[26951](https://github.com/rust-lang/rust/issues/26951)] *Abort on some large allocation requests, Panic on other* |
| 78 | + - [[29494](https://github.com/rust-lang/rust/issues/29494)] *Command does not escape arguments as expected on windows* |
| 79 | + - [[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* |
| 80 | + - [[39081](https://github.com/rust-lang/rust/issues/39081)] *\`std::env::temp\_dir\` should return env var XDG\_RUNTIME\_DIR under Linux if declared* |
| 81 | + - fcp close? |
| 82 | + - [[39186](https://github.com/rust-lang/rust/issues/39186)] *Document that std::os::unix::process::CommandExt.uid() triggers setgroups(0,0)* |
| 83 | + - Mara to check this. |
| 84 | + - [[42412](https://github.com/rust-lang/rust/issues/42412)] *Incorporate @raphlinus' container cheat sheet into the std::collections module docs* |
| 85 | + - [[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* |
| 86 | + - [[59878](https://github.com/rust-lang/rust/issues/59878)] *Box\<\[T\]\> should have an IntoIter implementation.* |
| 87 | + - [[60880](https://github.com/rust-lang/rust/issues/60880)] *Should Thumb bugs get increased visibility, e.g. Tier 1 status?* |
| 88 | + - [[62726](https://github.com/rust-lang/rust/issues/62726)] *Tracking issue for io\_slice\_advance* |
| 89 | + - [[79490](https://github.com/rust-lang/rust/issues/79490)] *Whether floating point value Display should adhere to IEEE standard?* |
| 90 | + - [[80823](https://github.com/rust-lang/rust/pull/80823)] *Add Cow::disown\_if() and Cow::disown\_if\_eq()* |
| 91 | + |
| 92 | +## Actions |
| 93 | + |
| 94 | +- [ ] Reply to all issues/PRs discussed in this meeting, or add them to the [open action items](https://hackmd.io/ovrbJj6CRduRgSA0Wzg2zg). |
0 commit comments