Convert apphost to C - #126367

Merged

elinor-fung merged 85 commits into

mainfrom

copilot/rewrite-apphost-in-c

Jul 24, 2026

Commits on Mar 27, 2026

Rewrite apphost in C for non-Windows platforms

Copilot
and
jkoritzinsky
authored
efa09f3
Fix compilation errors and add test_only_getenv support

Copilot
and
jkoritzinsky
authored
f22e1b6
Address code review feedback: remove dead code, add errno.h

Copilot
and
jkoritzinsky
authored
0186dcf

Commits on Mar 28, 2026

Deduplicate hostmisc C code and use CMake target for containers

Copilot
and
jkoritzinsky
authored
44f43e4
Use C bundle_marker implementation everywhere, remove C++ version

Copilot
and
jkoritzinsky
authored
098cfb9

Commits on Mar 30, 2026

Remove trace.cpp, use C trace implementation everywhere with minipal mutex

Copilot
and
jkoritzinsky
authored
c604d5d
Fix: Remove apphost_pal.h dependency from apphost_trace.c

Copilot
and
jkoritzinsky
authored
e14238f
Fix trace_error_v va_list UB and trace_warning duplication

Copilot
and
jkoritzinsky
authored
f639e5f

Commits on Mar 31, 2026

Rewrite hostmisc C++ to wrap C implementations

Copilot
and
jkoritzinsky
authored
92d7f1b
Address code review: fix arch param bug, use explicit type

Copilot
and
jkoritzinsky
authored
35b7572
Rename hostmisc C files: drop apphost_ prefix, headers get _c suffix

Copilot
and
elinor-fung
committed
7b1a73c
Add pal_char_t typedef to pal_c.h, use in function signatures

Copilot
and
elinor-fung
committed
41bd5dc
Dynamically allocate path buffers in apphost C code

Copilot
and
jkoritzinsky
authored
2b8c310
Fix app_path buffer to be large enough for pal_fullpath resolution

Copilot
and
jkoritzinsky
authored
93c6344
Make pal.h include pal_c.h (removing duplicates) and trace.c use pal functions

Copilot
and
elinor-fung
committed
e665819

Commits on Apr 1, 2026

Use pal functions in utils.c/fx_ver.c, rename pal.c to pal.unix.c, redirect utils.cpp to C

Copilot
and
elinor-fung
authored
331ddac

Commits on Apr 2, 2026

Commits on Apr 3, 2026

Commits on Apr 4, 2026

Move corehost.cpp to dotnet/dotnet.cpp, remove FEATURE_LIBHOST/CURHOST_EXE guards

Copilot
and
jkoritzinsky
authored
60daf24
Add extern "C" guards to _c.h headers; combine fx_ver_c.h with C++ equivalent

Copilot
and
jkoritzinsky
authored
f5df70d
Rename fx_ver_c.h to fx_ver.h; remove thin fxr/fx_ver.h wrapper

Copilot
and
jkoritzinsky
authored
73d8e39

Commits on Apr 6, 2026

Commits on Apr 8, 2026

Commits on Apr 17, 2026

Fix Windows build error: revert _X macro to single-step L ## s

Copilot
and
jkoritzinsky
authored
3bef172

Commits on Apr 20, 2026

Fix _X macro for C11 mode: use two-step pattern in C, single-step in C++

Copilot
and
jkoritzinsky
authored
b14e833

Commits on Apr 24, 2026

Commits on Apr 27, 2026

Consolidate test-only marker into a single embed in utils.c

Copilot
and
jkoritzinsky
authored
b3c62e3

Commits on Apr 28, 2026

Commits on Apr 29, 2026

Fix trace-to-file and directory-exists: handle TRACEFILE directory paths and use stat() for pal_directory_exists

Copilot
and
jkoritzinsky
authored
7b6e47c

Commits on Apr 30, 2026

Fix Windows pal_directory_exists and registry config location

jkoritzinsky
and
Copilot
committed
d7dd614

Commits on May 1, 2026

Fix missing DOTNET_ROOT(x86) WOW64 fallback in C implementation

jkoritzinsky
and
Copilot
committed
90e0e0c

Commits on May 11, 2026

Potential fix for pull request finding

elinor-fung
and
Copilot
authored
986cfef
Address code review comments: fix error handling, move bundle_marker, rename apphost_main.c

Copilot
and
elinor-fung
authored
f8771f0

Commits on May 12, 2026

Eliminate dn-containers dependency from fxr_resolver.c

Copilot
and
elinor-fung
authored
3cdefcc
Revert dn-containers infrastructure changes

Copilot
and
elinor-fung
authored
40cf8a1
Address code review comments

Copilot
and
elinor-fung
authored
2b0114b

Commits on May 13, 2026

Fix Windows build: add utils.cpp to hostmisc_c

Copilot
and
elinor-fung
authored
f36a122

Commits on Jun 22, 2026

Merge upstream/main into copilot/rewrite-apphost-in-c

elinor-fung
and
Copilot
committed
3bbda0c
Fix double-free of caller-owned app_root in standalone hostfxr_resolver_init

elinor-fung
and
Copilot
committed
e30ac21
Potential fix for pull request finding

elinor-fung
and
Copilot
authored
6073193

Commits on Jun 23, 2026

Implement corehost C PAL/utils helpers in C instead of extern "C" C++ shims

elinor-fung
and
Copilot
committed
ffe14b1
Use DIR_SEPARATOR/ALT_DIR_SEPARATOR in pal_is_path_fully_qualified

elinor-fung
and
Copilot
committed
97df417
Delegate C++ pal:: load/symbol/path/string helpers to the C implementations

elinor-fung
and
Copilot
committed
5e5fc17
Inline need_newer_framework_error into dotnet.cpp exe_start

elinor-fung
and
Copilot
committed
3cda8fc
Fix Android singlefilehost link by skipping static createdump init

Copilot
and
elinor-fung
authored
fe00e82
Normalize embedded app path separators on Windows in apphost.c

Copilot
and
elinor-fung
authored
0faaed3
Address remaining host review feedback

Copilot
and
elinor-fung
authored
d66ce58

Commits on Jun 25, 2026

Build the non-Windows apphost as a pure C binary

elinor-fung
and
Copilot
committed
59ca612
Remove unused apphost/static/hostfxr_resolver.cpp

elinor-fung
and
Copilot
committed
aa047e6

Commits on Jun 26, 2026

Update src/native/corehost/apphost/static/apphost_hostfxr_resolver.cpp

elinor-fung
and
am11
authored
22ce92f
Potential fix for pull request finding

elinor-fung
and
Copilot
authored
7ddc9fc
Apply suggestion from @elinor-fung
elinor-fung
authored
da7857f

Commits on Jul 2, 2026

Merge remote-tracking branch 'upstream/main' into copilot/rewrite-apphost-in-c
elinor-fung
committed
48364aa
Clean up apphost CMake: link fxr_resolver target, drop redundant includes

elinor-fung
and
Copilot
committed
6f098ac
Consolidate path checks onto pal_is_path_fully_qualified

elinor-fung
and
Copilot
committed
cd1717d
Gate static createdump init on a CMake-defined flag

elinor-fung
and
Copilot
committed
9fadcf9
Address host review feedback: resolver init and version-description guard

elinor-fung
and
Copilot
committed
7236afa
Inline host version at apphost trace site

elinor-fung
and
Copilot
committed
45bd71b

Commits on Jul 3, 2026

Address apphost/pal review feedback

elinor-fung
and
Copilot
committed
a553b72
Make pal_utf8_to_palstr cross-platform

elinor-fung
and
Copilot
committed
5f29a85

Commits on Jul 6, 2026

Use HOST_VERSION in apphost trace message instead of sccsid

Copilot
and
elinor-fung
authored
bba3837
Include commit hash in apphost trace version message

Copilot
and
elinor-fung
authored
4fb487d
Use REPO_COMMIT_HASH for commit in apphost trace version message

Copilot
and
elinor-fung
authored
e9328c8

Commits on Jul 8, 2026

Inline version macros in apphost trace message

elinor-fung
and
Copilot
committed
34f2da2
Simplify and clean up apphost path handling

elinor-fung
and
Copilot
committed
b49860a
Inline default search location in dotnet resolver

elinor-fung
and
Copilot
committed
bb93b6f
Clean up apphost and static resolver

elinor-fung
and
Copilot
committed
1a1f13e
Tidy embedded-size constant and unused include

elinor-fung
and
Copilot
committed
aefdb1e

Commits on Jul 9, 2026

Return a typed pal_proc_t from pal_get_symbol

elinor-fung
and
Copilot
committed
d30c6f7
Use typed pal_dll_t/pal_proc_t handles in the PAL

elinor-fung
and
Copilot
committed
5491b24

Commits on Jul 17, 2026