Done
Details
Details
Assignee
Charles Oliveira
Charles OliveiraReporter
Charles Oliveira
Charles OliveiraOriginal estimate
1w
Time tracking
4d 2h logged6h remaining
Priority
Checklist
Checklist
Sentry
Sentry
Created November 26, 2021 at 2:29 AM
Updated February 2, 2022 at 6:43 AM
Resolved February 2, 2022 at 6:43 AM
The overall verboseness of cpackget is very minimal. Perhaps making
-v
the default verbose option, and adding a new option for quiet working--quiet/-q
would do the trick.Since the installation may take quite long, it would be good to see progress on installing the pack. Is this possible? Example:
Installing ARM.CMSIS.5.7.0 100% |████████████████| (<installed files>/<total number of files>)
Installing packs with packID without the version would print info messages
I: Adding [ARM.CMSIS]
, where it could be more informativeI: Adding [ARM.CMSIS.5.8.0]
including the version, since cpackget would already know it by the time it prints out this message.The help text shows up like
cpackget [flags] cpackget [command]
, perhaps it could readcpackget [command] [flags]
cpackget should be able to handle SIGTERM and other forced termination cases. Just catching the termination intent and forcing cpackget to shutdown should be enough for now.
Running
./cpackget init
generates the error messageE: pack root not found
. It might be better to display a more helpful message. Ex:E: No CMSIS Pack Root directory specified. Either the environment CMSIS_PACK_ROOT needs to be set or the path specified using the command line option -R, --pack-root string
Running
./cpackget.exe pack add ARM.CMSIS -R ./tmp-cmsis-pack-root/
with that pack being pre-installed, the error message is justE: pack already installed
. It might be better to read something likeE: Nothing done. The pack <vendor>.<pack>.<version> is already installed here: <cmsis_pack_root>/<vendor>/<pack>/<version>.
When running
./cpackget.exe index https://www.keil.com/pack/index.pidx -R ./tmp-cmsis-pack-root
, nothing gets displayed, the download progress bar only shows up when the flag-v
is specified. It’d be convenient to display it by default.When re-running the command above, the error message shows like
I: Using pack root: "./tmp-cmsis-pack-root" I: Updating index [https://www.keil.com/pack/index.pidx] E: cannot overwrite original public index.pidx
A better error message would be
E: The file 'index.pidx' already exists, use the flag '-f' to overwrite the existing file.