Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Multi-Routing-Managers in one Process (#603) #654

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

akhzarj
Copy link

@akhzarj akhzarj commented Mar 18, 2024

Author: Aram Khzarjyan Aram.Khzarjyan@mercedes-benz.com, MBition GmbH.

Fix Multi-Routing-Managers in one Process (#603) #654

Removing statics (singletons) enables to run
multiple routing Managers in one process.


The program was tested solely for our own use cases, which might differ from yours.

The submission is provided under the main project license (LICENSE file in root of project).

Provider Information

Removing statics (singletons) enables to run
multiple routing Managers in one process.

Co-authored-by: Aram Khzarjyan <aram.khzarjyan@mercedes-benz.com>
Co-authored-by: Kamil Poleszczuk <kes@spyro-soft.com>
Signed-off-by: Gunnar Andersson <gunnar.andersson@mercedes-benz.com>
@gunnar-mb
Copy link

PR looks good to me.

@akhzarj akhzarj marked this pull request as ready for review March 22, 2024 15:53
@gpb76
Copy link

gpb76 commented Mar 26, 2024

PR looks good to me.

The fix of #239 missing from release 3.3.0 onwards. Missing merge?

Copy link
Contributor

@007herelsp 007herelsp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@fayed23
Copy link

fayed23 commented Jun 27, 2024

will this fix #566 ?

@akhzarj
Copy link
Author

akhzarj commented Jul 2, 2024

Hi @fayed23
Your problem looks similar that we have faced and fixed.
To be sure you need to take this fix and test on your setup.
Please let to know the outcome.

@zhaoxin39913
Copy link

zhaoxin39913 commented Oct 22, 2024

Hi @akhzarj,

Our SW would like to involve vsomeip libary 3.5.1, can you confirm if this fix is included in 3.5.1 release ? Otherwise I still need to apply the patch manually.

@fcmonteiro
Copy link
Collaborator

Hello, still not in 3.5.1
We are finishing some tests with some extra PRs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants