Skip to content

Commit

Permalink
Fixes #44 version printing (#51)
Browse files Browse the repository at this point in the history
  • Loading branch information
mescam committed Dec 28, 2020
1 parent 2f45fb5 commit c7c4753
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
REVISION := $(shell git rev-parse HEAD)
CHANGES := $(shell test -n "$$(git status --porcelain)" && echo '+CHANGES' || true)

LDFLAGS := -X main.Revision=$(REVISION)$(CHANGES)
LDFLAGS := -X main.Revision=$(REVISION)$(CHANGES) -X main.Version=$(TRAVIS_TAG)

build: build-linux build-osx

Expand Down
4 changes: 2 additions & 2 deletions cmd/assume-role-arn/main.go
Original file line number Diff line number Diff line change
Expand Up @@ -212,7 +212,7 @@ func printExport(val *AWSCreds) {
}

func setEnv(val *AWSCreds) {
os.Setenv(prefixEnvVar("AWS_ACCESS_KEY_ID"),val.AccessKeyID)
os.Setenv(prefixEnvVar("AWS_ACCESS_KEY_ID"), val.AccessKeyID)
os.Setenv(prefixEnvVar("AWS_SECRET_ACCESS_KEY"), val.AccessKey)
os.Setenv(prefixEnvVar("AWS_SESSION_TOKEN"), val.SessionToken)
}
Expand Down Expand Up @@ -246,7 +246,7 @@ func runCommand(args []string) error {

func main() {
if version {
fmt.Fprintf(os.Stderr, "assume-role-arn v%s\n", formattedVersion())
fmt.Fprintf(os.Stderr, "assume-role-arn %s\n", formattedVersion())
os.Exit(0)
}

Expand Down
2 changes: 1 addition & 1 deletion cmd/assume-role-arn/version.go
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ var Revision string

// Version contains the main version number that is being
// run at the moment.
const Version = "0.3.4"
var Version string

func formattedVersion() string {
var versionString strings.Builder
Expand Down

0 comments on commit c7c4753

Please sign in to comment.