Require iconv for mbstring

As the mbstring polyfill using the iconv functions a lot, if not in all calls being done, code without iconv using this polyfill will break. As this repo also provides a polyfill for iconv I don't think this will be a breaking change.
symfony · Nov 22, 2024 · 01e07e8 · 01e07e8
1 parent a5ae132
commit 01e07e8
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/Mbstring/composer.json b/src/Mbstring/composer.json
@@ -16,7 +16,8 @@
         }
     ],
     "require": {
-        "php": ">=7.2"
+        "php": ">=7.2",
+        "ext-iconv"
     },
     "provide": {
         "ext-mbstring": "*"