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
Remove useless dont_clobber_environment.patch for Godot #66349
Conversation
Reasons for: * It will break and prevent building * If this patch is really necessary it should be done upstream
The patch exists to unbreak building. This effectively prevents Godot from adding a bunch of global paths to the environment, which breaks non-sandboxed builds. Not in disagreement that it should be upstreamed, though! |
How does it do it though? I'm no python expert but from what I can tell the patch effectively does nothing, it just changes the way environment variables are passed through. The patch adds env vars in a loop when before it does it sequentially. |
The looping thing is a trivial refactor… the important part is that the patch assigns where the original appends. I'm curious as to why you think it breaks the build. This package has of course passed tests! |
What I mean by "break", is that like any patch, if the original file is changed upstream, the patch might fail to be applied which is the case with the master branch of godot. |
Actually the patch does fail which is the reason I did this PR. Sorry, I did this 2 months ago, I totally forgot about it:
|
Thank you for your contributions.
|
I marked this as stale due to inactivity. → More info |
Closing due to inactivity from author. |
Motivation for this change
Partly reverts #34971.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @Twey