fix docs

youknowone · youknowone · commit c9c754f54f2b · 2025-11-15T22:08:32.000+09:00
diff --git a/README.md b/README.md
@@ -66,18 +66,11 @@ Welcome to the magnificent Rust Python interpreter
 >>>>>
 ```
 
-// TODO: fixme
-If you'd like to make https requests, you can enable the `ssl` feature, which
-also lets you install the `pip` package manager. Note that on Windows, you may
-need to install OpenSSL, or you can enable the `ssl-vendor` feature instead,
-which compiles OpenSSL for you but requires a C compiler, perl, and `make`.
-OpenSSL version 3 is expected and tested in CI. Older versions may not work.
-
-Once you've installed rustpython with SSL support, you can install pip by
+You can install pip by
 running:
 
 ```bash
-cargo install --git https://github.com/RustPython/RustPython --features ssl
+cargo install --git https://github.com/RustPython/RustPython
 rustpython --install-pip
 ```
 
@@ -89,6 +82,13 @@ conda install rustpython -c conda-forge
 rustpython
 ```
 
+### SSL provider
+
+For HTTPS requests, `ssl-rustls` feature is enabled by default. You can replace it with `ssl-openssl` feature if your environment requires OpenSSL.
+Note that to use OpenSSL on Windows, you may need to install OpenSSL, or you can enable the `ssl-vendor` feature instead,
+which compiles OpenSSL for you but requires a C compiler, perl, and `make`.
+OpenSSL version 3 is expected and tested in CI. Older versions may not work.
+
 ### WASI
 
 You can compile RustPython to a standalone WebAssembly WASI module so it can run anywhere.
diff --git a/src/lib.rs b/src/lib.rs
@@ -142,9 +142,10 @@ __import__("io").TextIOWrapper(
 
 fn install_pip(installer: InstallPipMode, scope: Scope, vm: &VirtualMachine) -> PyResult<()> {
     if cfg!(feature = "ssl") {
-        if cfg!(not(any(feature = "ssl-rustls", feature = "ssl-openssl"))) {
-            // TODO: something
-        }
+        #[cfg(not(any(feature = "ssl-rustls", feature = "ssl-openssl")))]
+        compile_error!(
+            "Feature \"ssl\" is now enabled by either \"ssl-rustls\" or \"ssl-openssl\" to be enabled. Do not manually pass \"ssl\" feature. To enable ssl-openssl, use --no-default-features to disable ssl-rustls"
+        );
     } else {
         return Err(vm.new_exception_msg(
             vm.ctx.exceptions.system_error.to_owned(),
diff --git a/stdlib/src/lib.rs b/stdlib/src/lib.rs
@@ -80,6 +80,8 @@ mod sqlite;
 mod openssl;
 #[cfg(all(not(target_arch = "wasm32"), feature = "ssl-rustls"))]
 mod ssl;
+#[cfg(all(feature = "ssl-openssl", feature = "ssl-rustls"))]
+compile_error!("features \"ssl-openssl\" and \"ssl-rustls\" are mutually exclusive");
 
 #[cfg(all(unix, not(target_os = "redox"), not(target_os = "ios")))]
 mod termios;
