Use Ranges::from_iter over Ranges::iter_mut

[konstin]

Use astral-sh/pubgrub#34 (pubgrub-rs/pubgrub#273) for safer ranges modification that can't break pubgrub invariants.

In the process, I removed the &mut references in favor of a direct immutable-to-immutable mapping.

The tests currently fail, i think it's because we're producing empty segments where Ranges expects them to be non-empty; See pubgrub-rs/pubgrub#273 for more discussion on invariants.

[zanieb]

Should we change

uv/crates/pep508-rs/src/marker/algebra.rs

Lines 638 to 645 in 0335efd

	let mut range = Range::empty();
	// TODO(zanieb): We need to make sure this is performant, repeated unions like this do not
	// seem efficient.
	for version in versions {
	range = range.union(&Range::singleton(version.clone()));
	}

and

uv/crates/uv-resolver/src/pubgrub/report.rs

Lines 1177 to 1180 in e1a8beb

	// Note this repeated-union construction is not particularly efficient, but there's not
	// better API exposed by PubGrub. Since we're just generating an error message, it's
	// probably okay, but we should investigate a better upstream API.
	new_range = new_range.union(&Range::from_range_bounds((lower, upper)));

too?

[konstin]

Do those match the invariants from pubgrub-rs/pubgrub#273, or do those need Ranges:from_unsorted from pubgrub-rs/pubgrub#273?

[zanieb]

I don't know! :)