glest/glest-source

output library should be renamed

Closed this issue · 5 comments

The name of the library should just be "libzetaglest.a" not "liblib..."

[ 42%] Building CXX object source/shared_lib/CMakeFiles/libzetaglest.dir/sources/platform/unix/gl_wrap.cpp.o
[ 43%] Linking CXX static library liblibzetaglest.a

hi @mvilkin

Conflict? Oh I meant for the cmake file to be edited, so the library would have different name upon output.

Yep, but you will have two targets with the same names.

Absolutely. Thx for excellent full explanation!

@mathusummut that's a pretty fair assessment, but though things can sometimes appear automatic, there's almost always a way to override.

Logical target names must be globally unique.

Targets names created with add_executable, add_library, or add_custom_target are logical build target names. Logical target names must be globally unique because:

https://cmake.org/cmake/help/v3.7/policy/CMP0002.html?highlight=cmp0002

But this is how it's changed

set_target_properties

Targets can have properties that affect how they are built.

And this is the patch (but let's wait until after the build is fixed to merge it)

https://github.com/ZetaGlest/zetaglest-source/pull/277/files