Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add windows build instructions #926

Merged
merged 6 commits into from
Sep 27, 2023

Conversation

KolbyML
Copy link
Member

@KolbyML KolbyML commented Sep 19, 2023

What was wrong?

We had no instructions for compiling windows

How was it fixed?

By writing instructions

@KolbyML KolbyML requested a review from carver September 19, 2023 03:36
@KolbyML KolbyML self-assigned this Sep 19, 2023
Copy link
Collaborator

@carver carver left a comment

Choose a reason for hiding this comment

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

Cool, thanks for putting this together!

I would have looked for this under the developer section, under releasing. https://ethereum.github.io/trin/developers/contributing/releases/release_checklist.html#build-the-binary-for-release

Two options that feel reasonable to me:

  1. Move this to developer release building section (optionally linking this user section to it for now, until we are posting pre-built binaries for use)
  2. Add a link in the developer release section pointing here. (this is okay, but is probably further from where we want to be eventually, which is keeping these as developer notes)

@KolbyML KolbyML force-pushed the Add-windows-compilation-instructions branch from 8f20adb to 30f5e5f Compare September 27, 2023 18:44
@KolbyML KolbyML merged commit 343c6d2 into ethereum:master Sep 27, 2023
@KolbyML KolbyML deleted the Add-windows-compilation-instructions branch January 22, 2025 07:53
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants