Merge pull request #16 from fujiwara/fix-options

fix handling for empty global options.

cmd/lambroll/main.go

@@ -83,6 +83,10 @@ func _main() int {
 	}
 
 	command := kingpin.Parse()
+	if command == "version" {
+		fmt.Println("lambroll", Version)
+		return 0
+	}
 
 	filter := &logutils.LevelFilter{
 		Levels:   []logutils.LogLevel{"trace", "debug", "info", "warn", "error"},
@@ -96,10 +100,6 @@ func _main() int {
 		log.Println("[error]", err)
 		return 1
 	}
-	if command == "version" {
-		fmt.Println("lambroll", Version)
-		return 0
-	}
 
 	log.Println("[info] lambroll", Version)
 	switch command {

lambroll.go

@@ -64,19 +64,19 @@ type App struct {
 // New creates an application
 func New(opt *Option) (*App, error) {
 	awsCfg := &aws.Config{}
-	if opt.Region != nil {
+	if opt.Region != nil && *opt.Region != "" {
 		awsCfg.Region = aws.String(*opt.Region)
 	}
 	sessOpt := session.Options{Config: *awsCfg}
 	var profile string
-	if opt.Profile != nil {
+	if opt.Profile != nil && *opt.Profile != "" {
 		sessOpt.Profile = *opt.Profile
 		profile = *opt.Profile
 	}
 	sess := session.Must(session.NewSessionWithOptions(sessOpt))
 
 	loader := config.New()
-	if opt.TFState != nil {
+	if opt.TFState != nil && *opt.TFState != "" {
 		funcs, err := tfstate.Load(*opt.TFState)
 		if err != nil {
 			return nil, err