Bug 14171

Summary:	sxpm/Makefile.am:21: SED was already defined
Product:	xorg	Reporter:	Peter Dyballa <Peter_Dyballa>
Component:	Lib/Xpm	Assignee:	Xorg Project Team <xorg-team>
Status:	RESOLVED FIXED	QA Contact:	Xorg Project Team <xorg-team>
Severity:	normal
Priority:	medium
Version:	7.2 (2007.02)
Hardware:	Other
OS:	All
Whiteboard:
i915 platform:		i915 features:

Description Peter Dyballa 2008-01-21 05:42:26 UTC

Building libXpm I get this complaint:

	autoreconf: running: /sw/bin/autoconf-2.61
	autoreconf: running: /sw/bin/autoheader-2.61
	autoreconf: running: automake --add-missing --copy --no-force
	sxpm/Makefile.am:21: SED was already defined in condition TRUE, which includes condition BUILD_SXPM ...
	configure.ac:13: ... `SED' previously defined here
	autoreconf: Leaving directory `.'
	configure: WARNING: you should use --build, --host, --target

Probably it's just needed to remove the line

	SED = sed

from lib/libXpm/sxpm/sxpm/Makefile.am since lib/libXpm/configure.ac has on lines #12-14:

	# Checks for programs.
	AC_PROG_LIBTOOL
	AC_PROG_CC

meaning that libtool is bringing in this definition?

Comment 1 Alan Coopersmith 2008-01-24 14:27:47 UTC

Fixed in git master commit 3f7624048aa6064c69e2320a70fb7fc89e0bb7ef

Comment 2 Peter Dyballa 2008-01-25 01:57:43 UTC

Looks good now!

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.