#58523 closed defect (fixed)
libiconv +cp932fix fails in patch phase
Reported by: | MarcusCalhoun-Lopez (Marcus Calhoun-Lopez) | Owned by: | ryandesign (Ryan Carsten Schmidt) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | |
Keywords: | Cc: | chrstphrchvz (Christopher Chavez) | |
Port: | libiconv |
Description (last modified by ryandesign (Ryan Carsten Schmidt))
The recent update to libiconv, from a pull request, seems to have broken libiconv +cp932fix
.
The patchfile no longer applies.
Change History (7)
comment:1 follow-up: 3 Changed 5 years ago by chrstphrchvz (Christopher Chavez)
comment:2 Changed 5 years ago by ryandesign (Ryan Carsten Schmidt)
Description: | modified (diff) |
---|
comment:3 Changed 5 years ago by ryandesign (Ryan Carsten Schmidt)
Replying to chrstphrchvz:
Since the patch is not in macports-ports, how should this be resolved?
See #36921 for an example of how I solved a similar problem with the suhosin variant/patch in the php port.
comment:4 Changed 5 years ago by MarcusCalhoun-Lopez (Marcus Calhoun-Lopez)
I have opened a pull request to remove the cp932fix variant.
comment:5 Changed 5 years ago by MarcusCalhoun-Lopez (Marcus Calhoun-Lopez)
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
comment:6 Changed 5 years ago by ryandesign (Ryan Carsten Schmidt)
comment:7 Changed 5 years ago by ryandesign (Ryan Carsten Schmidt)
Note: See
TracTickets for help on using
tickets.
I did not try compiling the
+cp932fix
variant, so I did not notice this issue earlier.I believe this is the upstream change in 1.16 that broke the patch: https://git.savannah.gnu.org/cgit/libiconv.git/commit/tests/Makefile.in?id=1badfd5d85cedfe06a2d8262f0ae91b121e86fa5
Since the patch is not in macports-ports, how should this be resolved? The patch is from over 12 years ago ([2f85b61c43/macports-ports]), is it still needed?