Skip to content

Commit 052eec2

Browse files
committed
fix: add missing import mcpp.platform in package_fetcher.cppm
Also replace raw WIFEXITED/WEXITSTATUS with platform::process::extract_exit_code() in xlings.cppm.
1 parent b8df13e commit 052eec2

2 files changed

Lines changed: 3 additions & 4 deletions

File tree

src/pm/package_fetcher.cppm

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

1515
import std;
1616
import mcpp.config;
17+
import mcpp.platform;
1718
import mcpp.pm.compat;
1819
import mcpp.pm.index_spec;
1920
import mcpp.xlings;

src/xlings.cppm

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -622,10 +622,8 @@ int install_with_progress(const Env& env, std::string_view target,
622622
// Use std::system() directly — do NOT redirect stdin via </dev/null
623623
// because xlings may need stdin for subprocess coordination during
624624
// large package extraction.
625-
int directRc = std::system(directCmd.c_str());
626-
if constexpr (!mcpp::platform::is_windows) {
627-
directRc = WIFEXITED(directRc) ? WEXITSTATUS(directRc) : directRc;
628-
}
625+
int directRc = mcpp::platform::process::extract_exit_code(
626+
std::system(directCmd.c_str()));
629627
if (directRc == 0) return 0;
630628
}
631629

0 commit comments

Comments
 (0)