Skip to content

Update winget submission workflow include only default msi files#10487

Merged
fdncred merged 2 commits intonushell:mainfrom
hustcer:fix/winget
Sep 24, 2023
Merged

Update winget submission workflow include only default msi files#10487
fdncred merged 2 commits intonushell:mainfrom
hustcer:fix/winget

Conversation

@hustcer
Copy link
Copy Markdown
Contributor

@hustcer hustcer commented Sep 24, 2023

Update winget submission workflow include only default msi files:


The new release workflow after #10457 will add full featured packages like: nu-*-x86_64-windows-msvc-full.msi and nu-*-aarch64-windows-msvc-full.msi (A release preview could be found here: https://github.com/nushell/nightly/releases/tag/nightly-6a2fd91 ) and this change will make sure only the default packages will be included just the same as before to make sure that nothing will be broken.

BTW: Due to lots of changes made recently about the release workflow, I will stand-by for the next release.

@fdncred
Copy link
Copy Markdown
Contributor

fdncred commented Sep 24, 2023

ugh, i hate winget but we probably need to do this. thanks for your offer to stand-by.

@fdncred fdncred merged commit 95a4649 into nushell:main Sep 24, 2023
@hustcer hustcer deleted the fix/winget branch September 24, 2023 12:53
@fdncred
Copy link
Copy Markdown
Contributor

fdncred commented Oct 17, 2023

Winget failed. No clue why. microsoft/winget-pkgs#122956

@fdncred
Copy link
Copy Markdown
Contributor

fdncred commented Oct 17, 2023

This is the entry from the failed installer log.

CAQuietExec:  Error 0xc0000135: Command line returned an error.
CAQuietExec:  Error 0xc0000135: QuietExec Failed
CAQuietExec:  Error 0xc0000135: Failed in ExecCommon method
CustomAction ReplacePathsInWindowsTerminalProfile returned actual error code 1603 (note this may not be 100% accurate if translation happened inside sandbox)
Action ended 14:27:07: InstallFinalize. Return value 3.
Action ended 14:27:07: INSTALL. Return value 3.

Not sure what it means exactly although ReplacePathsInWindowsTerminalProfile is in our wix file.

@hustcer
Copy link
Copy Markdown
Contributor Author

hustcer commented Oct 17, 2023

Sorry that it fails again, I have no idea why it fails currently, as my change works as expected. So weird

@fdncred
Copy link
Copy Markdown
Contributor

fdncred commented Oct 17, 2023

I think the reason it's failing is because of trying to update the nu.json windows terminal fragment.

hardfau1t pushed a commit to hardfau1t/nushell that referenced this pull request Dec 14, 2023
…hell#10487)

### Update winget submission workflow include only default msi files:
---
The new release workflow after
nushell#10457 will add full featured
packages like: `nu-*-x86_64-windows-msvc-full.msi` and
`nu-*-aarch64-windows-msvc-full.msi` (A release preview could be found
here: https://github.com/nushell/nightly/releases/tag/nightly-6a2fd91 )
and this change will make sure only the default packages will be
included just the same as before to make sure that nothing will be
broken.

BTW: Due to lots of changes made recently about the release workflow, I
will stand-by for the next release.
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.

2 participants