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

Update docs with instructions for windows build. #911

Merged
merged 1 commit into from
Feb 9, 2022

Conversation

jefchien
Copy link
Member

Description: Added make target for windows build. Chocolatey is already required for create_msi.ps1, so it can likely be used to install make as well. https://community.chocolatey.org/packages/make

choco install wixtoolset --force -y

Link to tracking Issue: #893

@codecov-commenter
Copy link

codecov-commenter commented Jan 24, 2022

Codecov Report

Merging #911 (1680578) into main (07265a3) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #911   +/-   ##
=======================================
  Coverage   53.69%   53.69%           
=======================================
  Files          12       12           
  Lines         419      419           
=======================================
  Hits          225      225           
  Misses        177      177           
  Partials       17       17           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2604d6d...1680578. Read the comment docs.

@@ -1,6 +1,6 @@
### Build Artifacts
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we include requirements such as install make, git, golang version instead of having independent packages?

Copy link
Contributor

@bryan-aguilar bryan-aguilar left a comment

Choose a reason for hiding this comment

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

LGTM

@bryan-aguilar bryan-aguilar merged commit 35aa30c into aws-observability:main Feb 9, 2022
@jefchien jefchien deleted the 893-update-docs branch February 9, 2022 21:53
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.

Powershell script for creating windows MSI does not work - cannot deploy as service to windows server
4 participants