Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes to build with CMake #17

Closed
wants to merge 3 commits into from
Closed

Conversation

rexkerr
Copy link
Contributor

@rexkerr rexkerr commented Sep 24, 2015

  • Updated CMakeLists.txt files to match the current repository layout and current Jamfiles.
  • Updated unit tests to (consistently) not specify link configuration in the source, but to let the build configuration specify the variables.
  • Updated unit tests to properly import the main entry point for the tests.

@rexkerr
Copy link
Contributor Author

rexkerr commented Sep 24, 2015

FYI, not all tests are passing, but I didn't change any source, so I don't think I made anything worse:

Test project /Users/rexkerr/Development/boostache_build
Start 1: adapt_test
1/8 Test # 1: adapt_test ....................... Passed 0.01 sec
Start 2: grammar_basic
2/8 Test # 2: grammar_basic .................... Passed 0.01 sec
Start 3: render_traits
3/8 Test # 3: render_traits .................... Passed 0.00 sec
Start 4: traits
4/8 Test # 4: traits ........................... Passed 0.00 sec
Start 5: mustache_compiler
5/8 Test # 5: mustache_compiler ................_Failed 0.00 sec
Start 6: mustache_end2end
6/8 Test # 6: mustache_end2end .................
_Failed 0.00 sec
Start 7: mustache_parser
7/8 Test # 7: mustache_parser ..................***Failed 0.00 sec
Start 8: foreach
8/8 Test # 8: foreach .......................... Passed 0.00 sec

63% tests passed, 3 tests failed out of 8

@mjcaisse
Copy link
Member

Thanks Rex. Unfortunately, master was far out-of-date. I've taken care of getting master and develop back in sync. Please also make PR's against develop.

@mjcaisse mjcaisse closed this Sep 24, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants