Github user ottobackwards commented on a diff in the pull request:

    https://github.com/apache/metron/pull/849#discussion_r153328847
  
    --- Diff: metron-deployment/scripts/platform-info.sh ---
    @@ -73,6 +73,30 @@ echo "--"
     echo "npm"
     npm --version
     
    +# C++ compiler
    +echo "--"
    +echo "g++"
    +g++ --version
    +
    +# C++11 compliant compiler
    +echo "--"
    +OBJFILE=/tmp/test
    +CPPFILE=/tmp/test.cpp
    +cat > $CPPFILE <<- EOM
    +#include <iostream>
    +using namespace std;
    +int main() {
    +    cout << "Hello World!" << endl;
    +    return 0;
    +}
    +EOM
    +g++ -std=c++11 $CPPFILE -o $OBJFILE
    +if [ $? -eq 0 ]; then
    +    echo "Compiler is C++11 compliant"
    +else
    +    echo "Warning: Compiler is NOT C++11 compliant"
    +fi
    +rm -f $CPPFILE $OBJFILE
     
    --- End diff --
    
    i agree about it not being worth the trouble.
    
    as far as g++, I would expect us to check (pseudo)
    
    if c++ in path then
        build test program
    else
        echo No Cpp, you are in trouble!
    fi
    
    Maybe I'm mis-reading this, if there is no cpp, it will still try to build 
etc won't it?
    



---

Reply via email to