Skip to content

[depends] update iconv to 1.18#95

Open
oliver wants to merge 1 commit intoxbmc:Piersfrom
oliver:update-iconv-to-1.18
Open

[depends] update iconv to 1.18#95
oliver wants to merge 1 commit intoxbmc:Piersfrom
oliver:update-iconv-to-1.18

Conversation

@oliver
Copy link
Copy Markdown

@oliver oliver commented Apr 2, 2026

The entire depends/common/iconv/ directory was copied from https://github.com/xbmc/inputstream.ffmpegdirect/tree/527a71baa23a81e79b7ee2b1a658e73b54339978/depends/common/iconv

This PR fixes building this addon with GCC 15, e.g. under Debian Testing.

It compiles successfully, but I did not yet test whether the addon actually works.

@kodiai
Copy link
Copy Markdown

kodiai bot commented Apr 2, 2026

✅ Safe to merge

No breaking changes, hashes verified, limited impact scope.

Version Diff

Package Old New Hash Status
iconv 1.18

Changelog Highlights

Changelog unavailable -- check [upstream](upstream repository) manually

Impact Assessment

1 consuming file found in the codebase.

  • src/sacd/sacd_disc.cpp (line 18)

Patch Changes

  • depends/common/iconv/0001-patch-utf8mac.patch (added)
  • depends/common/iconv/0002-src-Makefile.in-darwin.patch (added)

@kodiai
Copy link
Copy Markdown

kodiai bot commented Apr 2, 2026

Review Details
  • Files reviewed: 5
  • Lines changed: +1700 -4
  • Profile: minimal (auto, lines changed: 1704)
  • Author: newcomer (adapted tone)
  • Findings: 0 critical, 0 major, 0 medium, 0 minor
  • Review completed: 2026-04-02T17:28:42.533Z
  • Keyword parsing:
    • focus hints: [DEPENDS]

oliver added a commit to oliver/repo-binary-addons that referenced this pull request Apr 6, 2026
…18" branch)

This is necessary for building the add-on under Debian Testing, until
xbmc/audiodecoder.sacd#95 is merged.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant