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

fleet:build: remove build warnings on go1.5 #1412

Merged
merged 1 commit into from Jan 27, 2016

Conversation

tixxdz
Copy link
Contributor

@tixxdz tixxdz commented Jan 27, 2016

When building fleet with go1.5 we got the following warning:
Building fleetd...
link: warning: option -X github.com/coreos/fleet/version.Version v0.10.1-117-g7b2a547 may not work in future releases; use -X github.com/coreos/fleet/version.Version=v0.10.1-117-g7b2a547
Building fleetctl...
link: warning: option -X github.com/coreos/fleet/version.Version v0.10.1-117-g7b2a547 may not work in future releases; use -X github.com/coreos/fleet/version.Version=v0.10.1-117-g7b2a547

Newer go versions will only support '-X name=value' format, so make our
build system smarter. It will make sure to use the right format for
future builds.

Reference:
golang/go#13498

Patch written by: Eugene Yakubovich
https://github.com/coreos/flannel/blob/master/build

tixxdz added a commit that referenced this pull request Jan 27, 2016
fleet:build: remove build warnings on go1.5
@tixxdz tixxdz merged commit 189dacf into coreos:master Jan 27, 2016
When building fleet with go1.5 we got the following warning:
Building fleetd...
link: warning: option -X github.com/coreos/fleet/version.Version v0.10.1-117-g7b2a547 may not work in future releases; use -X github.com/coreos/fleet/version.Version=v0.10.1-117-g7b2a547
Building fleetctl...
link: warning: option -X github.com/coreos/fleet/version.Version v0.10.1-117-g7b2a547 may not work in future releases; use -X github.com/coreos/fleet/version.Version=v0.10.1-117-g7b2a547

Newer go versions will only support '-X name=value' format, so make our
build system smarter. It will make sure to use the right format for
future builds.

Reference:
golang/go#13498

Patch written by: Eugene Yakubovich
https://github.com/coreos/flannel/blob/master/build
@tixxdz tixxdz deleted the tixxdz/fleet-27-01-2016 branch March 30, 2016 08:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant