From 9b07adfe82bf7fd045ba4cf643e05e63e6a01d18 Mon Sep 17 00:00:00 2001
From: CaptainCarpensir <amcarp@amazon.com>
Date: Thu, 22 Feb 2024 15:14:11 -0800
Subject: [PATCH] return early, enable job package --diff

---
 internal/pkg/cli/env_package.go | 1 +
 internal/pkg/cli/job_package.go | 2 ++
 internal/pkg/cli/svc_package.go | 1 +
 3 files changed, 4 insertions(+)

diff --git a/internal/pkg/cli/env_package.go b/internal/pkg/cli/env_package.go
index 69b824b3357..7092811a8a5 100644
--- a/internal/pkg/cli/env_package.go
+++ b/internal/pkg/cli/env_package.go
@@ -223,6 +223,7 @@ func (o *packageEnvOpts) Execute() error {
 				parentErr: err,
 			}
 		}
+		return nil
 	}
 	addonsTemplate, err := packager.AddonsTemplate()
 	if err != nil {
diff --git a/internal/pkg/cli/job_package.go b/internal/pkg/cli/job_package.go
index 238c9ecacc6..db1473aeac3 100644
--- a/internal/pkg/cli/job_package.go
+++ b/internal/pkg/cli/job_package.go
@@ -87,6 +87,7 @@ func newPackageJobOpts(vars packageJobVars) (*packageJobOpts, error) {
 				outputDir:          o.outputDir,
 				uploadAssets:       o.uploadAssets,
 				allowWkldDowngrade: o.allowWkldDowngrade,
+				showDiff:           o.showDiff,
 			},
 			runner:            o.runner,
 			ws:                ws,
@@ -96,6 +97,7 @@ func newPackageJobOpts(vars packageJobVars) (*packageJobOpts, error) {
 			newInterpolator:   newManifestInterpolator,
 			paramsWriter:      discardFile{},
 			addonsWriter:      discardFile{},
+			diffWriter:        os.Stdout,
 			fs:                fs,
 			sessProvider:      sessProvider,
 			newStackGenerator: newWorkloadStackGenerator,
diff --git a/internal/pkg/cli/svc_package.go b/internal/pkg/cli/svc_package.go
index 5b918a4873d..dd653483f32 100644
--- a/internal/pkg/cli/svc_package.go
+++ b/internal/pkg/cli/svc_package.go
@@ -233,6 +233,7 @@ func (o *packageSvcOpts) Execute() error {
 				parentErr: err,
 			}
 		}
+		return nil
 	}
 	if err := o.writeAndClose(o.templateWriter, stack.template); err != nil {
 		return err