diff options
| author | Craig Jennings <c@cjennings.net> | 2026-05-10 15:37:36 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-05-10 15:37:36 -0500 |
| commit | 5336d5c43cabd4132a6856a6f803cd2ef85053a6 (patch) | |
| tree | e293184d5d06f9b38d1c72f44be8b73525de6d39 /scripts | |
| parent | 4605a65006adabf334f0d40d9017c87360bf885e (diff) | |
| download | dotemacs-5336d5c43cabd4132a6856a6f803cd2ef85053a6.tar.gz dotemacs-5336d5c43cabd4132a6856a6f803cd2ef85053a6.zip | |
refactor(external-open): extract external-open-lib for shared helpers
Same shared-helpers split-pattern that ai-vterm/vterm-config use
through cj-window-toggle-lib and that calendar-sync uses through
cj-org-text-lib. Pull the two pure dispatch helpers out of the
external-open feature module into a sibling library so consumers
that only need the dispatch don't have to require the whole feature.
New `modules/external-open-lib.el' carries:
- `cj/external-open-command'
- `cj/external-open-launcher-p'
`modules/external-open.el' stays as the feature module: the
`default-open-extensions' defcustom, the `find-file' advice
(`cj/find-file-auto'), and the interactive commands (`cj/xdg-open',
`cj/open-this-file-with'). It now requires external-open-lib for
the dispatch helpers.
Migrate consumers:
- system-utils.el used to require `external-open' for
`cj/external-open-launcher-p' alone -- now requires
`external-open-lib' directly.
- dirvish-config.el calls `cj/external-open-command' from
`cj/dirvish-open-file-manager-here' -- add an explicit
`(require \='external-open-lib)'.
Test files renamed to match the system-lib naming pattern
(test-<library>-<feature>.el):
- test-external-open-command.el -> test-external-open-lib-command.el
- test-external-open-launcher-p.el -> test-external-open-lib-launcher-p.el
No behavior change.
Diffstat (limited to 'scripts')
0 files changed, 0 insertions, 0 deletions
