Skip to content

Make `rustic-format-start-process' work over tramp.#522

Open
sp1ff wants to merge 1 commit intobrotzeit:masterfrom
sp1ff:format-start-process-broken-on-remote
Open

Make `rustic-format-start-process' work over tramp.#522
sp1ff wants to merge 1 commit intobrotzeit:masterfrom
sp1ff:format-start-process-broken-on-remote

Conversation

@sp1ff
Copy link

@sp1ff sp1ff commented Jul 30, 2023

rustic-rustfmt-bin selects the rustfmt binary to run depending on whether the current buffer is local (in which case rustic-rustfmt-bin will be used) or remote (rustic-rustfmt-bin-remote).

rustic-format-start-process invokes rustic-rustfmt-bin inside a with-current-buffer form that makes the error buffer current, which will always seem local, regardless of whether the buffer being formatted is local or not

This patch invokes rustic-rustfmt-bin as one of the let bindings so that the correct binary will be selected.

`rustic-rustfmt-bin' selects the rustfmt binary to run depending on
whether the current buffer is local (in which case
`rustic-rustfmt-bin' will be used) or remote
(`rustic-rustfmt-bin-remote').

`rustic-format-start-process' invokes `rustic-rustfmt-bin' inside a
`with-current-buffer' form that makes the error buffer current, which
will always seem local, regardless of whether the buffer being
formatted is local or not

This patch invokes `rustic-rustfmt-bin' as one of the let bindings
so that the correct binary will be selected.
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.

1 participant