579104 – sys-libs/timezone-data on musl - ld: cannot find -lintl

Bug 579104 - sys-libs/timezone-data on musl - ld: cannot find -lintl

Summary: sys-libs/timezone-data on musl - ld: cannot find -lintl

Status:	RESOLVED DUPLICATE of bug 578424

Alias:	None

Product:	Gentoo Linux
Classification:	Unclassified
Component:	[OLD] Unspecified (show other bugs)
Hardware:	All Linux

Importance:	Normal normal (vote)
Assignee:	Gentoo musl team

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:	musl-porting
	Show dependency tree

Reported:	2016-04-05 18:59 UTC by Fabio Scaccabarozzi
Modified:	2016-04-08 10:29 UTC (History)
CC List:	2 users (show)

See Also:
Package list:
Runtime testing required:	---

Attachments
Ebuild patch for timezone-data (timezone-data_do_not_add_lintl_also_for_musl.patch,449 bytes, patch) 2016-04-05 19:01 UTC, Fabio Scaccabarozzi	Details \| Diff
View All 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 Fabio Scaccabarozzi 2016-04-05 18:59:57 UTC

When emerging timezone-data with USE=nls for use with musl libc, timezone-data fails to link because we add -lintl to LDLIBS, because of bug #154181. It turns out also musl doesn't need this (see =virtual/libintl-0-r2 RDEPENDS).
We can drop it for musl as well, see attached patch.

Reproducible: Always

Steps to Reproduce:
1. USE="nls" emerge -1 =sys-libs/timezone-data-2015g
2. 
3.
Actual Results:  
Failure due to -lintl LDLIBS

Expected Results:  
Emerge should succeed

Comment 1 Fabio Scaccabarozzi 2016-04-05 19:01:03 UTC

Created attachment 429704 [details, diff]
Ebuild patch for timezone-data

Comment 2 Felix Janda 2016-04-06 10:47:21 UTC

Thanks for reporting!

I've just noticed that this is actually a duplicate of bug 578424
(which is not linked to the musl tracker bug and does not contain musl
in its title).

Comment 3 Fabio Scaccabarozzi 2016-04-06 11:19:48 UTC

Sorry, didn't spot it because I was looking specifically for timezone-data+musl.
I guess this bug should be closed as a duplicate, and the other one should be updated accordingly. Maybe we should also mention bug #154181 and the link to virtual/libintl, in case someone wonders where we picked up that code in the ebuild and why it's ok to fix it this way.
Bug #578424 also has a better patch, which includes IUSE update.

Comment 4 Anthony Basile gentoo-dev

2016-04-08 10:29:33 UTC


*** This bug has been marked as a duplicate of bug 578424 ***