-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
brave: fix config dir path #2173
Conversation
Changes install path for extensions from `~/.config/brave` to `~/.config/BraveSoftware/Brave-Browser` on Linux.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
That was fast! Thanks y'all. |
Anyway to get this merged into 21.05 ? Thanks |
@yisraeldov I think you can just submit a PR against the |
|
Changes install path for extensions from `~/.config/brave` to `~/.config/BraveSoftware/Brave-Browser` on Linux. (cherry picked from commit 9e3c402)
I've cherry-picked the commit to release-21.05 in b39647e. Please give it a try. |
Great! This works now as expected on brave! Is there a way to get it to work for multiple browsers? |
Description
Checklist
Not if the user overrides
programs.brave.package
with an older version of Brave.Code formatted with
./format
.Code tested through
nix-shell --pure tests -A run.all
.Test cases updated/added. See example.
Commit messages are formatted like
See CONTRIBUTING for more information and recent commit messages for examples.
If this PR adds a new module
Added myself as module maintainer. See example.
Added myself and the module files to
.github/CODEOWNERS
.