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
firefox: Prefix $PATH with xdg_utils #105894
Conversation
❤️ Edit: testing this messed up my browser profile … wow. 😢 |
Yikes. I'm guessing this is the result of another firefox patch that landed in master yesterday. |
Yeah, sorry I wasn't clear about this. This change is totally fine and a
very welcome fix.
…On Fri, Dec 04, 2020 at 08:42:29AM -0800, ryneeverett wrote:
> Edit: testing this messed up my browser profile … wow.
Yikes. I'm guessing this is the result of another firefox patch that landed in master yesterday.
--
You are receiving this because you commented.
Reply to this email directly or view it on GitHub:
#105894 (comment)
|
Can you please add a motivation to your commit message, so that we don't need to rely on GitHub history to follow the intent of your commit. |
This is required for certain URIs that require launching external programs (e.g. mailto:, magnet:, or irc:) or setting the default browser via xdg-settings. Resolve NixOS#92751. Comparable to NixOS#96922.
8482e08
to
36436ee
Compare
@GrahamcOfBorg firefox |
ah oops |
I've run with this for close to a week. Thanks! |
Backported this in 1b5f650. |
Motivation for this change
Resolve #92751.
Comparable to #96922.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)