remote dev: Always upload binary in development mode (#19953)

Release Notes:

- N/A
This commit is contained in:
Thorsten Ball 2024-10-30 13:41:28 +01:00 committed by GitHub
parent f6cd97f6fd
commit 0ba40bdfb8
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -1736,13 +1736,19 @@ impl SshRemoteConnection {
}
}
if self.is_binary_in_use(dst_path).await? {
log::info!("server binary is opened by another process. not updating");
delegate.set_status(
Some("Skipping update of remote development server, since it's still in use"),
cx,
);
return Ok(());
if cfg!(not(debug_assertions)) {
// When we're not in dev mode, we don't want to switch out the binary if it's
// still open.
// In dev mode, that's fine, since we often kill Zed processes with Ctrl-C and want
// to still replace the binary.
if self.is_binary_in_use(dst_path).await? {
log::info!("server binary is opened by another process. not updating");
delegate.set_status(
Some("Skipping update of remote development server, since it's still in use"),
cx,
);
return Ok(());
}
}
let upload_binary_over_ssh = self.socket.connection_options.upload_binary_over_ssh;