php / php/php-src

Test failures with musl libc

Offen
#22,413 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bug Category: Tests Status: Verified
Vorherrschende Sprache
C
Sterne
40.4k
Forks
8.2k
Ø Merge
2 T. 13 Std.
Gemergte PRs (30 T.)
96

Beschreibung

Description

Follow-up to https://github.com/php/php-src/issues/13696. Musl is partially tested on the CI, but the Alpine job installs gnu-libiconv-dev which hides the following errors. With the latest php-8.5.7 release:

=====================================================================
FAILED TEST SUMMARY
---------------------------------------------------------------------
Bug #48147 (iconv with //IGNORE cuts the string) [ext/iconv/tests/bug48147.phpt]
Bug #52211 (iconv() returns part of string on error) [ext/iconv/tests/bug52211.phpt]
Bug #76249 (stream filter convert.iconv leads to infinite loop on invalid sequence) [ext/iconv/tests/bug76249.phpt]
EUC-JP to ISO-2022-JP [ext/iconv/tests/eucjp2iso2022jp.phpt]
iconv_mime_encode() [ext/iconv/tests/iconv_mime_encode.phpt]

I have an old PR at https://github.com/php/php-src/pull/16840 whose details I've mostly forgotten. I can start by cherry-picking the easy parts of that.

PHP Version
PHP 8.5.7
Operating System

Gentoo linux

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne mit dem Alpine-CI-Job und den fehlschlagenden Tests in ext/iconv/tests/, insbesondere bug48147.phpt, bug52211.phpt, bug76249.phpt, eucjp2iso2022jp.phpt und iconv_mime_encode.phpt. Vergleiche den Job mit und ohne gnu-libiconv-dev und prüfe anschließend die damit verbundene Arbeit in PR 16840. Die Aufgabe ist abgeschlossen, wenn die aufgeführten Tests gegen musl bestehen, ohne dass dieses Paket die Fehler maskiert.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
c, linux, php
Bereich
operating-systems, testing-qa
Issue-Typ
Bug
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.