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

Add msi #58

Merged
merged 5 commits into from
Jan 21, 2025
Merged

Add msi #58

merged 5 commits into from
Jan 21, 2025

Conversation

ale7714
Copy link
Member

@ale7714 ale7714 commented Jan 14, 2025

Since the workflow doesn't exist in the main branch, we can't use the UI but you can use the CLI to kick of the workflow

  1. Go to this link to see the latest successful build and the version on the artifact.
  2. Run gh workflow run build-msi.yaml --ref ap-add-msi -f msi_version=VERSION
  3. Go to the Workflow on Github to get the artifact

@ale7714 ale7714 marked this pull request as draft January 14, 2025 17:08
agent.msi Outdated
Copy link
Member

Choose a reason for hiding this comment

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

.gitignore probably?

agent.wixpdb Outdated
Copy link
Member

Choose a reason for hiding this comment

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

maybe .gitignore

Makefile Outdated
GOOS=windows GOARCH=amd64 go build -o $@ -trimpath -tags $(TAGS) -ldflags $(LDFLAGS) ./cmd/viam-agent
file $@
du -hc $@
go build -o $@ -trimpath ./cmd/viam-agent
Copy link
Member

Choose a reason for hiding this comment

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

$(LDFLAGS) here potentially affects file size

@ale7714 ale7714 marked this pull request as ready for review January 15, 2025 22:35
@ale7714 ale7714 requested a review from abe-winter January 15, 2025 22:35
@ale7714
Copy link
Member Author

ale7714 commented Jan 15, 2025

@abe-winter i kicked off a workflow using the latest of my PR and here's the artifact https://github.com/viamrobotics/agent/actions/runs/12798170184

do you want try it on your laptop?

@ale7714 ale7714 changed the title [wip] add msi Add msi Jan 15, 2025
@abe-winter
Copy link
Member

works!!! deleted my existing service + my opt folder and agent + RDK came up

@ale7714 ale7714 merged commit 9fdfb4e into aw-windows Jan 21, 2025
2 of 4 checks passed
@ale7714 ale7714 deleted the ap-add-msi branch January 21, 2025 17:05
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.

2 participants