search

L2-Technology / sensei

A lightning node implementation for everyone
https://l2.technology/sensei
Other
198 stars 39 forks source link

Fix/broadcaster interface #19

Closed w3irdrobot closed 2 years ago

w3irdrobot commented 2 years ago

I was getting compiler errors when trying to build it. The reference in the git URL for bdk-ldk was not recognized correctly by cargo. So I moved it to the ref attribute in the Cargo.toml for that dependency. I then updated the commit it was pulling down. This was because the version of lightning was different at this commit that other deps. So some traits were not recognized as being implemented.

johncantrell97 commented 2 years ago

Ah sorry you ran into this issue. Unfortunately (or maybe fortunately) that dependency is completely removed in my latest branch (https://github.com/L2-Technology/sensei/pull/17). I simplified a ton of the chain data code to rely more heavily on ldk's lightning-block-sync and now (only) support bitcoind directly.

w3irdrobot commented 2 years ago

Should we merge this in the mean time, just to get main building? If not, we can just close this PR.

johncantrell97 commented 2 years ago

sure!