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

browser_repository.rs: Fix Firefox incognito argument #129

Merged
merged 1 commit into from
Jan 27, 2024

Conversation

mpsijm
Copy link
Contributor

@mpsijm mpsijm commented Jan 27, 2024

This should not be --private, but --private-window. Probably explains why opening in incognito mode (when holding shift) does not work for me for Firefox profiles 😛

Also, I changed the incognito arguments for both Chrome and Firefox to use the canonical double dash instead of single dash.

P.S.: Since I don't need a Rust toolchain set up on my machine, I don't want to use https://aur.archlinux.org/packages/browsers-git, so I created https://aur.archlinux.org/packages/browsers-bin. Let me know if I need to transfer ownership to you, but I'm happy to bump the version number once in a while 🙂

This should not be `--private`, but `--private-window`.

Also changed the incognito arguments for both Chrome and Firefox
to use the canonical double dash instead of single dash.
@liias
Copy link
Contributor

liias commented Jan 27, 2024

Hey, Maarten. Good catch and thank you for the fix! Not sure how I never caught it before.

Regarding AUR - I wasn't aware there's browsers-git created by someone there.
I'm happy to outsource the bin repository as well for now.

@liias liias merged commit a5417d0 into Browsers-software:main Jan 27, 2024
@mpsijm mpsijm deleted the patch-1 branch January 28, 2024 09:52
@liias
Copy link
Contributor

liias commented Jan 31, 2024

The Firefox fix is now released in 0.5.1.

There's also a small addition for install.sh, if you pass --skip-desktop-database to it, then the desktop mime database is not updated and you should be able to remove line 24 from https://aur.archlinux.org/cgit/aur.git/tree/PKGBUILD?h=browsers-bin#n24

@mpsijm
Copy link
Contributor Author

mpsijm commented Jan 31, 2024

Thanks, you're the best! 😄

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