Rollup of 4 pull requests - #161999
Conversation
… on different platforms + fixed BSD-based systems to use fchmodat with AT_SYMLINK_NOFOLLOW
…tored non-BSD-based/non-Linux platforms to use OpenOptions open + set_permissions, and refactored tests accordingly
…ns ENOTSUP (e.g. for Ubuntu 20.04 returns ENOTSUP on non-symlinks + symlinks when using fchmodat with AT_SYMLINK_NOFOLLOW). Update docs accordingly as well and corrected behavior + docs for other Unix platforms with symlinks should return `FilesystemLoop` error instead of `InvalidInput` due to not setting `OpenOptions` with read enabled. Co-authored-by: Rachel Barker <rachel.barker@ferrous-systems.com>
…d fchmodat platform call on fchmodat and every platform falls back to open + fchmod when _res is set to ErrorKind::Unsupported; updated docs to reflect change
… r=clarfonthey Refactored docs for `std::fs::set_permissions_nofollow` + fix BSD-based systems to use fchmodat with AT_SYMLINK_NOFOLLOW flag Original Tracking Issue: rust-lang#141607 Fixes mistake made in rust-lang#158168 This PR refactors documentations for `std::fs::set_permissions_nofollow` and fixes BSD-based systems + Android to use `fchmodat` with `AT_SYMLINK_NOFOLLOW` flag (instead of no flag set) and refactors all other platforms to defer to `OpenOptions` with `O_NOFOLLOW` behavior. r? @clarfonthey Since they looked at the original `set_permissions_nofollow` PR I made cc @RalfJung
…ius-help-default, r=Kivooeo Report the configured Polonius default in -Z help Fixes rust-lang#161543
…nilla Add regression test for TAIT in extern fn ICE with the new solver Fixes rust-lang#156345
…=ShoyuVanilla add regression test for ICE unexpected region regression test for rust-lang#143896
|
@bors r+ p=5 |
This comment has been minimized.
This comment has been minimized.
What is this?This is an experimental post-merge analysis report that shows differences in test outcomes between the merged PR and its parent PR.Comparing d0ea9cd (parent) -> 3cabe36 (this PR) Test differencesShow 31 test diffsStage 1
Stage 2
Additionally, 23 doctest diffs were found. These are ignored, as they are noisy. Job group index
Test dashboardRun cargo run --manifest-path src/ci/citool/Cargo.toml -- \
test-dashboard 3cabe36ceb022e2f56d4d330b1e2886f31117f18 --output-dir test-dashboardAnd then open Job duration changes
How to interpret the job duration changes?Job durations can vary a lot, based on the actual runner instance |
|
Finished benchmarking commit (3cabe36): comparison URL. Overall result: no relevant changes - no action needed@rustbot label: -perf-regression Instruction countThis perf run didn't have relevant results for this metric. Max RSS (memory usage)Results (primary -3.0%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (secondary 2.1%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Binary sizeResults (secondary 0.0%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Bootstrap: 474.031s -> 475.389s (0.29%) |
|
📌 Perf builds for each rolled up PR:
parent commit: d0ea9cd000 In the case of a perf regression, run the following command for each PR you suspect might be the cause: |
Successful merges:
std::fs::set_permissions_nofollow+ fix BSD-based systems to use fchmodat with AT_SYMLINK_NOFOLLOW flag #160170 (Refactored docs forstd::fs::set_permissions_nofollow+ fix BSD-based systems to use fchmodat with AT_SYMLINK_NOFOLLOW flag)r? @ghost
Create a similar rollup