newsboat-og

• Path: arch/maintained/newsboat-og
• 2026-07-17: both known build failures fixed and published, pkgrel 3 -> 4, all four arches (x86_64, i486, i686, pentium4).
• This is the user’s own fork (andreasbaumann/newsboat-og on GitHub, same author as this project’s maintainer). Fixes for this package are pushed as upstream commits there first, then wired into the PKGBUILD as source=() entries pointing at that commit’s .patch URL – same convention as the pre-existing newsboat-og-curl-proxytype-cast.patch (commit f2b404c3). Always check upstream for a fix before writing a local patch by hand.
• Four patches now applied in prepare(), in this order:
  1. newsboat-og-curl-proxytype-cast.patch (commit f2b404c3c485...) – pre-existing, unrelated to this session.
  2. newsboat-og-json-c-isystem.patch (commit d11e3705b53c...) – config.sh demotes json-c’s -I include to -isystem after config.mk is generated (sed -i ... 's/^(DEFINES\+=)-I(...json-c...)/\1 -isystem \2/'), so json-c’s own header warnings (linkhash.h unused-function) don’t trip this package’s -Werror. This was the originally-reported json-c build problem.
  3. newsboat-og-image-count.patch (commit ae260fcecc78...) – removes genuinely dead code in src/htmlrenderer.cpp: unsigned int image_count was declared, incremented once (line ~264, inside the IMG-tag handling), and never read anywhere – a leftover counter. Modern GCC’s -Werror=unused-but-set-variable turns this into a hard error. This is the failure originally flagged in the 2026-07-07 triage (see “Superseded” below) – interestingly it only showed up on the x86_64 chroot in this session’s rebuild (i486/i686/pentium4 built fine even before this patch was added), so chroots/toolchains differ in how strictly they apply this warning; fix it regardless since the code really is dead.
  4. newsboat-og-json-hpp-3.11.3.patch (commit 1ab71d889a38...) – updates the bundled vendor header 3rd-party/json.hpp (nlohmann JSON, used by src/ttrss_api.cpp/include/ttrss_api.h for the Tiny Tiny RSS API backend – genuinely compiled, see mk/mk.deps) from 3.2.0 to 3.11.3. Large diff (~31k lines, it’s a vendored single-header library) but applied the same way as any other patch. Not actually required to fix a build error in this session (build succeeded on x86_64 with just patches 1-3) – added anyway per explicit maintainer request, likely for currency/security of the vendored dependency rather than a build fix.
• Don’t confuse patch 2 (“json-c”, the C JSON library via pkg-config) with patch 4 (“json.hpp”, the unrelated bundled nlohmann C++ JSON header) – same word, two unrelated dependencies in this codebase.
• Test-built clean across all four arches via scripts/repo_build.sh (0 compile errors on the full patch set). Signed and published all 8 package files (newsboat-og + newsboat-og-debug, x86_64/i486/i686/ pentium4) to the private repo.

Superseded (kept for context)

• Build failure during the 2026-07-07 “build all packages for x86_64” private-repo release pass (see repo_release_logs/maintained_newsboat-og.log).
• Real compile error, not dependency-related: src/htmlrenderer.cpp:91:22: error: variable 'image_count' set but not used [-Werror=unused-but-set-variable=]. Same class as memory/modest.md – a modern GCC’s stricter default warnings-as-errors tripping on old unused-variable code.
• Not fixed as part of that pass.