Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
This reverts commit 7d3c8ef.
  • Loading branch information
steveloughran committed Feb 17, 2023
1 parent 7e19bc3 commit 10e7ca4
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 29 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,6 @@
<!-- Basedir eeded for generating FindBugs warnings using parent pom -->
<yarn.basedir>${project.parent.parent.basedir}</yarn.basedir>
<container-executor.conf.dir>../etc/hadoop</container-executor.conf.dir>
<extra.libhadoop.rpath>../lib/native</extra.libhadoop.rpath>
<container-executor.additional_cflags></container-executor.additional_cflags>
</properties>

Expand Down Expand Up @@ -224,7 +223,6 @@
<source>${basedir}/src</source>
<vars>
<HADOOP_CONF_DIR>${container-executor.conf.dir}</HADOOP_CONF_DIR>
<EXTRA_LIBHADOOP_RPATH>${extra.libhadoop.rpath}</EXTRA_LIBHADOOP_RPATH>
<JVM_ARCH_DATA_MODEL>${sun.arch.data.model}</JVM_ARCH_DATA_MODEL>
</vars>
<env>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -25,8 +25,6 @@ set(GTEST_SRC_DIR ${CMAKE_SOURCE_DIR}/../../../../../hadoop-common-project/hadoo

set(HADOOP_COMMON_SEC_PATH ${HADOOP_COMMON_PATH}/src/main/native/src/org/apache/hadoop/security)

set(CMAKE_BUILD_WITH_INSTALL_RPATH TRUE)

# determine if container-executor.conf.dir is an absolute
# path in case the OS we're compiling on doesn't have
# a hook in get_executable. We'll use this define
Expand Down Expand Up @@ -159,18 +157,6 @@ add_executable(container-executor
main/native/container-executor/impl/main.c
)

# By embedding '$ORIGIN' into the RPATH of container-executor, dlopen will look in
# the directory containing container-executor. However, $ORIGIN is not supported by
# all operating systems.
if(${CMAKE_SYSTEM_NAME} MATCHES "Linux|SunOS")
set(RPATH "\$ORIGIN/")
if(EXTRA_LIBHADOOP_RPATH)
set(RPATH "${RPATH}:${EXTRA_LIBHADOOP_RPATH}/")
endif()
message("RPATH SET AS ${RPATH}.")
set_target_properties(container-executor PROPERTIES INSTALL_RPATH "${RPATH}")
endif()

target_link_libraries(container-executor
container
crypto
Expand All @@ -182,19 +168,6 @@ output_directory(container-executor target/usr/local/bin)
add_executable(test-container-executor
main/native/container-executor/test/test-container-executor.c
)

# By embedding '$ORIGIN' into the RPATH of test-container-executor, dlopen will look in
# the directory containing test-container-executor. However, $ORIGIN is not supported by
# all operating systems.
if(${CMAKE_SYSTEM_NAME} MATCHES "Linux|SunOS")
set(RPATH "\$ORIGIN/")
if(EXTRA_LIBHADOOP_RPATH)
set(RPATH "${RPATH}:${EXTRA_LIBHADOOP_RPATH}/")
endif()
message("RPATH SET AS ${RPATH}.")
set_target_properties(test-container-executor PROPERTIES INSTALL_RPATH "${RPATH}")
endif()

target_link_libraries(test-container-executor
container
${EXTRA_LIBS}
Expand Down

0 comments on commit 10e7ca4

Please # to comment.