If I do not request 4.0.0, 3.7.0 is reinstalled without panic on my machine 🤯

I can't figure out how to write this as a single iterator operation because it's weird working with nested results.

Do you mean, not collecting the entries here at all?

We must collect them to sort them afaik. I just couldn't filter to directories here.

Yeah. You would need to filter prior to the collect above.

Yeah I tried that haha it's awkward because of the nested Result types, it's beyond my understanding of iterators in Rust.

cc @snowsignal maybe you have a suggestion

@zanieb Here's one way you could do it:

let mut entries: Vec<_> = site_packages
                        .filter(|read_dir| match read_dir {
                            Ok(entry) => entry
                                .file_type()
                                .map(|file_type| file_type.is_dir())
                                .unwrap_or_default(),
                            Err(_) => true,
                        })
                        .collect::<std::io::Result<_>>()?;

Also, if you tweak the code to collect into a BTreeSet<PathBuf>, the sorting will happen as part of collect() and you don't have to call sort_by_key afterwards.

Here's how you could do that:

                    let paths: BTreeSet<_> = site_packages
                        .filter_map(|read_dir| match read_dir {
                            Ok(entry) => {
                                entry.file_type().ok()?.is_dir().then_some(Ok(entry.path()))
                            }
                            Err(err) => Some(Err(err)),
                        })
                        .collect::<Result<_>>()?;

Cool thank you!

This is the call we'd move into the iterator chain above

Before sorting the site-packages, this preferred anyio==3.7.0 on macOS and anyio==4.0.0 on Ubuntu