Conversation
|
Downside is that build times are more than doubled (~8m -> ~20m), but there's not really any way around it with cross-compiling, more work == more time needed, and this already found small issues when building for 32 bits. This new job is the last one, so most issues should fail a job early. |
.travis.yml
Outdated
| # compile assets via webpack and build those into bindata_assetfs.go file | ||
| - make bindata_assetfs.go | ||
| # and now compile using bakelite for all target archs | ||
| - go get -u github.com/terinjokes/bakelite |
There was a problem hiding this comment.
Although I don't expect to have breaking changes, you may prefer to fetch a release instead.
There was a problem hiding this comment.
Last time I checked it was not possible with go get and github, only packages hosted on gopkg.in support getting a specific release, is that no longer true?
There was a problem hiding this comment.
You can use dep's required. You can also fetch the release tarball from GitHub, extract it, and run it from there.
There was a problem hiding this comment.
release tarball it is
7231955 to
7a61e75
Compare
9b6c122 to
a4f4007
Compare
Solves #167 while keeping all code in
.travis.yml, there's only one new make target (show-version) but it's just a helper so we don't need to duplicate the code for getting version string.