From 0066c408f1e7b02697fb494abe1a6f6f43e215ca Mon Sep 17 00:00:00 2001 From: NTD Date: Tue, 23 Jan 2018 17:16:11 -0500 Subject: [PATCH] Preprocessor warnings should not be fatal errors --- python/mozbuild/mozbuild/jar.py | 2 +- python/mozbuild/mozbuild/preprocessor.py | 17 +++++------------ 2 files changed, 6 insertions(+), 13 deletions(-) diff --git a/python/mozbuild/mozbuild/jar.py b/python/mozbuild/mozbuild/jar.py index d40751b69b..98333482b9 100644 --- a/python/mozbuild/mozbuild/jar.py +++ b/python/mozbuild/mozbuild/jar.py @@ -460,7 +460,7 @@ class JarMaker(object): pp.setMarker('%') pp.out = outf pp.do_include(inf) - pp.failUnused(realsrc) + pp.warnUnused(realsrc) outf.close() inf.close() return diff --git a/python/mozbuild/mozbuild/preprocessor.py b/python/mozbuild/mozbuild/preprocessor.py index e8aac7057c..2e7eed1ca6 100644 --- a/python/mozbuild/mozbuild/preprocessor.py +++ b/python/mozbuild/mozbuild/preprocessor.py @@ -316,20 +316,13 @@ class Preprocessor: if defines: self.context.update(defines) - def failUnused(self, file): + def warnUnused(self, file): msg = None if self.actionLevel == 0 and not self.silenceMissingDirectiveWarnings: - msg = 'no preprocessor directives found' + sys.stderr.write('{0}: WARNING: no preprocessor directives found\n'.format(file)) elif self.actionLevel == 1: - msg = 'no useful preprocessor directives found' - if msg: - class Fake(object): pass - fake = Fake() - fake.context = { - 'FILE': file, - 'LINE': None, - } - raise Preprocessor.Error(fake, msg, None) + sys.stderr.write('{0}: WARNING: no useful preprocessor directives found\n'.format(file)) + pass def setMarker(self, aMarker): """ @@ -385,7 +378,7 @@ class Preprocessor: self.out = output self.do_include(input, False) - self.failUnused(input.name) + self.warnUnused(input.name) if depfile: mk = Makefile()