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

A0-2962: Dockerfile #51

Closed
mgralinski-bright opened this issue Aug 3, 2023 · 4 comments · Fixed by #52
Closed

A0-2962: Dockerfile #51

mgralinski-bright opened this issue Aug 3, 2023 · 4 comments · Fixed by #52

Comments

@mgralinski-bright
Copy link

I was trying to create a docker file where I could run ink-wrapper. For reasons unknown to me, ink-wrapper produces wrong output. I am attaching sample.zip where you will find everything to reproduce the issue.

@obrok
Copy link
Collaborator

obrok commented Aug 4, 2023

Thanks for the report, I'll look into it!

@obrok obrok changed the title Dockerfile A0-2962: Dockerfile Aug 4, 2023
@obrok
Copy link
Collaborator

obrok commented Aug 4, 2023

The problem seems to be with the version of rust you're using to install ink-wrapper. I believe this is related to an issue in a dependency of the library we use for code generation and the API it depends on, which is now being changed in nightly rust (udoprog/genco#39). For now, I'd recommend installing with:

cargo +nightly-2023-04-16 install ink-wrapper --locked --version 0.5.0

to basically lock the toolchain you're using to compile ink-wrapper to one that's known to work (I checked, and this one indeed works). I'll add a note about this to the README, and in the future, I hope the relevant changes will just be released into rust proper, so we can set min rust version and it will just work automatically or fail with an appropriate message.

@obrok
Copy link
Collaborator

obrok commented Aug 4, 2023

Please let me know if this solves your problem, or open another issue if you have any other problems.

@mgralinski-bright
Copy link
Author

Installing ink-wrapper with fixed version nightly-2023-04-16, solved the issue. Thanks!

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 a pull request may close this issue.

2 participants