Skip to content

Permissions Fix#1400

Merged
catilac merged 1 commit intoprocessing:mainfrom
Stefterv:nix-pm-fixes
Feb 6, 2026
Merged

Permissions Fix#1400
catilac merged 1 commit intoprocessing:mainfrom
Stefterv:nix-pm-fixes

Conversation

@Stefterv
Copy link
Contributor

@Stefterv Stefterv commented Feb 2, 2026

Set dir permission to fix an issue for the nix packagemanager

@catilac
Copy link
Collaborator

catilac commented Feb 4, 2026

omg throw back to before i was starting haha :)
do we need to worry about the failing checks regarding build names?

@Stefterv
Copy link
Contributor Author

Stefterv commented Feb 5, 2026

#1401 this fixes the same issue, I took the moment to expand functionality too

@catilac
Copy link
Collaborator

catilac commented Feb 5, 2026

Closing as this is already addressed

@catilac catilac closed this Feb 5, 2026
@Stefterv
Copy link
Contributor Author

Stefterv commented Feb 5, 2026

🥲
The issue is addressed but this PR is still relevant

@catilac catilac reopened this Feb 5, 2026
@catilac
Copy link
Collaborator

catilac commented Feb 5, 2026

oops!! i meant to close MINE. too many tabs

@Stefterv Stefterv force-pushed the nix-pm-fixes branch 2 times, most recently from 574ff88 to bfa968c Compare February 5, 2026 14:20
Set dir permission to fix an issue for the nix packagemanager
@catilac catilac merged commit 05bc755 into processing:main Feb 6, 2026
6 checks passed
neobrain added a commit to neobrain/processing4 that referenced this pull request Mar 5, 2026
This is required for building against read-only JDK installations (notably
used when building with Nix). processing#1400 already attempted to integrate my nixpkgs
patch for this purpose, but it accidentally placed the dirPermissions line
under a different task.
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