Skip to content
This repository has been archived by the owner on Sep 26, 2018. It is now read-only.

Commit

Permalink
* Fix header install step with new include sub-directory
Browse files Browse the repository at this point in the history
This commit also updates the .travis.yml to run `make install` instead
of just `make` so it'll catch this kind of error in the future. The
files are just installed to /tmp on the travis machine.
  • Loading branch information
Kevin Greene committed Jul 2, 2018
1 parent bb2ad7f commit 4a6f5bf
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 3 deletions.
8 changes: 6 additions & 2 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,10 @@ notifications:
os: linux
dist: trusty

env:
global:
- TMPDIR=/tmp

matrix:
include:

Expand Down Expand Up @@ -162,8 +166,8 @@ matrix:
script:
- cd "${TRAVIS_BUILD_DIR}"
- mkdir build && cd build
- cmake .. -DCMAKE_CXX_COMPILER=$COMPILER -DCMAKE_BUILD_TYPE=$BUILD_TYPE
- make
- cmake .. -DCMAKE_CXX_COMPILER=$COMPILER -DCMAKE_BUILD_TYPE=$BUILD_TYPE -DCMAKE_INSTALL_PREFIX=$TMPDIR/hfsm
- make install

- cd "${TRAVIS_BUILD_DIR}/examples/advanced_event_handling"
- mkdir build && cd build
Expand Down
2 changes: 1 addition & 1 deletion CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ install(

# Install hfsm headers.
install(
DIRECTORY "${PROJECT_NAME}"
DIRECTORY "include/${PROJECT_NAME}"
DESTINATION "${include_install_dir}"
)

Expand Down

0 comments on commit 4a6f5bf

Please sign in to comment.