Revert "Test case runner: fix passing of spaces in NANOPBFLAGS"
This reverts commit b945553bb3bf5098c2cb6260f113c53158937442.
--nanopb_opt isn't compatible with protoc 3.0, which is still being supported and covered by CI tests.
diff --git a/tests/site_scons/site_tools/nanopb.py b/tests/site_scons/site_tools/nanopb.py
index 6b82ce4..4abf02a 100644
--- a/tests/site_scons/site_tools/nanopb.py
+++ b/tests/site_scons/site_tools/nanopb.py
@@ -144,12 +144,15 @@
include_dirs += ' -I' + esc(d)
# when generating .pb.cpp sources, instead of pb.h generate .pb.hpp headers
- nanopb_flags = env['NANOPBFLAGS']
source_extension = os.path.splitext(str(target[0]))[1]
- if source_extension == '.cpp':
- nanopb_flags += ',--source-extension=.cpp,--header-extension=.hpp'
+ header_extension = '.h' + source_extension[2:]
+ nanopb_flags = env['NANOPBFLAGS']
+ if nanopb_flags:
+ nanopb_flags = '--source-extension=%s,--header-extension=%s,%s:.' % (source_extension, header_extension, nanopb_flags)
+ else:
+ nanopb_flags = '--source-extension=%s,--header-extension=%s:.' % (source_extension, header_extension)
- return SCons.Action.CommandAction('$PROTOC $PROTOCFLAGS %s --nanopb_out=. "--nanopb_opt=%s" %s' % (include_dirs, nanopb_flags, srcfile),
+ return SCons.Action.CommandAction('$PROTOC $PROTOCFLAGS %s --nanopb_out=%s %s' % (include_dirs, nanopb_flags, srcfile),
chdir = prefix)
def _nanopb_proto_emitter(target, source, env):