• Yonghong Song's avatar
    fix lua compilation issue with latest clang/llvm 7.0 · eb88b35a
    Yonghong Song authored
    Commit 46ebd9ef ("fix compilation with latest clang/llvm 7.0")
    tried to fix a compilation introduced with latest clang change.
    It did not fix the lua compilation issue since unfortunately
    my bcc environment did not have lua setup properly for that commit.
    
    This commit intends to fix for lua compilation issue as well.
    The intermediate static library bcc-lua-static is removed and
    now bcc-lua-static is just a list of static libraries used
    for final linking to produce bcc-lua. This way, we do not
    need the special linker flags like -Wl,--whole-archive and
    -Wl,--no-whole-archive. The bcc-static and bcc-shared libraries
    did not change since they will be installed and may be used
    by the application.
    
    Tested on FC27 based system, with the workaround in issue #1685,
    all tests passed.
    
    Fixes: 46ebd9ef ("fix compilation with latest clang/llvm 7.0")
    Signed-off-by: default avatarYonghong Song <yhs@fb.com>
    eb88b35a
CMakeLists.txt 4.34 KB