-
-
Notifications
You must be signed in to change notification settings - Fork 17
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Bug]: Cannot use space in package version #447
Labels
Comments
This was referenced Feb 3, 2024
I'm seeing this issue with smartmontools.smartmontools which has Running: >komac update --identifier smartmontools.smartmontools ^
-v "7.4 2023-08-01 r5530 (sf-7.4-1)" ^
--urls "https://sourceforge.net/projects/smartmontools/files/smartmontools/7.4/smartmontools-7.4-1.win32-setup.exe/download" ^
--submit returned:
Sigcheck reports >sigcheck smartmontools-7.4-1.win32-setup.exe
Sigcheck v2.90 - File version and signature viewer
Copyright (C) 2004-2022 Mark Russinovich
Sysinternals - www.sysinternals.com
smartmontools-7.4-1.win32-setup.exe:
Verified: Unsigned
Link date: 9:58 PM 8/14/2022
Publisher: n/a
Company: www.smartmontools.org
Description: SMART Monitoring Tools
Product: smartmontools
Prod version: 7.4.0.5530
File version: 7.4.0.5530
MachineType: 32-bit So I tried >komac update -i smartmontools.smartmontools ^
-v "7.4.0.5530" ^
--urls "https://sourceforge.net/projects/smartmontools/files/smartmontools/7.4/smartmontools-7.4-1.win32-setup.exe/download" ^
--submit but received:
|
Use |
I reported this in fosskers/rs-versions#23. There'll hopefully be a fix for this soon. |
> komac --version
komac 2.0.4
> komac new -i smartmontools.smartmontools ^
-v "7.4.0.5530" ^
--urls "https://sourceforge.net/projects/smartmontools/files/smartmontools/7.4/smartmontools-7.4-1.win32-setup.exe/download" ^
--submit retured
I also tried > komac new -i smartmontools.smartmontools ^
-v "7.4.0.5530" ^
--urls "https://sourceforge.net/projects/smartmontools/files/smartmontools/7.4/smartmontools-7.4-1.win32-setup.exe/download#/smartmontools-7.4-1.win32-setup.exe" ^
--submit but got the same error. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Is there an existing issue for this?
What happened?
7zip-zstd manifest uses versioning in the format of "7-Zip-Version ZS vZstandard version R(n)".
The latest release on GitHub would be "22.01 ZS v1.5.5 R3" in that format, but Komac doesn't allow it presumably due to the space characters.
Possible relevant issue that could cause another issue microsoft/winget-create#498
Command
komac update -i mcmilk.7zip-zstd -v "22.01 ZS v1.5.5 R3" -u https://github.com/mcmilk/7-Zip-zstd/releases/download/v22.01-v1.5.5-R3/7z22.01-zstd-x64.exe https://github.com/mcmilk/7-Zip-zstd/releases/download/v22.01-v1.5.5-R3/7z22.01-zstd-x32.exe https://github.com/mcmilk/7-Zip-zstd/releases/download/v22.01-v1.5.5-R3/7z22.01-zstd-arm64.exe https://github.com/mcmilk/7-Zip-zstd/releases/download/v22.01-v1.5.5-R3/7z22.01-zstd-arm.exe
Output
The text was updated successfully, but these errors were encountered: