Skip to content

rust: fix remote filesystem issue#24146

Merged
feckert merged 1 commit into
openwrt:masterfrom
TDT-AG:pr/20240513-rust
May 16, 2024
Merged

rust: fix remote filesystem issue#24146
feckert merged 1 commit into
openwrt:masterfrom
TDT-AG:pr/20240513-rust

Conversation

@feckert

@feckert feckert commented May 13, 2024

Copy link
Copy Markdown
Member

Maintainer: @1715173329 @lu-zero
Compile tested: x86_64, -, OpenWrt latest
Run tested: x86_64, -, OpenWrt latest, tests done

Description:
If the download directory is on another filesystem (NFS), then the current implementation of bootstrapping rust fails. Because the syscall (rename) does not work on crossing filesystem boundary.

This patch has already been merged upstream.
rust-lang/rust#124975

@lu-zero

lu-zero commented May 13, 2024

Copy link
Copy Markdown
Contributor

Thank you :)

@lu-zero

lu-zero commented May 13, 2024

Copy link
Copy Markdown
Contributor

Just update the version that got reviewed since between one and another review an error slipped in.

@lu-zero

lu-zero commented May 15, 2024

Copy link
Copy Markdown
Contributor

Please retake the commit now that landed.

Thank you

@feckert feckert force-pushed the pr/20240513-rust branch from 2b2e9b0 to 23aa958 Compare May 16, 2024 06:54
@feckert

feckert commented May 16, 2024

Copy link
Copy Markdown
Member Author

@lu-zero thanks for your work. If CI/CD is happy I will merge this.

If the download directory is on another filesystem (NFS), then the
current implementation of bootstrapping rust fails. Because the 'syscall'
(rename) does not work on crossing filesystem boundary.

This chnage was already merged upstream to the github main rust repository.
rust-lang/rust#124975

The patch has been rebased so that it can be applied correctly.
No functional change.

Signed-off-by: Florian Eckert <fe@dev.tdt.de>
@feckert feckert force-pushed the pr/20240513-rust branch from 23aa958 to 6b6c74d Compare May 16, 2024 11:20
@feckert feckert merged commit 4fac186 into openwrt:master May 16, 2024
@feckert feckert deleted the pr/20240513-rust branch May 16, 2024 13:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants