Bug 379887

Summary:	net-p2p/transmission-2.41: should DEPEND on dev-util/intltool
Product:	Gentoo Linux	Reporter:	nzqr <candysnell>
Component:	Current packages	Assignee:	Gentoo net-p2p team <net-p2p>
Status:	RESOLVED FIXED
Severity:	normal	CC:	barzog, pva
Priority:	Normal
Version:	unspecified
Hardware:	All
OS:	Linux
Whiteboard:
Package list:		Runtime testing required:	---
Attachments:	build.log config.log config log build log emerge --info

Description nzqr 2011-08-19 17:41:23 UTC

net-p2p/transmission-2.33-r1 fails to build with USE="-nls" and no dev-util/intltool installed, so it should depend on intltool unconditionally.

Reproducible: Always

Comment 1 nzqr 2011-08-19 17:41:53 UTC

Created attachment 283929 [details]
build.log

Comment 2 Maxim Koltsov (RETIRED) gentoo-dev

2011-08-19 22:14:17 UTC

Why do you think it is just missing intltool?

./configure: line 18604: syntax error near unexpected token `0.35.0,no-xml'
./configure: line 18604: `    IT_PROG_INTLTOOL(0.35.0,no-xml)'

Comment 3 nzqr 2011-08-20 12:55:56 UTC

>Why do you think it is just missing intltool?

Because it builds just fine with installed intltool.

Comment 4 nzqr 2011-08-20 13:19:43 UTC

Created attachment 284011 [details]
config.log

Comment 5 Peter Volkov (RETIRED) gentoo-dev

2011-10-19 11:00:25 UTC

Please, try to reproduce with transmission-2.41. At least I'm unable to reproduce this failure there.

Comment 6 nzqr 2011-10-21 18:19:27 UTC

Created attachment 290475 [details]
config log

Comment 7 nzqr 2011-10-21 18:19:48 UTC

Created attachment 290477 [details]
build log

Comment 8 nzqr 2011-10-21 18:20:52 UTC

Created attachment 290479 [details]
emerge --info

Comment 9 nzqr 2011-10-21 18:22:14 UTC

transmission-2.41 fails too. My emerge --info output is in attachments.

Comment 10 Tomáš Chvátal (RETIRED) gentoo-dev

2012-01-02 21:40:43 UTC

Confirmed in latest.

The macro is not present on clean system when you eautoreconf -> the gettext intltool is harddep if you eautoreconf, but can be kept in nls usecondition when no eautoreconf is done.

Fixed in cvs and comment added to explain why the dep is there.