Skip to content

Add symlinks to solc-bin/wasm for releases.#8227

Merged
chriseth merged 2 commits intodevelopfrom
wasmReleaseSymlink
Feb 3, 2020
Merged

Add symlinks to solc-bin/wasm for releases.#8227
chriseth merged 2 commits intodevelopfrom
wasmReleaseSymlink

Conversation

@ekpyron
Copy link
Copy Markdown
Collaborator

@ekpyron ekpyron commented Feb 3, 2020

Unfortunately, I'm not sure there's a good way to actually test this... we might just have to keep in mind having an eye on this for the next release.

I also partially addressed #8127 in a separate commit here, since I was touching the file anyways, but I can move that to a separate PR, if you prefer. We still need to decide how we want to deal with the leading zeros in commit hashes of past releases in any case.

Copy link
Copy Markdown
Contributor

@chriseth chriseth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is interesting: We publish the first commit on the release branch instead of publishing the tagged commit. This is a remnant of the time where we did not have github releases. We should change this at some point.

@leonardoalt
Copy link
Copy Markdown

This is interesting: We publish the first commit on the release branch instead of publishing the tagged commit. This is a remnant of the time where we did not have github releases. We should change this at some point.

Wanna create an issue for that?
@chriseth

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 this pull request may close these issues.

3 participants