Created attachment 149473 [details]
mplayer uses iconv, but had no USES clause for it. I fixed this, and brought patch for -liconv closer to what https://www.freebsd.org/doc/en/books/porters-handbook/using-iconv.html recommends. (removed some hackish parts from configure script)
Please note that dependencies still use /usr/local/lib/libiconv.so so this one will probably be actually used anyway.
Additionally, ignored patch file files/patch-configure.orig should probably be deleted from the repository.
Auto-assigned to maintainer riggs@FreeBSD.org
A commit references this bug:
Date: Sun Nov 16 17:20:33 UTC 2014
New revision: 372644
Correct iconv handling
While on it:
- Remove @dirrm(try) from pkg-plists
Submitted by: email@example.com
Reviewed by: riggs
Committed a modified version of your patch. Thanks!
(The changes in patch-configure were not necessary.)