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

go: buildGoPackage: 1.8 -> 1.9 #29173

Merged
merged 5 commits into from
Sep 28, 2017
Merged

go: buildGoPackage: 1.8 -> 1.9 #29173

merged 5 commits into from
Sep 28, 2017

Conversation

orivej
Copy link
Contributor

@orivej orivej commented Sep 10, 2017

Motivation for this change
  • gcsfuse 0.23.0 has become compatible with go1.9
  • syncthing012: go1.9 requires that -tags are space separated
  • envoy was broken before this change
  • go: fix tests and impurity #29489
Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • Linux
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

@mention-bot
Copy link

@orivej, thanks for your PR! By analyzing the history of the files in this pull request, we identified @MostAwesomeDude, @kamilchm and @wkennington to be potential reviewers.

@orivej
Copy link
Contributor Author

orivej commented Sep 10, 2017

CC @cstrahan, @wkennington.

@orivej
Copy link
Contributor Author

orivej commented Sep 10, 2017

I've finished testing with nox-review wip on Linux. Now I'd wait for golang/go#21825.

@orivej
Copy link
Contributor Author

orivej commented Sep 28, 2017

@globin This rebuilds 245 derivations with Go 1.9. I think that this is not too much and may be merged to master. Is that OK?
acbuild altcoins.go-ethereum altcoins.go-ethereum-classic aptly assh awless azure-vhd-utils boringssl bosun browserpass c14 caddy cadvisor camlistore cayley certstrap cfssl cide cloud-print-connector cni cockroachdb codesearch compile-daemon confd consul consul-alerts consul-template container-linux-config-transpiler containerd convoy coredns coyim ctop curl_unix_socket deis deisctl delve dex-oidc dgraph direnv docker docker-distribution docker-gc docker-machine docker-machine-kvm docker-proxy docker_17_06 doctl drive drone ecs-agent elvish envconsul etcd exercism filebeat filegive flannel fleet fscrypt-experimental fsql fzf gawp gcsfuse gdrive ghq git-lfs git-lfs1 gitAndTools.git-annex-remote-b2 gitAndTools.git-annex-remote-rclone gitAndTools.hub gitaly gitlab-runner gitlab-runner_1_11 gitlab-shell gitlab-workhorse glide glock go go-bindata go-bindata-assetfs go-dependency-manager go-md2man go-mtpfs go-pup go-repo-root go-sct go-upower-notify go2nix goa gocode gocryptfs godef godep gogs goimports golint gopherclient gosu gotags gotools gotty govers gox grafana gx gx-go heapster heartbeat hecate hologram hound httplab hugo i3cat influxdb interlock ipfs jd jid jmespath journalbeat jp kapacitor kbfs keybase keybase-go kops kt kube-aws kubernetes kubernetes-helm leaps lf linuxkit loccount logstash-forwarder lxd madonctl mailhog manul marathonctl matterbridge matterircd mesos-dns metricbeat minify minikube minio minio-client mirrorbits mm mmake moby mongodb-tools mop mqtt-bench mynewt-newt ngrok-1 nomad nsq oauth2_proxy oh openshift pachyderm packer packet packetbeat panamax_api peco phraseapp-client pipework platinum-searcher pond pprof prom2json prometheus prometheus-alertmanager prometheus-bind-exporter prometheus-blackbox-exporter prometheus-collectd-exporter prometheus-consul-exporter prometheus-fritzbox-exporter prometheus-haproxy-exporter prometheus-json-exporter prometheus-mesos-exporter prometheus-mysqld-exporter prometheus-nginx-exporter prometheus-node-exporter prometheus-openvpn-exporter prometheus-pushgateway prometheus-snmp-exporter prometheus-statsd-bridge prometheus-statsd-exporter prometheus-unifi-exporter prometheus-varnish-exporter python27Packages.syncthing-gtk qsyncthingtray rancher-compose rancher-compose_0_10 rclone restic rkt runc s3gof3r scaleway-cli serfdom shfmt sift skopeo skydns sops ssm-agent styx syncthing syncthing-inotify syncthing-tray syncthing012 syncthing013 systemd-journal2gelf telegraf terminal-parrot terraform terraform-inventory terraform_0_10 terraform_0_10-full terraform_0_8 terraform_0_8_5 terragrunt terragrunt_0_11_1 terragrunt_0_9_8 textql tmsu traefik ua uchiwa vault vimPlugins.YouCompleteMe vndr vultr wego wuzz xmpp-client xurls ycmd

@globin
Copy link
Member

globin commented Sep 28, 2017

Master is fine for me anyway, but please don't cherry-pick to the release branch.

@orivej orivej merged commit 1e0ec42 into NixOS:master Sep 28, 2017
@orivej orivej deleted the golang branch September 28, 2017 17:13
@copumpkin
Copy link
Member

It seems a little odd to me (to the point of causing me to second guess myself for the past 15 minutes) that the release branch has go = go_1_9, but buildGoPackage = buildGo18Package.

@orivej
Copy link
Contributor Author

orivej commented Nov 17, 2017

Do you have a preference about what to have in release, go = go_1_8 or buildGoPackage = buildGo19Package? I don't expect that one option is more difficult to maintain than another.

@copumpkin
Copy link
Member

Slight preference towards 1.9 because Terraform wants 1.9 and we reverted the commit that upgraded to the latest Terraform because buildGoPackage was on 1.8. But if we go with 1.8 I'd think it should probably be consistent across the two. In general it probably makes sense for the two attributes to automatically sync up (so buildGoPackage would be expressed in terms of go and thus automatically follow the same version)

@orivej orivej mentioned this pull request Nov 24, 2017
8 tasks
orivej added a commit that referenced this pull request Nov 25, 2017
See #29173, #32003 for the reasons to update buildGoPackage in the release.

(cherry picked from commit 1e0ec42)
@orivej
Copy link
Contributor Author

orivej commented Nov 26, 2017

Released in 87bfe2c.

@copumpkin
Copy link
Member

Thanks!

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.

4 participants