Bug 147658 - audio/mpc 0.17 still requires libintl.so.8 after gettext update
Summary: audio/mpc 0.17 still requires libintl.so.8 after gettext update
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: Ade Lovett
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-06-07 17:50 UTC by Tobias Rehbein
Modified: 2010-09-13 17:34 UTC (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Tobias Rehbein 2010-06-07 17:50:00 UTC 
	
        After updating devel/gettext (following the procedure described in UPDATING) audio/mpc 0.17 still
        requires libintl.so.8 .

How-To-Repeat: 	
        After updating devel/gettext using the procedure described in UPDATING:

        > mpc
        /libexec/ld-elf.so.1: Shared object "libintl.so.8" not found, required by "mpc"
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2010-06-07 17:50:12 UTC 
Maintainer of audio/mpc,

Please note that PR ports/147658 has just been submitted.

If it contains a patch for an upgrade, an enhancement or a bug fix
you agree on, reply to this email stating that you approve the patch
and a committer will take care of it.

The full text of the PR can be found at:
    http://www.freebsd.org/cgi/query-pr.cgi?pr=ports/147658

-- 
Edwin Groothuis via the GNATS Auto Assign Tool
edwin@FreeBSD.org
Comment 2 Edwin Groothuis freebsd_committer freebsd_triage 2010-06-07 17:50:14 UTC 
State Changed
From-To: open->feedback

Awaiting maintainers feedback (via the GNATS Auto Assign Tool)
Comment 3 Tobias Rehbein 2010-06-25 22:28:16 UTC 
I digged a little deeper. It seems audio/mpc links itself against libintl if
WITH_ICONV is set and devel/gettext is present on the system.

How to handle this I dont know. Perhaps it's better to close this PR and handle
this case manually ;)

The best solution I came up was something like this (untested) snippet:

        .if defined(WITH_ICONF)
        .if exists(/usr/local/lib/libintl.so)
        USE_GETTEXT=yes
        .endif
        .endif

Regards Tobias
Comment 4 Tobias Rehbein 2010-06-26 09:38:37 UTC 
Am Fri, Jun 25, 2010 at 11:28:16PM +0200 schrieb Tobias Rehbein:
> The best solution I came up was something like this (untested) snippet:
> 
>         .if defined(WITH_ICONF)
>         .if exists(/usr/local/lib/libintl.so)
>         USE_GETTEXT=yes
>         .endif
>         .endif
> 


It seems it was too late when I wrote this mail yesterday. The snippet should
read (of course):

         .if defined(WITH_ICONV)
         .if exists(${LOCALBASE}/lib/libintl.so)
         USE_GETTEXT=yes
         .endif
         .endif

-- 
Tobias Rehbein

PGP key:         4F2AE314
    server:      keys.gnupg.net
    fingerprint: ECDA F300 1B6E 9B87 8524  8663 E8B6 3138 4F2A E314
Comment 5 Philip M. Gollucci freebsd_committer freebsd_triage 2010-09-13 02:28:54 UTC 
State Changed
From-To: feedback->open

to ade
Comment 6 Philip M. Gollucci freebsd_committer freebsd_triage 2010-09-13 02:28:54 UTC 
Responsible Changed
From-To: freebsd-ports-bugs->ade

to ade
Comment 7 avl 2010-09-13 02:54:52 UTC 
Hi!

 I've fixed this problem with the update to 0.19 version:

$ ldd /usr/local/bin/mpc
/usr/local/bin/mpc:
	libiconv.so.3 => /usr/local/lib/libiconv.so.3 (0x28084000)
	libmpdclient.so.2 => /usr/local/lib/libmpdclient.so.2 (0x2817a000)
	libc.so.7 => /lib/libc.so.7 (0x28188000)


-- 
Best regards,
Alexander
Comment 8 Ade Lovett freebsd_committer freebsd_triage 2010-09-13 17:34:12 UTC 
State Changed
From-To: open->closed

Closed at submitters request.