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

PR check for comparing release binaries added to solc-bin with those in github releases #9930

Closed
cameel opened this issue Sep 30, 2020 · 4 comments
Assignees
Labels
closed due inactivity The issue/PR was automatically closed due to inactivity. low effort There is not much implementation work to be done. The task is very easy or tiny. low impact Changes are not very noticeable or potential benefits are limited. solcbin stale The issue/PR was marked as stale because it has been open for too long. testing 🔨

Comments

@cameel
Copy link
Member

cameel commented Sep 30, 2020

Extracted from #9258.

We want the release binaries added to solc-bin to be identical with those on github release page. Since the process of adding them to solc-bin is manual right now, we should really have an automated check for that, running on all solc-bin PRs.

The easiest way to do so is to just download the binaries from github release page and compare. I already have a script for getting the binaries from both Github and CircleCI: ethereum/solc-bin#35, ethereum/solc-bin#41. It was originally meant to make it easier for the uploader to get them but we shelved that idea. It can be easily repurposed as a part of the check. I just need to do minor modification and run it inside a github action.

Relevant parts from the original issue

#9258 (comment)

  • Binaries in solc-bin must be the exact same binaries as the ones on the release page.
    • Maybe we could have the manual update script check that.

#9258 (comment)

Right now @chriseth wants release binaries to be uploaded manually so it's easy to ensure that they're the same. Just upload the same files to both places :) This would be just a sanity check to ensure you did just that.

And if we automate publishing releases, I think it would just be a matter of having the same CI job upload both to solc-bin and to the release page

@cameel
Copy link
Member Author

cameel commented Sep 30, 2020

We discussed it on the call today. We decided that the check itself does not make much sense since the binaries on github release page can actually be changed at any time. We might still want to go back to the idea of the script for getting the binaries though.

We're going to wait until @ekpyron is back to discuss it.

@cameel cameel changed the title PR check for release binaries added to solc-bin PR check for comparing release binaries added to solc-bin with those in github releases Sep 14, 2022
@cameel cameel added testing 🔨 low effort There is not much implementation work to be done. The task is very easy or tiny. low impact Changes are not very noticeable or potential benefits are limited. labels Sep 14, 2022
@cameel
Copy link
Member Author

cameel commented Sep 14, 2022

We should discuss if we still want it. The process is still manual but with #12929 the binaries are unlikely to go out of sync, unless someone changes them intentionally. We're also considering automating the process of submitting binaries to solc-bin so a discrepancy would be even less likely (though not impossible).

@github-actions
Copy link

github-actions bot commented Mar 9, 2023

This issue has been marked as stale due to inactivity for the last 90 days.
It will be automatically closed in 7 days.

@github-actions github-actions bot added the stale The issue/PR was marked as stale because it has been open for too long. label Mar 9, 2023
@github-actions
Copy link

Hi everyone! This issue has been automatically closed due to inactivity.
If you think this issue is still relevant in the latest Solidity version and you have something to contribute, feel free to reopen.
However, unless the issue is a concrete proposal that can be implemented, we recommend starting a language discussion on the forum instead.

@github-actions github-actions bot added the closed due inactivity The issue/PR was automatically closed due to inactivity. label Mar 17, 2023
@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Mar 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed due inactivity The issue/PR was automatically closed due to inactivity. low effort There is not much implementation work to be done. The task is very easy or tiny. low impact Changes are not very noticeable or potential benefits are limited. solcbin stale The issue/PR was marked as stale because it has been open for too long. testing 🔨
Projects
None yet
Development

No branches or pull requests

2 participants