Bug 221415

Summary:	x11-base/xorg-server: probably needs flex build-time dependency
Product:	Gentoo Linux	Reporter:	Diego Elio Pettenò (RETIRED) <flameeyes>
Component:	New packages	Assignee:	Gentoo X packagers <x11>
Status:	RESOLVED FIXED
Severity:	normal	Keywords:	Inclusion
Priority:	High
Version:	unspecified
Hardware:	All
OS:	Linux
Whiteboard:
Package list:		Runtime testing required:	---
Bug Depends on:
Bug Blocks:	221311
Attachments:	xorg-server-flex.patch

Description Diego Elio Pettenò (RETIRED) gentoo-dev

2008-05-11 13:19:00 UTC

The configure of your package checks for flex, even though your package might not contain .l files that flex uses to generate parsers. Your ebuild does not list a dependency on flex as per http://tinderbox.dev.gentoo.org/misc/dindex/sys-devel/flex .

Either disable the flex test during configure to avoid false positive, or add the dependency upon flex if it's needed by your package.

Thanks in advance,
Diego

Comment 1 Donnie Berkholz (RETIRED) gentoo-dev

2008-05-12 08:27:50 UTC

Diego,

This bug would be much more helpful if it contained a suggestion for how to prevent autotools from checking for flex.

Comment 2 Arfrever Frehtes Taifersar Arahesis (RETIRED) gentoo-dev

2008-06-15 00:06:57 UTC

configure.ac contains AC_PROG_LEX.

x11-base/xorg-server contains these files:
hw/dmx/config/scanner.l
hw/xwin/winprefslex.l

Please add sys-devel/flex to DEPEND.

Comment 3 Arfrever Frehtes Taifersar Arahesis (RETIRED) gentoo-dev

2008-06-15 00:24:44 UTC

(In reply to comment #2)
> Please add sys-devel/flex to DEPEND.

I meant:
dmx? ( sys-devel/flex )

And a patch to configure.ac.

Comment 4 Arfrever Frehtes Taifersar Arahesis (RETIRED) gentoo-dev

2008-06-15 00:46:20 UTC

Created attachment 156813 [details, diff]
xorg-server-flex.patch

Comment 5 Rémi Cardona (RETIRED) gentoo-dev

2009-02-27 18:59:50 UTC

That's actually been fixed for a while now. Closing.

Thanks