Skip to content

Conversation

@ctz
Copy link
Member

@ctz ctz commented Dec 27, 2025

  • additional commands for printing the configuration file path, and current configuration text.
  • use of xdg crate to have sane behaviour on operating systems meeting XDG Base Directory specification.
  • simplify fetch subcommand interface

@ctz ctz force-pushed the jbp-config-file branch 5 times, most recently from f89047b to 7b9728d Compare December 27, 2025 13:31
@ctz ctz marked this pull request as ready for review December 27, 2025 13:36
@ctz ctz requested a review from djc December 27, 2025 13:36
@ctz ctz force-pushed the jbp-config-file branch 3 times, most recently from 60f117e to 603e8ad Compare December 28, 2025 14:10
@ctz
Copy link
Member Author

ctz commented Dec 29, 2025

Have applied those suggestions, but I still wasn't happy with an aspect of this. Specifically how upki --config-file /not-exist show-config just ignored the specified path and gave the default config. Just fixed that.

Copy link
Member

@djc djc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, that's an important fix!

- additional commands for printing the configuration file path, and
  current configuration text.
- use of `xdg` crate to have sane behaviour on operating systems meeting
  XDG Base Directory specification.
- simplify `fetch` subcommand interface
- polyfill xdg crate on non-linux platforms
@ctz ctz added this pull request to the merge queue Dec 30, 2025
Merged via the queue into main with commit 26d733e Dec 30, 2025
7 checks passed
@ctz ctz deleted the jbp-config-file branch December 30, 2025 19:28
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.

3 participants