Skip to content

Commit 7e9a020

Browse files
committed
fix: remove import mcpp.platform from package_fetcher.cppm
The added import changed the module dependency graph fingerprint, invalidating BMI cache on macOS CI and exposing a pre-existing Xcode 16.4 SDK incompatibility during std module precompilation. Use #if defined(_WIN32) instead of if constexpr for the USERPROFILE check to avoid changing the module import set.
1 parent ee85314 commit 7e9a020

1 file changed

Lines changed: 3 additions & 4 deletions

File tree

src/pm/package_fetcher.cppm

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,6 @@ export module mcpp.pm.package_fetcher;
1414

1515
import std;
1616
import mcpp.config;
17-
import mcpp.platform;
1817
import mcpp.pm.compat;
1918
import mcpp.pm.index_spec;
2019
import mcpp.xlings;
@@ -614,9 +613,9 @@ Fetcher::resolve_xpkg_path(std::string_view target,
614613
// reason (xlings subprocess XLINGS_HOME propagation is unreliable).
615614
if (!std::filesystem::exists(verdir)) {
616615
const char* xhome = nullptr;
617-
if constexpr (mcpp::platform::is_windows) {
618-
xhome = std::getenv("USERPROFILE");
619-
}
616+
#if defined(_WIN32)
617+
xhome = std::getenv("USERPROFILE");
618+
#endif
620619
if (!xhome) xhome = std::getenv("HOME");
621620
if (xhome) {
622621
// xlings stores xpkgs at <home>/.xlings/data/xpkgs/ or

0 commit comments

Comments
 (0)