Skip to content

fix(cli): run update installer silently - #1352

Open
MAN$I VERMA (mansiverma897993) wants to merge 1 commit into
qualcomm:mainfrom
mansiverma897993:fix/cli-silent-update
Open

fix(cli): run update installer silently#1352
MAN$I VERMA (mansiverma897993) wants to merge 1 commit into
qualcomm:mainfrom
mansiverma897993:fix/cli-silent-update

Conversation

@mansiverma897993

@mansiverma897993 MAN$I VERMA (mansiverma897993) commented Aug 16, 2026

Copy link
Copy Markdown
Contributor

Summary

Run the Windows installer silently from geniex update so upgrades work in headless/CI environments without pop-up windows. The setup script uninstalls the previous version with no UI as well.

Closes #1344

Test plan

  • gofmt clean on changed Go files
  • TestUpdateInstallerCmd passes (verifies /VERYSILENT /SUPPRESSMSGBOXES /NORESTART args)
  • TestCompareVersion passes (33 cases)
  • bazelisk test //cli/... (linux-arm64 + windows-arm64) - CI in progress

Signed-off-by: mansiverma897993 <vmansi756@gmail.com>
@mengshengwu

Mengsheng Wu (mengshengwu) commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Hi MAN$I VERMA (@mansiverma897993) This looks great! Thanks for your work .
It would be better if you test it e2e. Just build local and try real update and comment your steps and results. Also re-edit the pr description plz

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We rely real update test instead of text matching, So I think this tests code is unnecessary

Comment thread cli/cmd/geniex/update.go
return nil
}

func updateInstallerCmd(dst string) *exec.Cmd {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only once call, so just inline it.

Comment thread cli/cmd/geniex/update.go
}

func updateInstallerCmd(dst string) *exec.Cmd {
return exec.Command(dst, "/VERYSILENT", "/SUPPRESSMSGBOXES", "/NORESTART")
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CLI: geniex update silently without pop window

3 participants