Open-CMSIS-Pack / cpackget

Open-CMSIS-Pack Package Installer
Apache License 2.0
24 stars 15 forks source link

This command should result in error `cpackget add Keil::STM32U5xx_DFP@>=3.0.0` #383

Closed ReinhardKeil closed 1 week ago

ReinhardKeil commented 1 week ago

Describe the bug

This command should result in error cpackget add Keil::STM32U5xx_DFP@>=3.0.0

Instead it is silently accepted and no error is issued.

Expected behavior

Issue error, i.e. pack version incorrect format

Environment (please complete the following information):

jkrech commented 1 week ago

IMHO this is not a bug.

a) it is the command shell that interprets the > before the cpackget is even launched. Therefore there is no means for the tool to throw an error. b) if you put the parameter in quotes the command works as expected:

cpackget add "Keil::STM32U5xx_DFP@>=3.0.0"

on windows the quotes are also required for the ^ as it will be removed by the command shell:

cpackget add "Keil::STM32U5xx_DFP@^3.0.0"
ReinhardKeil commented 1 week ago

@jkrech you are correct. I overlooked that > is the pipe command