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

doc: small install doc updates #1744

Merged
merged 1 commit into from
Nov 5, 2024

Conversation

MarnixCroes
Copy link
Contributor

Small doc improvements/updates

@@ -58,8 +58,7 @@ Then build and install a local copy of libsecp256k1 for python-bitcointx:
cd deps
git clone https://github.com/bitcoin-core/secp256k1.git
cd secp256k1
git checkout v0.4.1
make clean
Copy link
Contributor Author

Choose a reason for hiding this comment

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

remove make clean, it doesn't do anything
(also not mentioned in secp doc)

@@ -58,8 +58,7 @@ Then build and install a local copy of libsecp256k1 for python-bitcointx:
cd deps
git clone https://github.com/bitcoin-core/secp256k1.git
cd secp256k1
git checkout v0.4.1
make clean
git checkout v0.5.0
Copy link
Contributor Author

Choose a reason for hiding this comment

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

docs/INSTALL.md Outdated
@@ -203,7 +201,7 @@ Once you have finished installing the program via one of the two above methods f
#### Configuring JoinMarket
Lastly we must configure JoinMarket to allow it to connect to Bitcoin Core. Refer to [this](USAGE.md#managing-your-joinmarket-data) section in the usage guide to generate a `joinmarket.cfg` file using `scripts/wallet-tool.py`.

Edit your `joinmarket.cfg` file (at `~/.joinmarket` in Ubuntu if you used WSL, or in `C:\Users\<your username>\AppData\Roaming\joinmarket` if not) and replace the following lines in the section `[BLOCKCHAIN]`
Edit your `joinmarket.cfg` file (at `~/.joinmarket` in Ubuntu if you used WSL, or in `C:\Users\<your username>\AppData\Local\joinmarket` if not) and replace the following lines in the section `[BLOCKCHAIN]`
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I don't use windows, but afaik this changed there recently: roaming -> local

Copy link
Member

Choose a reason for hiding this comment

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

Depends on a Windows version I guess? On my Windows 10 laptop it's Roaming.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes probably win11,

at least I thought so but a more specific web search says otherwise :) https://www.makeuseof.com/tag/appdata-roaming-vs-local/

my bad, I dropped the change

@@ -151,7 +150,6 @@ There are currently two choices for installing on Windows; one, directly install

#### Installation directly on Windows

As per the note above, binaries for JoinmarketQt are being offered with releases as of 0.7.0+.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

removed this, as latest releases didn't have .exe

Copy link
Member

@kristapsk kristapsk left a comment

Choose a reason for hiding this comment

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

ACK 6de1bab

@kristapsk kristapsk merged commit 4e9a08d into JoinMarket-Org:master Nov 5, 2024
10 of 11 checks passed
@MarnixCroes MarnixCroes deleted the install-doc branch November 6, 2024 06:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants