CMake fixes

[kgerheiser]

I would test this out. The changes are simple, but I made these changes quickly and didn't have the various libraries to build with.

Link to target instead of .a

Make shared_all's headers public

Use target_compile_definitions

Use the target's name directly. Making it a variable is kind of redundant and doesn't really save anything. We did that in some of the other builds because we looped over target names and expanded them in code.

Prefer target_include_directories rather than include_directories. Working with targets instead of global function is generally preferred.

[GeorgeGayno-NOAA]

Thanks for your help. I will try this.