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

Update versions in install.sh, fix demo-rgb.sh #23

Merged
merged 3 commits into from
Jul 8, 2023

Conversation

adambor
Copy link
Contributor

@adambor adambor commented Jun 24, 2023

Hey,

I updated the cargo crates versions in install.sh to 0.9. Was going through the installation and also made necessary edits to demo-rgb.sh file, there were some typos (rgb -> rgb-cli), some services were using electrum nodes which are not online anymore (rgbd, rgb) and added a hint to download some apt dependencies which caused my cargo installs to fail.

Tested and working on my machine running ubuntu 22 making a transfer to myself (on a single machine - so without lnpd and stormd).

#22

Fixed typos: on some lines "rgb" should be "rgb-cli".
Add electrum url to some commands (which were using an electrum server which is offline now by default - rgb, rgbd).
Added apt dependencies that caused my cargo installs to fail.
Copy link
Member

@dr-orlovsky dr-orlovsky left a comment

Choose a reason for hiding this comment

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

Thank you. Please pay attention that version v0.9 is not supported and is deprecated in favour of v0.10 (which is backwards-incompatible with all previous versions). v0.10 doesn't require RGB node and works from the command line; you can find more information on https:/

We also need to change the way we install dependencies.

contrib/demo-rgb.sh Outdated Show resolved Hide resolved
contrib/demo-rgb.sh Outdated Show resolved Hide resolved
@dr-orlovsky dr-orlovsky merged commit a783a17 into LNP-BP:master Jul 8, 2023
0 of 6 checks passed
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