Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[beta] When uplifting directories, symlink them instead of hard-link them. #4739

Merged
merged 1 commit into from
Nov 22, 2017

Conversation

kennytm
Copy link
Member

@kennytm kennytm commented Nov 22, 2017

Backport of #4672 to 1.22 (cargo 0.23)

The current stable RC (cee38cd) contains #4616 but not #4672. Without the latter it is known to cause #4671 on APFS (enabled by default on macOS 10.13 "High Sierra").

The issue was found by kpy3 on https://internals.rust-lang.org/t/rust-1-22-0-prerelease-testing/6282/2.

@rust-highfive
Copy link

r? @alexcrichton

(rust_highfive has picked a reviewer for you, use r? to override)

@rust-highfive
Copy link

warning Warning warning

  • Pull requests are usually filed against the master branch for this repo, but this one is against rust-1.22.0. Please double check that you specified the right target!

@matklad
Copy link
Member

matklad commented Nov 22, 2017

Ok, let's get this going!

@bors r+

After this is merged, you'll need to send a PR against rust-lang/rust, as desribed here: https://github.com/rust-lang-nursery/rust-forge/blob/master/beta-backporting.md#backporting-in-rust-langcargo

@bors
Copy link
Collaborator

bors commented Nov 22, 2017

📌 Commit 2c6a0fe has been approved by matklad

@bors
Copy link
Collaborator

bors commented Nov 22, 2017

⌛ Testing commit 2c6a0fe with merge 61fa024...

bors added a commit that referenced this pull request Nov 22, 2017
[beta] When uplifting directories, symlink them instead of hard-link them.

Backport of #4672 to 1.22 (cargo 0.23)

The current stable RC (cee38cd) contains #4616 *but not* #4672. Without the latter it is known to cause #4671 on APFS (enabled by default on macOS 10.13 "High Sierra").

The issue was found by kpy3 on https://internals.rust-lang.org/t/rust-1-22-0-prerelease-testing/6282/2.
@bors
Copy link
Collaborator

bors commented Nov 22, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: matklad
Pushing 61fa024 to rust-1.22.0...

@bors bors merged commit 2c6a0fe into rust-lang:rust-1.22.0 Nov 22, 2017
@kennytm kennytm deleted the backport-4672 branch November 22, 2017 15:12
kennytm added a commit to kennytm/rust that referenced this pull request Nov 22, 2017
This backport includes rust-lang/cargo#4739, required for building projects
in debug mode with cargo on macOS High Sierra.
bors added a commit to rust-lang/rust that referenced this pull request Nov 22, 2017
[stable] Update cargo to fix breakage on macOS High Sierra; Update version to 1.22.1.

This backport includes rust-lang/cargo#4739, required for building projects in debug mode with Cargo on macOS High Sierra (see the Cargo PR for details).
bors added a commit to rust-lang/rust that referenced this pull request Nov 22, 2017
[stable] Update cargo to fix breakage on macOS High Sierra; Update version to 1.22.1.

This backport includes rust-lang/cargo#4739, required for building projects in debug mode with Cargo on macOS High Sierra (see the Cargo PR for details).
@ehuss ehuss added this to the 1.22.0 milestone Feb 6, 2022
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.

None yet

6 participants