Uploaded image for project: 'Mesos'
  1. Mesos
  2. MESOS-7965

WebUI folder not set in CMake

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Open
    • Major
    • Resolution: Unresolved
    • None
    • None
    • cmake, webui
    • None
    • Any build using CMake.

    Description

      The default directory for the WebUI assets is not correctly set in CMake builds. While a user can work around this via ./src/mesos-master --webui_dir=../src/webui, ideally the default would "just work."

      src/master/flags.cpp
      166:  add(&Flags::webui_dir,
      167:      "webui_dir",
      168:      "Directory path of the webui files/assets",
      169:      PKGDATADIR "/webui");
      199:      "Human readable name for the cluster, displayed in the webui.");
      

      We currently set PKGDATADIR to something not quite right which ends up with the default search path for webui assets not working.

      cmake/CompilationConfigure.cmake
      351:  -DPKGDATADIR="${DATA_INSTALL_PREFIX}")
      
      cmake/CompilationConfigure.cmake
      246:  set(DATA_INSTALL_PREFIX  ${SHARE_INSTALL_PREFIX}/mesos)
      351:  -DPKGDATADIR="${DATA_INSTALL_PREFIX}")
      
      cmake/CompilationConfigure.cmake
      245:  set(SHARE_INSTALL_PREFIX ${CMAKE_INSTALL_PREFIX}/share)
      246:  set(DATA_INSTALL_PREFIX  ${SHARE_INSTALL_PREFIX}/mesos)
      
      cmake/CompilationConfigure.cmake
      321:  set(EXEC_INSTALL_PREFIX     "WARNINGDONOTUSEME")
      322:  set(LIBEXEC_INSTALL_DIR     "WARNINGDONOTUSEME")
      323:  set(PKG_LIBEXEC_INSTALL_DIR "WARNINGDONOTUSEME")
      324:  set(LIB_INSTALL_DIR         "WARNINGDONOTUSEME")
      325:  set(TEST_LIB_EXEC_DIR       "WARNINGDONOTUSEME")
      326:  set(PKG_MODULE_DIR          "WARNINGDONOTUSEME")
      327:  set(S_BIN_DIR               "WARNINGDONOTUSEME")
      

      Attachments

        Activity

          People

            Unassigned Unassigned
            andschwa Andrew Schwartzmeyer
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated: