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

Creating release binaries? #5447

Closed
prologic opened this issue Mar 20, 2019 · 2 comments
Closed

Creating release binaries? #5447

prologic opened this issue Mar 20, 2019 · 2 comments

Comments

@prologic
Copy link

I'm trying to cross-compile a fairly recent version of master (actually this branch which I have #5446 up for) using the prerelease and release targets in your GNUMakefile as per documentation.

This doesn't work however:

james@Jamess-MacBook-Pro
Wed Mar 20 15:27:27
~/go/src/github.com/hashicorp/nomad
 (update_go-getter) 2
$ make prerelease && make release
==> Linting source code...
client/allocrunner/taskrunner/task_runner_test.go:713:27:warning: unnecessary conversion (unconvert)
command/agent/config_test.go:987:35:warning: unnecessary conversion (unconvert)
command/agent/config_test.go:946:35:warning: unnecessary conversion (unconvert)
command/agent/config_test.go:926:35:warning: unnecessary conversion (unconvert)
command/agent/config_test.go:905:35:warning: unnecessary conversion (unconvert)
command/agent/config_test.go:965:35:warning: unnecessary conversion (unconvert)
drivers/shared/executor/executor_test.go:230:35:warning: unnecessary conversion (unconvert)
drivers/shared/executor/executor_test.go:144:36:warning: unnecessary conversion (unconvert)
nomad/autopilot_test.go:222:29:warning: unnecessary conversion (unconvert)
nomad/stats_fetcher_test.go:71:34:warning: unnecessary conversion (unconvert)
nomad/stats_fetcher_test.go:72:48:warning: unnecessary conversion (unconvert)
scheduler/generic_sched_test.go:625:22:warning: unnecessary conversion (unconvert)
make: *** [check] Error 1

What am I doing wrong? Is this failing because of the linter warnings? Have we merged code that doesn't pass the linter checks in the prerelease target?

@notnoop
Copy link
Contributor

notnoop commented Mar 20, 2019

That's odd and it doesn't match our "Linting source code" experience in CI: e.g. https://travis-ci.org/hashicorp/nomad/jobs/508879932 . It's curious that all warnings are related to test files and I wonder if there is a configuration difference in there.

I'll look into this when I get a chance.

@github-actions
Copy link

I'm going to lock this issue because it has been closed for 120 days ⏳. This helps our maintainers find and focus on the active issues.
If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 25, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants