net-im/freetalk: update to version 4.1

[jrm]

Summary

net-im/freetalk: update to version 4.1

Differences from submitted patch:

• Add License information.
• Remove the NLS option because --disable/enable-nls had no effect (binary still linked to libintl.so.8). There were also no %%NLS%% lines in pkg-plist.
• Add libgc-threaded.so:devel/boehm-gc-threaded to LIB_DEPENDS.
• Remove GH_COMMIT definition; GH_TAGNAME is defined.
• Add autoreconf, gettext-runtime, localbase, makeinfo, and readline to USES=.
  • With localbase, CPPFLAGS+= and LDFLAGS+= are no longer necessary.
  • With autoreconf, pre-configure target is no longer necessary.
• Remove info-related lines from pkg-plist and set INFO.

Test Plan

portlint -AC: OK
testport: OK (poudriere: 9.3-RELEASE-p48, i386)
testport: OK (poudriere: 9.3-RELEASE-p48, amd64)
testport: OK (poudriere: 10.3-RELEASE-p10, i386)
testport: OK (poudriere: 10.3-RELEASE-p10, amd64)
testport: OK (poudriere: 11.0-RELEASE-p1, i386)
testport: OK (poudriere: 11.0-RELEASE-p1, amd64)

[swills]

Approved

[AMDmi3]

net-im/freetalk/Makefile

22

You probably need gettext here, not gettext-runtime.

28

Please use DISTVERSIONPREFIX=v instead.

[AMDmi3]

net-im/freetalk/Makefile

18

It uses libguile, it should be LIB_DEPENDS

20

Why? It doesn't seem to use curl.

[jrm]

Changes since last diff:

• add DISTVERSIONPREFIX
• remove unnecessary libcurl from libdepends
• remove BUILD/RUN depends
• remove GH_TAGNAME; it's not necessary since upstream rolled a release

[jrm]

net-im/freetalk/Makefile

22

How can I verify this? It builds fine without a build dependency on gettext-tools.

[AMDmi3]

net-im/freetalk/Makefile

22

If the package uses NLS it needs both runtime in almost all cases, so it's safe to assume it just needs gettext. However you're right, in this case only runtime is used.