Skip to content

Conversation

@Sjors
Copy link
Member

@Sjors Sjors commented Feb 12, 2019

Otherwise make src/bitcoin-wallet will fail with No rule to make target.

Also adds bitcoin-wallet.exe to the Windows installer.

@Sjors
Copy link
Member Author

Sjors commented Feb 12, 2019

I just duplicated everything to make it consistent with make src/bitcoin-tx.

@jnewbery: I think this was missing from #13926

@ken2812221 does the Windows installer bit look OK? @MarcoFalke this probably needs a Gitian build.

Thanks @luke-jr & @laanwj for IRC help.

@laanwj
Copy link
Member

laanwj commented Feb 12, 2019

LGTM utACK c9741df

@jnewbery
Copy link
Contributor

utACK c9741df229652ff1ea2a14d705a8fac727d6cd55. Thanks @Sjors!

@promag
Copy link
Contributor

promag commented Feb 12, 2019

Tested ACK c9741df on macOS.

@Sjors
Copy link
Member Author

Sjors commented Feb 13, 2019

Also added to share/setup.nsi.in.

@bitcoin bitcoin deleted a comment from DrahtBot Feb 13, 2019
@Sjors Sjors force-pushed the 2019/02/wallet_tool_make branch from cb870bb to 1852eff Compare February 14, 2019 08:59
@Sjors
Copy link
Member Author

Sjors commented Feb 14, 2019

I added 7640765 to see if busting AppVeyor cache (again) can help with breaking builds in master. It's otherwise unrelated to this PR so I can drop the commit or move it to another PR later if needed.

@Sjors Sjors force-pushed the 2019/02/wallet_tool_make branch 2 times, most recently from 7640765 to 1852eff Compare February 14, 2019 10:39
@Sjors
Copy link
Member Author

Sjors commented Feb 14, 2019

I moved the AppVeyor commits to a new PR #15405. See also #15407 for a likely solution to the AppVeyor failures.

@Sjors Sjors force-pushed the 2019/02/wallet_tool_make branch from 1852eff to 4a43eb8 Compare February 14, 2019 15:18
Copy link
Member

@luke-jr luke-jr left a comment

Choose a reason for hiding this comment

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

utACK

@maflcko maflcko merged commit 4a43eb8 into bitcoin:master Feb 14, 2019
maflcko pushed a commit that referenced this pull request Feb 14, 2019
4a43eb8 [build] Makefile.am: add rule for src/bitcoin-wallet (Sjors Provoost)

Pull request description:

  Otherwise `make src/bitcoin-wallet` will fail with `No rule to make target`.

  Also adds `bitcoin-wallet.exe` to the Windows installer.

Tree-SHA512: 932c34b349ec3efa17d1f1251d386434d0db6a07939d500236364a851468981c69392e2a00e26c9753d9b2d8ef056b5de870c5bd5202aa0c73326f9ed1403681
@Sjors Sjors deleted the 2019/02/wallet_tool_make branch February 14, 2019 15:38
@DrahtBot
Copy link
Contributor

Gitian builds for commit 0d1160e (master):

Gitian builds for commit a88d457155881cda117dbf84bd73c05f6bbdf804 (master and this pull):

PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Jul 1, 2021
…llet

4a43eb8 [build] Makefile.am: add rule for src/bitcoin-wallet (Sjors Provoost)

Pull request description:

  Otherwise `make src/bitcoin-wallet` will fail with `No rule to make target`.

  Also adds `bitcoin-wallet.exe` to the Windows installer.

Tree-SHA512: 932c34b349ec3efa17d1f1251d386434d0db6a07939d500236364a851468981c69392e2a00e26c9753d9b2d8ef056b5de870c5bd5202aa0c73326f9ed1403681
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Sep 11, 2021
…llet

4a43eb8 [build] Makefile.am: add rule for src/bitcoin-wallet (Sjors Provoost)

Pull request description:

  Otherwise `make src/bitcoin-wallet` will fail with `No rule to make target`.

  Also adds `bitcoin-wallet.exe` to the Windows installer.

Tree-SHA512: 932c34b349ec3efa17d1f1251d386434d0db6a07939d500236364a851468981c69392e2a00e26c9753d9b2d8ef056b5de870c5bd5202aa0c73326f9ed1403681
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Dec 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants