Remove autoupdate flag in package-builder (#465)

Remove autoupdate flag in package-builder. It's usage is implied by `update_channel`, no point to both.

cmd/package-builder/package-builder.go

@@ -84,15 +84,10 @@ func runMake(args []string) error {
 			env.Bool("INSECURE_TRANSPORT", false),
 			"whether or not the launcher packages should invoke the launcher's --insecure_transport flag",
 		)
-		flAutoupdate = flagset.Bool(
-			"autoupdate",
-			env.Bool("AUTOUPDATE", false),
-			"whether or not the launcher packages should invoke the launcher's --autoupdate flag",
-		)
 		flUpdateChannel = flagset.String(
 			"update_channel",
 			env.String("UPDATE_CHANNEL", ""),
-			"the value that should be used when invoking the launcher's --update_channel flag",
+			"the value that should be used when invoking the launcher's --update_channel flag. Autoupdates will be disabled unless this is specified",
 		)
 		flControlHostname = flagset.String(
 			"control_hostname",
@@ -197,7 +192,6 @@ func runMake(args []string) error {
 		Transport:         *flTransport,
 		Insecure:          *flInsecure,
 		InsecureTransport: *flInsecureTransport,
-		Autoupdate:        *flAutoupdate,
 		UpdateChannel:     *flUpdateChannel,
 		InitialRunner:     *flInitialRunner,
 		ControlHostname:   *flControlHostname,

pkg/packaging/packaging.go

@@ -36,7 +36,6 @@ type PackageOptions struct {
 	Transport         string
 	Insecure          bool
 	InsecureTransport bool
-	Autoupdate        bool
 	UpdateChannel     string
 	InitialRunner     bool
 	ControlHostname   string
@@ -117,7 +116,7 @@ func (p *PackageOptions) Build(ctx context.Context, packageWriter io.Writer, tar
 		launcherBoolFlags = append(launcherBoolFlags, "control")
 	}
 
-	if p.Autoupdate && p.UpdateChannel != "" {
+	if p.UpdateChannel != "" {
 		launcherBoolFlags = append(launcherBoolFlags, "autoupdate")
 		launcherMapFlags["update_channel"] = p.UpdateChannel
 	}