Uploaded image for project: 'Core Server'
  1. Core Server
  2. SERVER-12898

SConscript for enterprise modules requires SCons version 2.1.0 or greater.

    XMLWordPrintableJSON

Details

    • Icon: Improvement Improvement
    • Resolution: Done
    • Icon: Major - P3 Major - P3
    • None
    • None
    • Build
    • None
    • Fully Compatible

    Description

      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)]
      

      Attachments

        Activity

          People

            Unassigned Unassigned
            sverch Shaun Verch
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: