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

Rewire cmake internals for cmake's target_source() #219

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 2 additions & 4 deletions .travis.yml
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
dist: xenial
language: cpp
compiler: gcc
os:
Expand All @@ -8,15 +9,12 @@ before_install:

install:
- echo $TRAVIS_OS_NAME
- sudo add-apt-repository -y ppa:ubuntu-toolchain-r/test
- sudo apt-get update -qq
- sudo apt-get install -qq g++-4.8
- sudo update-alternatives --install /usr/bin/g++ g++ /usr/bin/g++-4.8 50
- sudo apt-get install -qq cmake
- sudo apt-get install -qq ccache

script:
- echo $TRAVIS_OS_NAME
- cmake --version
- git --version
- python --version
# test fips command itself
Expand Down
14 changes: 14 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,20 @@ http://floooh.github.io/fips/getstarted.html

### Public Service Announcements

- **08-Feb-2019**:
- **internal cmake script cleanup:** fips is now creating "empty" targets in the
```fips_begin_*()``` calls and uses modern (post 3.0) commands to add
source files, dependencies and options to the targets (previously those
were gathered in custom variables, and creating the actual targets happend
in the ```fips_end_*()``` functions). This simplifies the fips cmake
scripts a lot, and it allows better mixing of native cmake commands with
fips commands (e.g. you can now use commands which modify cmake target
properties **inside** the fips begin/end blocks, previously such
commands had to be added *after* the ```fips_end_*```. Unfortunately the
changes break the Oryol unit testing cmake macros (which depend on internal
implementation details, which obviously is a bit naughty), so you need
to update fips and Oryol at the same time.

- **20-Jan-2019**:
- NaCl and UWP support have been removed (both haven't been maintained for a very long time)
- remove the builtin unittest support, this was hardwired to UnitTest++ and should
Expand Down
Loading