Deprecate the WinPTY backend

CHANGELOG.md

@@ -13,6 +13,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 - The snapcraft.yaml file has been removed
 - Updated `setab`/`setaf` capabilities in `alacritty-direct` to use colons
 - WinPTY is now enabled only when targeting MSVC
+- Deprecated the WinPTY backend feature, disabling it by default
 
 ### Added
 

alacritty/Cargo.toml

@@ -56,7 +56,7 @@ winapi = { version = "0.3.7", features = ["impl-default", "wincon"]}
 embed-resource = "1.3"
 
 [features]
-default = ["wayland", "x11", "winpty"]
+default = ["wayland", "x11"]
 x11 = ["copypasta/x11", "glutin/x11", "x11-dl"]
 wayland = ["copypasta/wayland", "glutin/wayland", "wayland-client"]
 winpty = ["alacritty_terminal/winpty"]

alacritty/src/config/mod.rs

@@ -312,6 +312,15 @@ fn print_deprecation_warnings(config: &Config) {
             "Config dynamic_title is deprecated; please use window.dynamic_title instead",
         )
     }
+
+    #[cfg(all(windows, not(feature = "winpty")))]
+    if config.winpty_backend {
+        warn!(
+            target: LOG_TARGET_CONFIG,
+            "Config winpty_backend is deprecated and requires a compilation flag; it should be \
+             removed from the config",
+        )
+    }
 }
 
 #[cfg(test)]

alacritty_terminal/Cargo.toml

@@ -41,7 +41,7 @@ mio-anonymous-pipes = "0.1"
 winpty = { version = "0.2.0", optional = true }
 
 [features]
-default = ["winpty"]
+default = []
 bench = []
 
 [dev-dependencies]

alacritty_terminal/src/config/mod.rs

@@ -55,11 +55,6 @@ pub struct Config<T> {
     #[serde(default, deserialize_with = "failure_default")]
     pub cursor: Cursor,
 
-    /// Use WinPTY backend even if ConPTY is available.
-    #[cfg(windows)]
-    #[serde(default, deserialize_with = "failure_default")]
-    pub winpty_backend: bool,
-
     /// Shell startup directory.
     #[serde(default, deserialize_with = "option_explicit_none")]
     pub working_directory: Option<PathBuf>,
@@ -72,6 +67,11 @@ pub struct Config<T> {
     #[serde(skip)]
     pub hold: bool,
 
+    // TODO: DEPRECATED
+    #[cfg(windows)]
+    #[serde(default, deserialize_with = "failure_default")]
+    pub winpty_backend: bool,
+
     // TODO: DEPRECATED
     #[serde(default, deserialize_with = "failure_default")]
     pub visual_bell: Option<BellConfig>,