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.

Sign up for GitHub

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

Jump to bottom

Use default GOOS and allow to specify GOOS on cmd-line #10

Merged
merged 1 commit into from
Nov 15, 2019
Merged

Use default GOOS and allow to specify GOOS on cmd-line #10

merged 1 commit into from
Nov 15, 2019

Conversation

huksley
Copy link

@huksley huksley commented Nov 13, 2019 

# Build for current OS
make build

# Build for Linux
GOOS=linux make build

@kovetskiy
Copy link
Owner

Thanks! I'm ready to merge, but GitHub doesn't recognize your email and didn't associate commit with your account. Do you want to fix it or I can merge as is?

You can see your commit's user.email here: https://github.com/kovetskiy/mark/commit/7f310205ae56802eb90e06ed50397ef5b170aa32.patch

@kovetskiy kovetskiy merged commit 22f3e6c into kovetskiy:master Nov 15, 2019
@huksley
Copy link
Author

huksley commented Nov 24, 2019

Hi, thanks for merge!
Now email confirmed but I was not aware of this, thanks for pointing it out!

@huksley huksley deleted the build-os branch November 24, 2019 22:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@huksley @kovetskiy