Page MenuHomeFreeBSD

net-im/freetalk: update to version 4.1
ClosedPublic

Authored by jrm on Oct 26 2016, 2:56 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Oct 7, 5:37 AM
Unknown Object (File)
Thu, Oct 3, 2:17 AM
Unknown Object (File)
Sun, Sep 22, 7:37 PM
Unknown Object (File)
Sun, Sep 8, 10:23 PM
Unknown Object (File)
Sep 7 2024, 10:10 PM
Unknown Object (File)
Aug 30 2024, 4:51 AM
Unknown Object (File)
Aug 19 2024, 2:24 AM
Unknown Object (File)
Aug 17 2024, 6:14 PM
Subscribers

Details

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)

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jrm retitled this revision from to net-im/freetalk: update to version 4.1.
jrm updated this object.
jrm edited the test plan for this revision. (Show Details)
jrm added reviewers: swills, AMDmi3.
swills edited edge metadata.

Approved

This revision is now accepted and ready to land.Oct 26 2016, 12:22 PM
AMDmi3 requested changes to this revision.Oct 26 2016, 12:50 PM
AMDmi3 edited edge metadata.
AMDmi3 added inline comments.
net-im/freetalk/Makefile
22 ↗(On Diff #21685)

You probably need gettext here, not gettext-runtime.

28 ↗(On Diff #21685)

Please use DISTVERSIONPREFIX=v instead.

This revision now requires changes to proceed.Oct 26 2016, 12:50 PM
net-im/freetalk/Makefile
18 ↗(On Diff #21685)

It uses libguile, it should be LIB_DEPENDS

20 ↗(On Diff #21685)

Why? It doesn't seem to use curl.

jrm edited edge metadata.

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 marked 3 inline comments as done.Oct 26 2016, 2:06 PM
jrm added inline comments.
net-im/freetalk/Makefile
22 ↗(On Diff #21685)

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

AMDmi3 edited edge metadata.
AMDmi3 added inline comments.
net-im/freetalk/Makefile
22 ↗(On Diff #21685)

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.

This revision is now accepted and ready to land.Oct 26 2016, 3:05 PM
This revision was automatically updated to reflect the committed changes.