Index | Thread | Search

From:
Kirill A. Korinsky <kirill@korins.ky>
Subject:
Re: [IMPROVEMENT] Integrate the app server daemon into devel/codex.
To:
David Uhden Collado <daviduhden@gmail.com>
Cc:
ports@openbsd.org
Date:
Tue, 13 Oct 2026 21:38:28 +0200

Download raw body.

Thread
On Tue, 13 Oct 2026 20:42:00 +0200,
David Uhden Collado <daviduhden@gmail.com> wrote:
> 
> Please find attached a patch proposing proper integration of the Codex
> app-server daemon with the OpenBSD package system.
> 
> The port already includes the daemon functionality, but upstream
> normally expects a complete standalone installation and may try to
> install another Codex executable under ~/.codex/packages. This is
> unsuitable for a package managed by pkg_add.
> 
> The proposed changes address this without disabling the daemon or
> rewriting its lifecycle management.
>

Sorry, you tried to ship way more than only this changes.

You had touched many patches and reformat port's Makefile.

> [2 codex-0.162.1p0.diff <text/x-patch; UTF-8 (base64)>]
> ? devel/codex/files
> ? devel/codex/patches/patch-codex-rs_app-server-daemon_src_backend_pid_rs
> ? devel/codex/patches/patch-codex-rs_app-server-daemon_src_lib_rs
> ? devel/codex/patches/patch-codex-rs_app-server-daemon_src_managed_install_rs
> ? devel/codex/patches/patch-codex-rs_app-server-daemon_src_prepare_install_rs
> ? devel/codex/patches/patch-codex-rs_modcargo-crates_v8-150_4_0_build_config_clang_clang_gni
> ? devel/codex/patches/patch-codex-rs_modcargo-crates_v8-150_4_0_build_detect_host_arch_py
> ? devel/codex/patches/patch-codex-rs_modcargo-crates_v8-150_4_0_third_party_libunwind_src_src_AddressSpace_hpp
> ? devel/codex/patches/patch-codex-rs_modcargo-crates_v8-150_4_0_third_party_libunwind_src_src_EHHeaderParser_hpp
> ? devel/codex/patches/patch-codex-rs_modcargo-crates_v8-150_4_0_third_party_libunwind_src_src_UnwindCursor_hpp
> ? devel/codex/patches/patch-codex-rs_modcargo-crates_v8-150_4_0_v8_gni_v8_gni
> ? devel/codex/patches/patch-codex-rs_modcargo-crates_v8-150_4_0_v8_src_wasm_stacks_cc
> ? devel/codex/patches/patch-codex-rs_tui_src_app_daemon_menu_rs

I thunk you also forgot quite a few patches.

Can you resend your diff to minimal version which adds support of that codex daemon?

-- 
wbr, Kirill