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

Pin the installed zkLLVM version #30

Closed
Tracked by #3
NickVolynkin opened this issue Jun 21, 2023 · 0 comments · Fixed by #31
Closed
Tracked by #3

Pin the installed zkLLVM version #30

NickVolynkin opened this issue Jun 21, 2023 · 0 comments · Fixed by #31
Assignees

Comments

@NickVolynkin
Copy link
Contributor

NickVolynkin commented Jun 21, 2023

The latest-released version is sometimes not the latest when sorted by version. Currently, the latest version is 0.0.71, but we have the following packages:

# apt-cache policy zkllvm
zkllvm:
  Installed: (none)
  Candidate: 0.1.0
  Version table:
     0.1.0 500
        500 http://deb.nil.foundation/ubuntu all/main amd64 Packages
     0.0.71 500
        500 http://deb.nil.foundation/ubuntu all/main amd64 Packages
     0.0.68 500
        500 http://deb.nil.foundation/ubuntu all/main amd64 Packages
     0.0.58 500
        500 http://deb.nil.foundation/ubuntu all/main amd64 Packages
     0.0.0-45 500
        500 http://deb.nil.foundation/ubuntu all/main amd64 Packages

More details can be found in NilFoundation/zkLLVM#120.

Solution:

  • pin the installed version in Dockerfile
  • tag images like nilfoundation/zkllvm-template:0.0.71
  • use specific versions ci.sh
NickVolynkin added a commit that referenced this issue Jun 21, 2023
Specify version explicitly to test and deliver a specific version of
zkLLVm compiler.

Resolve #30
NickVolynkin added a commit that referenced this issue Jun 21, 2023
Specify version explicitly to test and deliver a specific version of
zkLLVm compiler.

Resolve #30
NickVolynkin added a commit that referenced this issue Jun 21, 2023
Specify version explicitly to test and deliver a specific version of
zkLLVm compiler.

Change input format in the example according to the changes in
zkLLVM version 0.0.71. For more details, see
NilFoundation/zkLLVM@83134d1.

Resolve #30
NickVolynkin added a commit that referenced this issue Jun 21, 2023
Specify version explicitly to test and deliver a specific version of
zkLLVm compiler.

Change input format in the example according to the changes in
zkLLVM version 0.0.71. For more details, see
NilFoundation/zkLLVM@83134d1.

Resolve #30
NickVolynkin added a commit that referenced this issue Jun 21, 2023
Specify version explicitly to test and deliver a specific version of
zkLLVm compiler.

Resolve #30
NickVolynkin added a commit that referenced this issue Jun 21, 2023
zkLLVM and Proof Market toolchain are based on a few common libraries,
which should be updated simultaneously in both tools.

Now CI and ci.sh script will use explicitly pinned versions of both tools
to show how to use them together.

Resolve #30
NickVolynkin added a commit that referenced this issue Jun 21, 2023
zkLLVM and Proof Market toolchain are based on a few common libraries,
which should be updated simultaneously in both tools.

Now CI and ci.sh script will use explicitly pinned versions of both tools
to show how to use them together.

Resolve #30
NickVolynkin added a commit that referenced this issue Jun 21, 2023
zkLLVM and Proof Market toolchain are based on a few common libraries,
which should be updated simultaneously in both tools.

Now CI and ci.sh script will use explicitly pinned versions of both tools
to show how to use them together.

Resolve #30
NickVolynkin added a commit that referenced this issue Jun 26, 2023
zkLLVM and Proof Market toolchain are based on a few common libraries,
which should be updated simultaneously in both tools.

Now CI and ci.sh script will use explicitly pinned versions of both tools
to show how to use them together.

Resolve #30
@NickVolynkin NickVolynkin self-assigned this Jul 9, 2023
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.

1 participant