Skip to content
This repository has been archived by the owner on Sep 9, 2020. It is now read-only.

Releases targeting Windows now have a .exe suffix #1292

Merged
merged 3 commits into from
Oct 24, 2017
Merged

Releases targeting Windows now have a .exe suffix #1292

merged 3 commits into from
Oct 24, 2017

Conversation

denisbrodbeck
Copy link
Contributor

What does this do / why do we need it?

The release tab on github has links to windows binaries. These links are named wrongly. After downloading, I have to rename dep-windows-amd64 into dep-windows-amd64.exe in order to use the binary.

Releases targeting Windows should have a .exe suffix.

Which issue(s) does this PR fix?

fixes #1291

Copy link
Collaborator

@ibrasho ibrasho left a comment

Choose a reason for hiding this comment

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

LGTM

@sdboyer sdboyer merged commit bf2481a into golang:master Oct 24, 2017
@denisbrodbeck denisbrodbeck deleted the build-all-windows-exe branch October 24, 2017 14:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Windows release file is named dep-windows-amd64 instead of dep-windows-amd64.exe
4 participants