Project

General

Profile

Actions

Backport #18878

closed

jewel: cmake: should define the install destination

Added by Wei-Chung Cheng about 7 years ago. Updated about 7 years ago.

Status:
Closed
Priority:
Normal
Target version:
-
Release:
jewel
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Actions #1

Updated by Wei-Chung Cheng about 7 years ago

7a602ec292b7e92c994d1718402e35a6c6692a02

will cherry-pick above commit to sole it

Actions #2

Updated by Wei-Chung Cheng about 7 years ago

after cherry-pick 7a602ec292b7e92c994d1718402e35a6c6692a02

there is another problem as following:

CMake Error at src/CMakeLists.txt:993 (add_library):
  Cannot find source file:

    journal/AsyncOpTracker.cc

  Tried extensions .c .C .c++ .cc .cpp .cxx .m .M .mm .h .hh .h++ .hm .hpp
  .hxx .in .txx

simply remove it

Actions #4

Updated by Wei-Chung Cheng about 7 years ago

  • Status changed from New to Fix Under Review
Actions #5

Updated by Nathan Cutler about 7 years ago

  • Backport set to jewel
Actions #6

Updated by Nathan Cutler about 7 years ago

  • Tracker changed from Bug to Backport
  • Description updated (diff)
  • Release set to jewel

description

in jewel, version(10.2.5)(8c87d09)

run `run_cmake_check` would meet error as follow:

CMake Error at src/CMakeLists.txt:970 (install):
  install TARGETS given no RUNTIME DESTINATION for executable target
  "mount.ceph".

CMake Error at src/CMakeLists.txt:980 (install):
  install PROGRAMS given no DESTINATION!

-- Configuring incomplete, errors occurred!

mount.ceph and mount.fuse.ceph do not know where they install to

Actions #7

Updated by Wei-Chung Cheng about 7 years ago

  • Subject changed from cmake: should define the install destination to jewel: cmake: should define the install destination
Actions #8

Updated by Nathan Cutler about 7 years ago

  • Status changed from Fix Under Review to Closed

Prefer to leave jewel cmake as it is, since it is not used.

Actions

Also available in: Atom PDF