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

an argument of type std::option::Option<Reveal> is missing #18

Open
Goosie opened this issue Nov 28, 2022 · 4 comments
Open

an argument of type std::option::Option<Reveal> is missing #18

Goosie opened this issue Nov 28, 2022 · 4 comments

Comments

@Goosie
Copy link

Goosie commented Nov 28, 2022

A second warning comes by after running ./install.sh
Compiling rgb-cli v0.8.0-rc.1
error[E0061]: this function takes 4 arguments but 3 arguments were supplied
--> /home/rgbalice/.cargo/registry/src/github.com-1ecc6299db9ec823/rgb-cli-0.8.0-rc.1/src/command.rs:279:41
|
279 | let status = client.consume_transfer(consignment, force, progress)?;
| ^^^^^^^^^^^^^^^^ -------- an argument of type std::option::Option<Reveal> is missing
|
note: associated function defined here
--> /home/rgbalice/.cargo/registry/src/github.com-1ecc6299db9ec823/rgb_rpc-0.8.1/src/client.rs:260:12
|
260 | pub fn consume_transfer(
| ^^^^^^^^^^^^^^^^
help: provide the argument
|
279 | let status = client.consume_transfer(consignment, force, /* std::option::Option */, progress)?;
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

For more information about this error, try rustc --explain E0061.
error: could not compile rgb-cli due to previous error
error: failed to compile rgb-cli v0.8.0-rc.1, intermediate artifacts can be found at /tmp/cargo-install6AIY4I
Downloaded storm-cli v0.8.0

@crisdut
Copy link
Member

crisdut commented Dec 2, 2022

Hi @Goosie,

Here: #20

I updated all libs to correct version.

Please, try again.

@dr-orlovsky
Copy link
Member

I merged #20. Was this issue fixed?

@Goosie
Copy link
Author

Goosie commented Dec 28, 2022

Well following the instruction in the readme I'm still get an error:
I do:
cd repos
rm -rf nodes
git clone https://github.com/LNP-BP/nodes.git
cd repos/nodes
rustup update nightly
cargo +nightly install lnpbp_nodes -Z bindeps

Gives the error:
error: could not find lnpbp_nodes in registry crates-io with version *

@dr-orlovsky
Copy link
Member

Yes, since the original intent I had with doing this crate has failed: rust cargo package manager doesn't allow to publish crates made of binaries of other crates. This crate was created to simplify the installation of all LNP/BP nodes - which reside as binaries in their own crates - and cargo prohibits exactly that. So there is no way to solve that issue; the only possible approach is to have a docker containers and automation on building nodes into them. Related discussion: LNP-BP/docker#27 (comment)

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

No branches or pull requests

3 participants