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

Install fails on macOS Catalina #18

Open
sunknudsen opened this issue Apr 9, 2021 · 6 comments
Open

Install fails on macOS Catalina #18

sunknudsen opened this issue Apr 9, 2021 · 6 comments
Labels

Comments

@sunknudsen
Copy link

sunknudsen commented Apr 9, 2021

Installed Rust using brew install rust on macOS Catalina and ran cargo install --git https://github.com/dsprenkels/sss-cli --branch v0.1.

Following error is thrown:

  = note: Undefined symbols for architecture x86_64:
            "_crypto_secretbox_xsalsa20poly1305_tweet", referenced from:
                shamirsecretsharing_cli::crypto_secretbox::hec24fa59a0001d22 in libshamirsecretsharing_cli-d5d35100b5254f45.rlib(shamirsecretsharing_cli-d5d35100b5254f45.shamirsecretsharing_cli.8d6pcdmd-cgu.0.rcgu.o)
          ld: symbol(s) not found for architecture x86_64
          clang: error: linker command failed with exit code 1 (use -v to see invocation)

New to Rust, so might be missing something obvious.

@sunknudsen
Copy link
Author

@vitkabele I see you maintain a Homebrew Formula. Would you happen to know what I am doing wrong?

@vitkabele
Copy link
Contributor

It seems to me that there is a missing dependency to tweetnacl-sys library. I check it and create PR

@vitkabele
Copy link
Contributor

vitkabele commented Apr 11, 2021

It is already fixed in the master branch by cece61d
@dsprenkels Do you plan to add a new release with this fix? The brew formula depends on the packed release tarball.

@dsprenkels
Copy link
Owner

Ah I did not know of the Homebrew scripts. I will push a new release asap!

@sunknudsen
Copy link
Author

Thanks for helping out guys!

I was able to compile shamirsecretsharing-cli using cargo install --git https://github.com/dsprenkels/sss-cli --branch master.

@dsprenkels Should README.md reference master vs v0.1?

@dsprenkels
Copy link
Owner

dsprenkels commented Apr 13, 2021

Hey all, I pushed a new release with this fix. It should be fixed now.

This was initially caused by me upgrading the sss-rs crate to a full only-Rust version. This crate reused some functions from that crate that were exported from C, and removing those functions from the sss-rs crate broke this crate.

I "fixed" it for now by pinning the sss-rs dependency to version 0.1.4. In other words: this crate uses the old sss-rs dependency. The plan is to also implement this crate completely in Rust.

Sorry for the inconvenience.

@dsprenkels dsprenkels added the bug label Apr 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants