Uploaded image for project: 'MariaDB Server'
  1. MariaDB Server
  2. MDEV-4154

LevelDB storage engine build hardcodes path to LevelDB include files and libraries

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:

      Description

      Builds of the LevelDB branch fail for me because /home/psergey is used in two places:
      storage/leveldb/CMakeLists.txt:INCLUDE_DIRECTORIES(AFTER /home/psergey/leveldb/leveldb-1.7.0/include)
      storage/leveldb/CMakeLists.txt:MYSQL_ADD_PLUGIN(leveldb_se $

      {LEVELDB_SOURCES}

      STORAGE_ENGINE STATIC_ONLY MANDATORY LINK_LIBRARIES /home/psergey/leveldb/leveldb-1.7.0/libleveldb.so)

      The error output is:

      cd /home/mcallaghan/b/mysql-5.6-leveldb/storage/leveldb && /usr/bin/c++ -DHAVE_CONFIG_H -Wall -Wno-unused-parameter -O3 -g -fabi-version=2 -fno-omit-frame-pointer -fno-strict-aliasing -DDBUG_OFF -I/home/mcallaghan/b/mysql-5.6-leveldb/include -I/home/psergey/leveldb/leveldb-1.7.0/include -I/home/mcallaghan/b/mysql-5.6-leveldb/sql -I/home/mcallaghan/b/mysql-5.6-leveldb/regex -I/home/mcallaghan/b/mysql-5.6-leveldb/extra/yassl/include -I/home/mcallaghan/b/mysql-5.6-leveldb/extra/yassl/taocrypt/include -I/home/mcallaghan/b/mysql-5.6-leveldb/zlib -o CMakeFiles/leveldb_se.dir/ha_leveldb.cc.o -c /home/mcallaghan/b/mysql-5.6-leveldb/storage/leveldb/ha_leveldb.cc
      In file included from /home/mcallaghan/b/mysql-5.6-leveldb/storage/leveldb/ha_leveldb.cc:22:0:
      /home/mcallaghan/b/mysql-5.6-leveldb/storage/leveldb/ha_leveldb.h:26:24: fatal error: leveldb/db.h: No such file or directory

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              elenst Elena Stepanova added a comment -

              Right, so far we've just been modifying the paths locally according to our local environment.

              Show
              elenst Elena Stepanova added a comment - Right, so far we've just been modifying the paths locally according to our local environment.
              Hide
              jani Jani Tolonen added a comment -

              Fixed, see MDEV-4351 for details.

              Show
              jani Jani Tolonen added a comment - Fixed, see MDEV-4351 for details.

                People

                • Assignee:
                  psergey Sergei Petrunia
                  Reporter:
                  mdcallag Mark Callaghan
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: