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

install-darwin: fix _nixbld uids for macOS sequoia (backport #10919) #11421

Merged
merged 3 commits into from
Sep 4, 2024

Conversation

mergify[bot]
Copy link

@mergify mergify bot commented Sep 3, 2024

Motivation

Starting in macOS 15 Sequoia, macOS daemon UIDs are encroaching on our default UIDs of 301-332. This commit relocates our range up to avoid clashing with the current UIDs of 301-304 and buy us a little time while still leaving headroom for people installing more than 32 users.

It also adopts GID 350 (same as first UID), since @emilazy pointed out that this will keep our build group from showing up in the Users & Groups interface. (See #10919 (comment))

Context

Priorities and Process

Add 👍 to pull requests you find important.

The Nix maintainer team uses a GitHub project board to schedule and track reviews.


This is an automatic backport of pull request #10919 done by [Mergify](https://mergify.com).

abathur and others added 3 commits September 3, 2024 23:59
Starting in macOS 15 Sequoia, macOS daemon UIDs are encroaching on our
default UIDs of 301-332. This commit relocates our range up to avoid
clashing with the current UIDs of 301-304 and buy us a little time
while still leaving headroom for people installing more than 32 users.

(cherry picked from commit df36ff0)
@mergify mergify bot requested a review from edolstra as a code owner September 3, 2024 23:59
@mergify mergify bot added the merge-queue label Sep 3, 2024
@edolstra edolstra merged commit b53b07b into 2.24-maintenance Sep 4, 2024
25 checks passed
@edolstra edolstra deleted the mergify/bp/2.24-maintenance/pr-10919 branch September 4, 2024 16:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants