profile
viewpoint

rgb-org/spec 144

[OLD!] RGB Protocol specifications for Bitcoin-based digital assets

gabridome/gabridome.github.io 1

Test di repository per pages

gabridome/trustedcoin 1

scripts to use trustedcoin service with Vitalik Buterin's pybitcointools

gabridome/21-lessons.github.io 0

21 Lessons - What I've Learned from Falling Down the Bitcoin Rabbit Hole

gabridome/abcore 0

ABCore - Android Bitcoin Core

gabridome/assobit.github.io 0

Repository creato per ospitare il sito web di AssoB.it

gabridome/bips 0

Bitcoin Improvement Proposals

gabridome/bitcoindevkit.org 0

BDK project home page (originally magicalbitcoin.org 🧙)

PR opened bitcoin-core/HWI

Fix trezor T multisig and tests

Fixes the Trezor T multisig tests. Part of this requires fixes to displayaddress and the Trezor implementation. The majority of the changes here are to make the multisig derivation paths conform to Trezor's purpose 48.

+39 -18

0 comment

4 changed files

pr created time in 2 hours

PR opened bitcoin-core/HWI

Refactor client.display_address into single sig and multisig variants

Instead of having display_address handle both single sig and multisig address display, it is refactored into display_singlesig_address to display single sig addresses, and display_multisig_address to display multisig addresses. Additionally the redeem_script parameter is removed as descriptors provide everything we need.

Supersedes #416. I think this approach is nicer to work with.

+443 -202

0 comment

13 changed files

pr created time in 4 hours

PR closed bitcoin-core/HWI

Refactor client.display_address

Refactors the display_address method in each device to split up display address by path and display address by descriptor. Instead of having both be done by the same function, we now have display_address_path and display_address_descriptor for each. This PR includes some changes to Descriptor in order to support this.

Builds on #415 for the address type changes done there.

+443 -156

0 comment

12 changed files

achow101

pr closed time in 4 hours

pull request commentbitcoin-core/HWI

Bitbox02 multisig

@benma Just for your information, #416 may end up being closed and a different approach taken. So It might be better for you to leave the display multisig address part for last. I'm still trying to work out a good API.

benma

comment created time in 6 hours

pull request commentbitcoin-core/HWI

Bitbox02 multisig

Thanks! I will get on this asap.

benma

comment created time in 6 hours

PR merged bitcoin-core/HWI

Allow python 3.9, update dependencies, and bump version to 2.0.0-dev

This is general pyproject.toml maintenance. Python 3.9 is the latest python version, so we should allow it. Ideally, we would not set a python version upper bound, but the pyside2 package sets a python version upper bound, so poetry requires us to also set that.

Additionally, the version number in the master branch was super out of date. 2.0.0-dev is more accurate - we are developing for 2.0.0 in master currently.

Lastly, all of the dependencies in poetry.lock are updated.

+172 -251

0 comment

4 changed files

achow101

pr closed time in 7 hours

push eventbitcoin-core/HWI

Andrew Chow

commit sha 9d395fb73fa8ff0fcbf2fdf7a76fbd78493a6f2d

Allow python 3.9

view details

Andrew Chow

commit sha 225d10e75ab8fa2f5e95e68b7997c1526f5aee5a

Set version to 2.0.0-dev

view details

Andrew Chow

commit sha a00128b2fb426cba3f00d5d6aa38f4713267681d

Merge #421: Allow python 3.9, update dependencies, and bump version to 2.0.0-dev 225d10e75ab8fa2f5e95e68b7997c1526f5aee5a Set version to 2.0.0-dev (Andrew Chow) 9d395fb73fa8ff0fcbf2fdf7a76fbd78493a6f2d Allow python 3.9 (Andrew Chow) Pull request description: This is general pyproject.toml maintenance. Python 3.9 is the latest python version, so we should allow it. Ideally, we would not set a python version upper bound, but the pyside2 package sets a python version upper bound, so poetry requires us to also set that. Additionally, the version number in the master branch was super out of date. 2.0.0-dev is more accurate - we are developing for 2.0.0 in master currently. Lastly, all of the dependencies in `poetry.lock` are updated. ACKs for top commit: laanwj: Code review (except poetry.lock) ACK 225d10e75ab8fa2f5e95e68b7997c1526f5aee5a Tree-SHA512: 1db93a5e68d146a418c636a78c419d0e675935da008fab2266f40df944b723955aa28ce1fb26ed95cf489991f164545d7581c06213ce5d1f2c1660897f47411f

view details

push time in 7 hours

pull request commentbitcoin-core/HWI

Bitbox02 multisig

Sorry for taking so long to get to this.

I would like for this PR to be split up as there are some upcoming changes that are going to conflict with this, namely #415 and #416 which refactor display_address. Given my lack of familiarity with the BitBox02 implementation, I think it would be better for those parts of this PR to be based on those PRs and for other components that do not conflict to be in separate PRs to be merged soon.

Here are the PRs that I suggest this be split into:

  • PSBT_GLOBAL_XPUB parsing:
    • 9cc1fd1615771ec0f53e0349cf32fc7164d37d9f PSBT: also parse and serialize PSBT_GLOBAL_XPUB
  • Bitbox02 5.1.0 based on #421
    • c9c4d7540b4162f50c3fcf66321c1631f9c0cca5 bitbox02: update bitbox02 to 5.1.0
  • BitBox02 Multisig signing
    • e9aa60063f3f64679564fdb6b6194d5fc0fe2b19 bitbox02: restructure signing code
    • 5916c300dd4b447d9d4191ea149b3c884e41b4ba bitbox02: implement multisig signing
    • Some of e689ed0d4ec9f29e26bce612ea54e60c4adb01f0 bitbox02: update docs
  • BitBox02 multisig address display based on #416
    • 37ef0626c36031ea1c68d0999a8e70e207ef6228 bitbox02: implement multisig display_address()

1deefb0f2635a52e6f62acca9137c50097e71ee7 can be dropped if this is based on #416 because that removes the whole redeem script thing from multisig address display.

af619729f266bfaad443d0af60046feccdaa617f can be dropped since it is an unrelated formatting change.

f8f44a6e68bd797ce4eb79b9bd607ed587180e18 could be dropped or squashed into another commit.

benma

comment created time in 7 hours

issue commentbitcoin-core/HWI

Bitbox02 failing type check

This particular issue is fixed in this commit:

https://github.com/bitcoin-core/HWI/pull/407/commits/c9c4d7540b4162f50c3fcf66321c1631f9c0cca5

I could extract it into a separate PR if you want.

achow101

comment created time in 16 hours

issue openedbitcoin-core/HWI

Bitbox02 failing type check

BitBox02 is failing the type checker: https://cirrus-ci.com/task/6500295159054336

cc @benma

created time in a day

PR closed bitcoin-core/HWI

Remove keepkey

Closes #386

+41 -585

8 comments

17 changed files

achow101

pr closed time in a day

pull request commentbitcoin-core/HWI

Remove keepkey

Seems like we should keep Keepkey support.

achow101

comment created time in a day

PR opened bitcoin-core/HWI

Allow python 3.9, update dependencies, and bump version to 2.0.0-dev

This is general pyproject.toml maintenance. Python 3.9 is the latest python version, so we should allow it. Ideally, we would not set a python version upper bound, but the pyside2 package sets a python version upper bound, so poetry requires us to also set that.

Additionally, the version number in the master branch was super out of date. 2.0.0-dev is more accurate - we are developing for 2.0.0 in master currently.

Lastly, all of the dependencies in poetry.lock are updated.

+171 -250

0 comment

3 changed files

pr created time in a day

issue closedbitcoin-core/HWI

pysdie2 package causing poetry to hang

Doing any command which requires updating dependencies (i.e. poetry update or poetry lock) causes poetry to hang. It seems like it is hanging on resolving the dependencies for the pyside2 package.

Edit: This originally said bitbox02 but I did some debugging and it seems to be pyside2 that's causing the problem.

closed time in a day

achow101

issue commentbitcoin-core/HWI

pysdie2 package causing poetry to hang

Apparently I have a caching problem. Clearing some poetry http cache fixes this.

achow101

comment created time in a day

startedcloudflare/ebpf_exporter

started time in 2 days

created repositorylaanwj/laanwj

created time in 3 days

pull request commentbitcoin-core/HWI

Remove keepkey

I am using keepkey with Specter and Sparrow Instead of removing Keepkey I would recommend to copy the trezorlib implementation and just make it Keepkey specific similar as Electrum has done (so Keepkey would be implemented as a separate device instead of as a Trezor device. And yes this will mean that there's a lot of duplicated code)

Maybe @keepkeyjoncan help/assist you through the work of patching trezorlib to work with the keepkey every time it is updated.

achow101

comment created time in 3 days

startedcloudhead/nakamoto

started time in 4 days

pull request commentbitcoin-core/HWI

Improve setup_environment.sh instructions

Oops, I completely forgot about this PR. I don't even remember making it, so let's call it up for grabs.

Sjors

comment created time in 5 days

issue openedbitcoin-core/HWI

BitBox02 package causing poetry to hang

Doing any command which requires updating dependencies (i.e. poetry update or poetry lock) causes poetry to hang. It seems like it is hanging on resolving the dependencies for the bitbox02 package.

created time in 5 days

PR closed bitcoin-core/HWI

Remove several system libraries from PyInstaller build build system up for grabs

PyInstaller was including a bunch of system libraries that we don't need. Removing them makes the final build smaller and lets us not have to worry about them when it comes to reproducibility. These libraries are not used by HWI but are imported by python built-ins that we aren't using. These all have handling for the libraries not being there so it is fine to exclude them.

The libraries being excluded are:

  • libz
  • libbz2
  • libzma
  • libreadline

libz (zlib) has to be handled more specially when python is being built during the docker container setup. The python compilation will include it if it detects it, so we apply an additional patch to CPython for it to not do that.

+23 -2

1 comment

3 changed files

achow101

pr closed time in 5 days

PR closed bitcoin-core/HWI

Pin additional dependency versions to make builds reproducible build system up for grabs

Adds additional version pinning so that builds are more reproducible. Poetry for the build also has a fixed version number. Cython is now a dependency so that hidapi is built reproducibly.

Fixes #303

+306 -24

2 comments

5 changed files

achow101

pr closed time in 5 days

pull request commentbitcoin-core/HWI

Remove keepkey

Ok for 2.0 this is probably fine.

On Sat, Jan 23, 2021, 7:28 AM Andrew Chow notifications@github.com wrote:

Could we put it behind a deprecation flag or something for a release cycle? If no complaints show up, remove?

Since I am trying to do SemVer correctly, that seems like it would require 2 major releases to do that since it is ostensibly an API breaking change. The big promise of 2.0 is that we can break the API.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/bitcoin-core/HWI/pull/418#issuecomment-765741844, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABMAFUZQB3GTPGCNNWGPVBTS3IC2JANCNFSM4WLHWRDA .

achow101

comment created time in 5 days

PR closed bitcoin-core/HWI

Allow CLI to limit enumerate search to specific device type

This is helpful when debugging a specific device module import/dependency issue.

+13 -4

4 comments

2 changed files

PierreRochard

pr closed time in 5 days

pull request commentbitcoin-core/HWI

Allow CLI to limit enumerate search to specific device type

Closing due to lack of activity. marking as up for grabs

PierreRochard

comment created time in 5 days

PR closed bitcoin-core/HWI

Improve setup_environment.sh instructions tests
+3 -3

1 comment

1 changed file

Sjors

pr closed time in 5 days

pull request commentbitcoin-core/HWI

Improve setup_environment.sh instructions

Closing due to lack of activity.

Sjors

comment created time in 5 days

pull request commentbitcoin-core/HWI

Remove keepkey

Could we put it behind a deprecation flag or something for a release cycle? If no complaints show up, remove?

Since I am trying to do SemVer correctly, that seems like it would require 2 major releases to do that since it is ostensibly an API breaking change. The big promise of 2.0 is that we can break the API.

achow101

comment created time in 5 days

PR closed bitcoin-core/HWI

WIP Try to fix bitcoind path for travis

Looks like all the default paths for bitcoind and various simulators are off for Travis. Poking around to see if I can get to more meaningful travis failures/results.

+16 -20

1 comment

2 changed files

instagibbs

pr closed time in 5 days

more