Skip to content

Conversation

MichaelBarnett
Copy link
Collaborator

Patches commec setup to correctly unpack user defined paths containing ~, terminal vars, and use the correctly validated path during the rest of the setup process.

Changes

  • commec setup now correctly validates user input (~) for path to download commec databases into.
  • Includes feedback to the user if a ~name/ user is specifically invalid, this is to catch ~commec-dbs/ instead of ~/commec-dbs/ typos as well as username typos.
  • commec setup now exits gracefully when stopped with keyboard interrupts (Ctrl+C for example).

Bug fixes

  • User expansion on a path now works correctly (~/downloadlocation/commec-dbs/ will expand to the user root, instead of creating a ~ folder in the current directory)
  • The correctly validated path is now carried through to be used when the folders are actually created, instead of only during the validation step.

Michael Barnett added 2 commits July 31, 2025 12:34
…ownload location, including user tilde expansion, and terminal variables. Also added additional graceful exit when the user exits prematurely using Ctrl C keyboard interrupt.
@MichaelBarnett MichaelBarnett requested a review from alexanian July 31, 2025 00:48
@MichaelBarnett MichaelBarnett merged commit 4aeefb7 into develop Aug 4, 2025
1 check passed
@MichaelBarnett MichaelBarnett deleted the setup-filepath-expansion branch August 4, 2025 08:05
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