Skip to content

use confstr(_CS_DARWIN_USER_TEMP_DIR, ...) as a TMPDIR fallback on Darwin#100824

Closed
thomcc wants to merge 2 commits intorust-lang:masterfrom
thomcc:darwin_user_temp_dir
Closed

use confstr(_CS_DARWIN_USER_TEMP_DIR, ...) as a TMPDIR fallback on Darwin#100824
thomcc wants to merge 2 commits intorust-lang:masterfrom
thomcc:darwin_user_temp_dir

Conversation

@thomcc
Copy link
Copy Markdown
Member

@thomcc thomcc commented Aug 20, 2022

Followup to #100196 but should wait on rust-lang/libc#2883 before anything is done with it. (Edit: Now it needs rust-lang/libc#2931... sigh)

See #100824 (comment) for an explanation of the change this makes and why I think this is worth doing.

Closes #99608.

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

disposition-merge This issue / PR is in PFCP or FCP with a disposition to merge it. finished-final-comment-period The final comment period is finished for this PR / Issue. needs-fcp This change is insta-stable, or significant enough to need a team FCP to proceed. O-apple Operating system: Apple / Darwin (macOS, iOS, tvOS, visionOS, watchOS) S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. T-libs-api Relevant to the library API team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Consider returning /private/tmp instead of /tmp on macOS with std::env::temp_dir()