Matthew Russell

Results 75 comments of Matthew Russell

@duartenfonseca I'll take a look! Was this on Fabio's 3.5.x? He did push a change to the network tests a few days back. I'll also fix that conflict that appeared.

> @kheaactua is the increase of minimum version on cmake is really necessary? from our side, we would like to avoid that I raised it to 3.15 to accommodate Conan...

> > > @kheaactua is the increase of minimum version on cmake is really necessary? from our side, we would like to avoid that > > > > > >...

The build error is weird.. Though I do get a similar one locally: ``` Run on (40 X 4800 MHz CPU s) CPU Caches: L1 Data 48 KiB (x20) L1...

> Hi @kheaactua Please find our contacts related to Co-Authority: > > * Aram Khzarjyan [[email protected]](mailto:[email protected]) > > * Patnaik, Sambit [[email protected]](mailto:[email protected]) Got it! Thanks! ![image](https://github.com/user-attachments/assets/e734ddb5-025c-40e1-94e2-41e08f49ca93)

This seems identical to https://github.com/COVESA/capicxx-core-runtime/pull/49 .. I should have checked before posting

I love that a clang-format file is being added! Might I also propose a `.cmake-format` file?

These weren't touched in this PR, but Aram's team found several bad reference captures that should probably be addressed in this PR if not a separate PR. I can't upload...

hey @fcmonteiro I get build errors on gcc (11.4) and clang (18.1.8): ``` implementation/runtime/src/application_impl.cpp:1513:46: error: could not convert ‘vsomeip_v3::configuration::get_policy_manager()’ from ‘shared_ptr’ to ‘shared_ptr’ 1513 | return configuration_->get_policy_manager(); | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~ |...

> @kheaactua could you try it now, does it work correctly? hey @fcmonteiro, it appears to work, I just built this with gcc and clang with `-DVSOMEIP_DISABLE_SECURITY` Even the `build_tests`...