Skip to content

Commit

Permalink
Re-enable building a dist tarball from within distcheck
Browse files Browse the repository at this point in the history
  • Loading branch information
gjanssens committed Jan 29, 2018
1 parent 49a936c commit f319a8e
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 13 deletions.
7 changes: 0 additions & 7 deletions CMakeLists.txt
Expand Up @@ -781,10 +781,6 @@ endif()
dist_add_generated(ChangeLog)

############################ BEGIN MAKE DIST #################
# For now only enable the dist target when building from git
# the generation of dist_generated should be improved to work
# also when running from a tarball before that can be generalized.
IF (BUILDING_FROM_VCS)

SET(PACKAGE_PREFIX "${PACKAGE}-${PACKAGE_VERSION}")
SET(DIST_FILE "${PACKAGE_PREFIX}.tar")
Expand Down Expand Up @@ -847,9 +843,6 @@ ADD_CUSTOM_TARGET(distcheck DEPENDS dist
-P ${CMAKE_SOURCE_DIR}/common/cmake_modules/MakeDistCheck.cmake
)



ENDIF()
############################# END MAKE DIST #################

# uninstall target
Expand Down
12 changes: 6 additions & 6 deletions common/cmake_modules/MakeDistCheck.cmake
Expand Up @@ -76,12 +76,12 @@ FUNCTION(RUN_DIST_CHECK PACKAGE_PREFIX EXT)
ERROR_MSG "Ninja check failed."
)

# # Run ninja dist
# EXECUTE_PROCESS_AND_CHECK_RESULT(
# COMMAND ${MY_CMAKE_COMMAND} ${NINJA_COMMAND} dist
# WORKING_DIRECTORY ${BUILD_DIR}
# ERROR_MSG "Ninja dist failed."
# )
# Run ninja dist
EXECUTE_PROCESS_AND_CHECK_RESULT(
COMMAND ${MY_CMAKE_COMMAND} ${NINJA_COMMAND} dist
WORKING_DIRECTORY ${BUILD_DIR}
ERROR_MSG "Ninja dist failed."
)

MESSAGE("distcheck complete.")

Expand Down

0 comments on commit f319a8e

Please sign in to comment.