Skip to content

Add information for a usage with Windows#10

Open
wikijm wants to merge 1 commit into
christophetd:masterfrom
wikijm:master
Open

Add information for a usage with Windows#10
wikijm wants to merge 1 commit into
christophetd:masterfrom
wikijm:master

Conversation

@wikijm

@wikijm wikijm commented Mar 4, 2018

Copy link
Copy Markdown

No description provided.

Comment thread README.md
```

- To specify a path to a Firefox profile directory, use the `--profile` option (by default: the first file found matching `~/.mozilla/firefox/*.default` on Ubuntu or `~/Library/Application\ Support/Firefox/Profiles/*.default` on Mac OS
- To specify a path to a Firefox profile directory, use the `--profile` option (by default: the first file found matching `~/.mozilla/firefox/*.default` on Ubuntu, `~/Library/Application\ Support/Firefox/Profiles/*.default` on Mac OS or C:\Users\%username%\AppData\Roaming\Mozilla\Firefox\Profiles\*.default on Windows)

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

that's indeed how the code should behave, but for now it doesn't use this by default for Windows. Tracked in #11

@christophetd

christophetd commented Mar 5, 2018

Copy link
Copy Markdown
Owner

Hi,

Thanks for the PR! I got some feedback from someone else using Windows that firepwned didn't work out of the box for him, and that he needed to install NSS in order to make it work (which is significantly harder on Windows than on Unix).

If that's fine for you, I think it would make sense to:

I'll take care of this as soon as I have a bit of time, otherwise contributions are obviously welcomed. :)

Cheers!

@wikijm

wikijm commented Mar 5, 2018

Copy link
Copy Markdown
Author

Hi @christophetd,

I just gave an answer in the issue #11.
Tell me if it fits your expectations 😃

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