[Communication] Move BaseSocketCommunication destructor to header to fix C++20 compilation

[loumalouomega]

📝 Description

Move BaseSocketCommunication destructor to header to fix C++20 compilation. See for example: https://github.com/KratosMultiphysics/Kratos/actions/runs/18677406599/job/53257969885?pr=13549

🆕 Changelog

• Move BaseSocketCommunication destructor to header to fix C++20 compilation

[matekelemen]

This really isn't necessary. The problem is that the template parameter is specified in the destructor's name as well (which has already been fixed).

[matekelemen]

The main issue is that I started an update to CoSim but didn't have time to finish it yet.

Check these WIP pull requests:

• [CoSim][External] Update CoSimIO Kratos#13374
• Set install directories depending on target type #374

[loumalouomega]

The main issue is that I started an update to CoSim but didn't have time to finish it yet.

Check these WIP pull requests:

* [[CoSim][External] Update CoSimIO Kratos#13374](https://github.com/KratosMultiphysics/Kratos/pull/13374)

* [Set install directories depending on target type #374](https://github.com/KratosMultiphysics/CoSimIO/pull/374)

Ah, OK, sorry for the noise

[loumalouomega]

Are you sure this is fixed? https://github.com/KratosMultiphysics/Kratos/actions/runs/18912407359/job/53987835109, or do we need to do something in Kratos side?

[loumalouomega]

Ah, Okay, is hardcopied, it is not a subdmoule. My bad.

[matekelemen]

I'd be all for submodules but your boss vehemently opposes the idea. None of Kratos' dependencies are submodules.