posix.pathconf_names only for linux

youknowone · youknowone · commit 955347e42601 · 2023-02-21T19:45:04.000+09:00
diff --git a/extra_tests/snippets/stdlib_os.py b/extra_tests/snippets/stdlib_os.py
@@ -509,9 +509,8 @@ def __exit__(self, exc_type, exc_val, exc_tb):
         assert_raises(TypeError, os.system, arg)
 
 # Testing for os.pathconf_names
-if not sys.platform.startswith("win"):
+if sys.platform.startswith('linux'):
     assert len(os.pathconf_names) > 0
     assert 'PC_NAME_MAX' in os.pathconf_names
     for option,index in os.pathconf_names.items():
         assert os.pathconf('/', index) == os.pathconf('/', option)
-
diff --git a/vm/src/stdlib/posix.rs b/vm/src/stdlib/posix.rs
@@ -50,9 +50,7 @@ pub mod module {
         ffi::{CStr, CString},
         fs, io,
         os::unix::{ffi as ffi_ext, io::RawFd},
-        str::FromStr,
     };
-    use strum::VariantNames;
     use strum_macros::{EnumString, EnumVariantNames};
 
     #[pyattr]
@@ -1883,8 +1881,12 @@ pub mod module {
         pathconf(PathOrFd::Fd(fd), name, vm)
     }
 
+    // TODO: this is expected to be run on macOS as a unix, but somehow not.
+    #[cfg(target_os = "linux")]
     #[pyattr]
     fn pathconf_names(vm: &VirtualMachine) -> PyDictRef {
+        use std::str::FromStr;
+        use strum::VariantNames;
         let pathname = vm.ctx.new_dict();
         for variant in PathconfVar::VARIANTS {
             // get the name of variant as a string to use as the dictionary key
