Bug 46344

Summary:	Hardcoded WError -- no cookie
Product:	Ytstenut	Reporter:	Robert Staudinger <robert.staudinger>
Component:	plugins	Assignee:	Robert Staudinger <robert.staudinger>
Status:	NEW ---	QA Contact:
Severity:	normal
Priority:	medium	CC:	olli.salli
Version:	unspecified
Hardware:	Other
OS:	All
Whiteboard:
i915 platform:		i915 features:
Attachments:	0001-build-Do-not-hardcode-WError.patch

Description Robert Staudinger 2012-02-20 06:53:05 UTC

I'm worried that this bit of code will make the buildbot might choke more often than strictly needed because of hardcoded -WError:

CFLAGS="$CFLAGS -Werror -Wall -Wextra -Wno-missing-field-initializers \
	-Wno-unused-parameter -Wdeclaration-after-statement -Wshadow \
	-Wmissing-prototypes -Wmissing-declarations"

Comment 1 Robert Staudinger 2012-02-20 07:05:23 UTC

Created attachment 57321 [details] [review]
0001-build-Do-not-hardcode-WError.patch

commit a85ec0a7c278d1902541fc69be3f92da57277edb
Author: Rob Staudinger <robsta@linux.intel.com>
Date:   Mon Feb 20 16:01:22 2012 +0100

    build: Do not hardcode -WError
    
    Add -WError and other zealous warnings only when configured with
    --enable-debug, so to prevent the buildbot from raising a red alert
    as a response to venial sins.
    
    Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=46344

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.