Link internal library as CMake object library

[sksat]

Related issues

N/A

Description

s2e-core-internal libraries like COMPONENT have complex (and cyclic) dependencies with each other, since they are only for build S2E.

However, these libraries are built as static libraries.
This causes the final executable link to not succeed unless non-trivial link configuration is performed many times.

s2e-core/CMakeLists.txt

Lines 185 to 198 in 6ca93ed

	target_link_libraries(COMPONENT DYNAMICS GLOBAL_ENVIRONMENT LOCAL_ENVIRONMENT MATH_PHYSICS SETTING_FILE_READER LOGGER UTILITIES)
	target_link_libraries(DYNAMICS GLOBAL_ENVIRONMENT LOCAL_ENVIRONMENT SIMULATION MATH_PHYSICS)
	target_link_libraries(DISTURBANCE DYNAMICS GLOBAL_ENVIRONMENT LOCAL_ENVIRONMENT MATH_PHYSICS)
	target_link_libraries(SIMULATION DYNAMICS GLOBAL_ENVIRONMENT LOCAL_ENVIRONMENT DISTURBANCE MATH_PHYSICS)
	target_link_libraries(GLOBAL_ENVIRONMENT ${CSPICE_LIB} MATH_PHYSICS)
	target_link_libraries(LOCAL_ENVIRONMENT GLOBAL_ENVIRONMENT ${CSPICE_LIB} MATH_PHYSICS)
	target_link_libraries(MATH_PHYSICS ${NRLMSISE00_LIB})
	target_link_libraries(SETTING_FILE_READER INIH)
	target_link_libraries(${PROJECT_NAME} DYNAMICS)
	target_link_libraries(${PROJECT_NAME} DISTURBANCE)
	target_link_libraries(${PROJECT_NAME} SIMULATION)
	target_link_libraries(${PROJECT_NAME} GLOBAL_ENVIRONMENT LOCAL_ENVIRONMENT)
	target_link_libraries(${PROJECT_NAME} COMPONENT)

This PR change these libraries to be built as object libraries feature provided by CMake.
It eliminates the need to specify dependencies mid-build by allowing circular dependencies to exist until the final executable link.

s2e-core/CMakeLists.txt

Lines 183 to 200 in 7f8ae7c

	target_link_libraries(${PROJECT_NAME}
	LOGGER
	SETTING_FILE_READER
	INIH
	UTILITIES
	COMPONENT
	DISTURBANCE
	DYNAMICS
	GLOBAL_ENVIRONMENT
	LOCAL_ENVIRONMENT
	MATH_PHYSICS
	SIMULATION
	# ExtLibraries
	${NRLMSISE00_LIB}
	${CSPICE_LIB}
	)

Test results

N/A

Impact

The build process is a bit changed; S2E users also need to make a similar modification to this PR in their CMakeLists.txt.

Supplementary information

This simplifies the build process and makes it easier to expand use cases, such as interoperation with Rust.

[sksat]

Win, 32bit, C2A=ON の時だけこの変更で落ちるのはあまりに変なので、Actions の minor update などを疑って #711 をやったが、違いそう

[sksat]

rebase

[sksat]

64bit で USE_C2A=ONの時、CI はパスしてるけどビルドがスキップされてる

[sksat]

#409 で変更されてるな

[sksat]

そもそも 64bit ビルドが追加されたのがここか

[sksat]

単純に libC2A をリンクしようとした時にリンクエラーになる、のであればまだ make sense。しかし、SILS-S2E の際は 32bit ビルドを強制させたい、のであれば後出しジャンケン的に無かったことにするべきではない。

[200km]

しかし、SILS-S2E の際は 32bit ビルドを強制させたい、のであれば後出しジャンケン的に無かったことにするべきではない。

そうですね。これはこれ以外のやり方がいまいちわからなかったので無理やりやったという感じです。
スマートな感じに修正していただけると嬉しいです！

[sksat]

@200km #712 で、include という機能を使って存在する組み合わせをベタ書きするようにしてみました

[sksat]

rebase

[sksat]

Visual Studio のリンカ（link.exe）の挙動が微妙だったためやや dirty hack も入っているが、通った（ここらへんは native Windows のサポートを続けるかどうかも含めて考えたい）

[sksat]

@200km いつもの摩訶不思議なことになってしまっている target_link_libraries() をシンプルにするパッチです。ビルドがシンプルになると AE 側の Rust の資産との接続が行いやすくなるといったメリットもあるため、マージできるとうれしいです。

[sksat]

FYI: @suzuki-toshihir0 @tarotene

[200km]

修正提案ありがとうございます！