From 697e53df8b75470911d3abc006bcd3dedd7ec5ca Mon Sep 17 00:00:00 2001 From: Mario Fetka Date: Mon, 22 Jun 2026 08:09:51 +0200 Subject: [PATCH] build: update warning-cleanup submodule pins --- AI.md | 35 +++++++++++++++++------------------ directory | 2 +- third_party/flaim | 2 +- third_party/matrixssl | 2 +- 4 files changed, 20 insertions(+), 21 deletions(-) diff --git a/AI.md b/AI.md index a2a8687..df7b5e6 100644 --- a/AI.md +++ b/AI.md @@ -46,7 +46,7 @@ unfinished work out of `TODO.md` merely because its architecture is documented. Latest commit after this handoff refresh in an up-to-date tree: -- `nwnss: remove src/nwnss include/nwnss tests/nwnss tree` +- `build: update warning-cleanup submodule pins` Compare `git log -1 --format="%s"` with this subject. If the tree is older, check which documentation/import-boundary commits are missing and replay them @@ -54,27 +54,26 @@ before starting new work. ## Current patch handoff block -Last completed work block: **Remove libnwnss from src/include/tests; prepare for root-level NSS re-import** +Last completed work block: **Clean GCC 15 build warnings in maintained submodules** -The former `src/nwnss/`, `include/nwnss/` and `tests/nwnss/` tree has been -removed. The `mars_nwe::nwnss` CMake target, the `MARS_NWE_BUILD_NWNSS_FUSE_TOOLS` -option and all associated CTest entries are gone. NSS sources will be -re-imported directly under a root-level `nss/` tree in the next step. +Three maintained mars-* submodules were updated and pushed, then the root +gitlinks were advanced with `update-submodules.sh`: -The companion-file rules remain in force for the re-import: +- `directory` / mars-tinyldap: guard local `_GNU_SOURCE` definitions so the root + CMake compile definition does not trigger duplicate macro warnings. +- `third_party/flaim` / mars-flaim: remove obsolete C++ `register` use, format + dynamic printf buffers safely, and make static month string tables const. +- `third_party/matrixssl` / mars-matrixssl: remove misleading fixed-size array + annotations from pointer-style TLS/crypto APIs that confused GCC 15, guard + the `LINUX` platform macro, and fix ECDSA signature length byte packing. -- Imported original NSS files stay as original-like as possible. -- `*Userspace.c` / `*Userspace.h` for libnwfs/Mars/OtherFS/HostFS adapter boundaries. -- `*Fuse.c` / `*Fuse.h` for FUSE/nwnssmount-facing provider callbacks. -- `*Io.c` / `*Io.h` for concrete Linux pread/pwrite block/image I/O. -- `*Runtime.c` / `*Runtime.h` for NSS runtime/cache/workqueue semantics. -- Shared helpers use `Nss*` file/symbol names. -- Do not use global `read`/`write` macros; userspace block I/O goes through `zlssBioIO*` only. +The real GitHub-pinned external submodules (`yyjson`, `termbox2`, `iniparser`) +were not modified. -Build/test expectation: the uploaded `ncpfs-2.2.7.tar.gz` is part of the local -dependency set and can be built when full build targets need `libncp`. Separate -build/link success and offline NSS unit tests from server-dependent integration -CTests. +Verification used the out-of-tree build directory +`/home/mario/mars/mars-nwe-build`: final `cmake --build` log had no +`warning:`/`error:` lines, and `ctest --test-dir /home/mario/mars/mars-nwe-build +-LE networked --output-on-failure` passed all 9 offline tests. ## Next import/use-case direction diff --git a/directory b/directory index 84a6f10..b3c1c9c 160000 --- a/directory +++ b/directory @@ -1 +1 @@ -Subproject commit 84a6f10d459b3e2294158836965db397054452d6 +Subproject commit b3c1c9c8ad05e2534dae4ea978b0558a485a396d diff --git a/third_party/flaim b/third_party/flaim index 3890b78..ecb1807 160000 --- a/third_party/flaim +++ b/third_party/flaim @@ -1 +1 @@ -Subproject commit 3890b78c3fdceba996a27ee7443101c865cd14a7 +Subproject commit ecb1807f1d397458ab3885954dd560f30bf9cd34 diff --git a/third_party/matrixssl b/third_party/matrixssl index 9c36190..26aeee2 160000 --- a/third_party/matrixssl +++ b/third_party/matrixssl @@ -1 +1 @@ -Subproject commit 9c3619078c1929792b6e08e0f7f62d5937d6e355 +Subproject commit 26aeee2e30922be0f8a289bcc3ec3b053080ea9d