SConscript for enterprise modules requires SCons version 2.1.0 or greater.

XMLWordPrintableJSON

    • Type: Improvement
    • Resolution: Done
    • Priority: Major - P3
    • None
    • Affects Version/s: None
    • Component/s: Build
    • None
    • Fully Compatible
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      This causes the configure step to fail:

      scons: Configure: Checking for C++ header file net-snmp/net-snmp-config.h...
      .scons/Linux/nohost/sconf_temp/conftest_21.cpp <-
        |
        |#include "net-snmp/net-snmp-config.h"
        |
        |
      g++ -o .scons/Linux/nohost/sconf_temp/conftest_21.o -c -Wnon-virtual-dtor -Woverloaded-virtual -fPIC -fno-strict-aliasing -ggdb -pthread -Wall -Wsign-compare -Wno-unknown-pragmas -Winvalid-pch -Werror -pipe -O3 -Wno-unused-function -Wno-deprecated-declarations -fno-builtin-memcmp -DBOOST_ALL_NO_LIB -D_SCONS -DMONGO_EXPOSE_MACROS -DSUPPORT_UTF8 -DMONGO_OPTIMIZED_BUILD -D_FILE_OFFSET_BITS=64 -DMONGO_SSL -DMONGO_HAVE___THREAD -DMONGO_HAVE_HEADER_UNISTD_H -DMONGO_HAVE_EXECINFO_BACKTRACE "-D{'MONGO_ENTERPRISE_VERSION': 1}" -Ibuild/linux2/cpppath__usr_include_/ssl/third_party/snappy -Ibuild/linux2/cpppath__usr_include_/ssl/third_party/libstemmer_c/include -Ibuild/linux2/cpppath__usr_include_/ssl/third_party/s2 -Ibuild/linux2/cpppath__usr_include_/ssl/third_party/boost -Ibuild/linux2/cpppath__usr_include_/ssl/third_party/pcre-8.30 -Ibuild/linux2/cpppath__usr_include_/ssl -Ibuild/linux2/cpppath__usr_include_/ssl/mongo -I/usr/include .scons/Linux/nohost/sconf_temp/conftest_21.cpp
      <command-line>: error: macro names must be identifiers
      scons: Configure: no
      

      The problem is at this line: https://github.com/10gen/mongo-enterprise-modules/blob/master/build.py#L6

      It should be changed to:

      env.Append(CPPDEFINES=[(MONGO_ENTERPRISE_VERSION, 1)]
      

            Assignee:
            Unassigned
            Reporter:
            Shaun Verch (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved: