Fix Debug build when Azure Presigned and Curl managers are disabled (!927) · Merge requests · OSDU / OSDU Data Platform / Domain Data Management Services / Seismic / Open VDS

Merged Alena Chaikouskaya requested to merge feature/achaikou/lto_debug_fix into master 4 months ago

Dec 30, 2024

Update DISABLE_DMS_IOMANAGER to disable compilation of all files used by DMS. · e9e2eed7
Morten Ofstad authored 3 months ago

e9e2eed7

Make sure we include the Curl backend if needed by any enabled IOManager.... · b65e417b

Morten Ofstad authored 3 months ago

Make sure we include the Curl backend if needed by any enabled IOManager. Change the DISABLE_CURL_IOMANAGER to effectively work like 'DISABLE_HTTP_IOMANAGER' which is what it really should have been from the start.

b65e417b

Add missing AzurePresigned_IOManager guard · e2383d7f
Alena Chaikouskaya authored 5 months ago and Morten Ofstad committed 3 months ago

e2383d7f

Fix compilation with disabled Curl/Azure Presigned · 2ddad8df

Alena Chaikouskaya authored 5 months ago and

Morten Ofstad committed 3 months ago

When DISABLE_AZURE_PRESIGNED_IOMANAGER / DISABLE_CURL_IOMANAGER are set
to on, Debug build fails while Release builds fine.

This seems to be caused by LTO being enabled by default for Release
builds and disabled for Debug ones.

The actual issue is that additional source files have to be
marked as "header file only" for linking to succeed.

2ddad8df

Extend conditions to disable IOManagerCurl · 006ee2b2

Alena Chaikouskaya authored 4 months ago and

Morten Ofstad committed 3 months ago

AWS compilation and compilation of IOManagerDmsFactory started to depend
on IOManagerCurl after DMS Factory code introduction, but the setting
was not updated.

Also the description is updated to better reflect purpose of the
setting.

006ee2b2

Fix Debug build when Azure Presigned and Curl managers are disabled

Merge request reports

Activity