Bug 6649

Summary:	configure assumes gcc
Product:	XCB	Reporter:	Alan Coopersmith <alan.coopersmith>
Component:	Library	Assignee:	Jamey Sharp <jamey>
Status:	RESOLVED FIXED	QA Contact:
Severity:	minor
Priority:	medium
Version:	unspecified
Hardware:	All
OS:	Solaris
Whiteboard:
i915 platform:		i915 features:
Attachments:	Only set CWARNFLAGS to gnu flags if $GCC is set. Otherwise check for Sun compiler and use it's enhanced warning flag.

Description Alan Coopersmith 2006-04-19 12:34:19 UTC

Building the current sources from git on Solaris fails if you attempt to use the
Sun compilers because configure.ac hardcodes gcc flags.   It's easy to check for
gcc using the $GCC variable set by AC_PROG_CC, so this is trivial to fix.
(Patch coming momentarily.)

Comment 1 Alan Coopersmith 2006-04-19 12:37:27 UTC

Created attachment 5370 [details] [review]
Only set CWARNFLAGS to gnu flags if $GCC is set.   Otherwise check for Sun compiler and use it's enhanced warning flag.

Comment 2 Jamey Sharp 2006-04-20 14:43:02 UTC

Thanks, Alan! I've been wanting to get these sorts of portability fixes. I've
committed and pushed your patch.

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.