duckstax/actor-zeta

Change the mechanics of the build target.

kotbegemot opened this issue · 7 comments

kotbegemot commented

now after building the project two targets: actor-zetta-core , actor-zeta-io .
There should be one target actor-zeta.
Add checkboxes that tell cmake which components to include in the final build of the library.
By default, all library components are enabled.

Add build options:

  • shared = off
  • rtti = off
  • exception = off
  • CMake_RUN_CLANG_TIDY = off
  • example = off
  • dev mode = off
  • test = off
petrovigor commented

IO can't build itself without CORE, so I'll remove CORE check, leave IO option. Also I'll add STATIC/SHARED option for the actor-zeta target.

kotbegemot commented

check box : IO,STATIC/SHARED
@petrovigor you want so ?

petrovigor commented

Yes, let user chose STATIC/SHARED build.
And only two additional functionalities for "IO" and "EXAMPLES".

petrovigor commented

It's done. See for pull request #23.

kotbegemot commented

@petrovigor

  1. Why was the replacement?
    list(APPEND -> set(
  2. Why can't I use a multi-target system?
petrovigor commented
  1. Ok. I got it wrong. going to code
  2. Let's discuss that difference (set/list). I think 'set' works faster here.
petrovigor commented

Okay, please review and ask questions. #23