Skip to content

Conversation

@Hadatko
Copy link
Member

@Hadatko Hadatko commented Feb 8, 2023

Signed-off-by: Cervenka Dusan [email protected]

Pull request

Choose Correct

  • bug
  • feature

Describe the pull request

Newer c++ allow us to build app without boost library dependency. This is simplifying dependencies and build time.

To Reproduce

Expected behavior

Screenshots

Desktop (please complete the following information):

  • OS:
  • eRPC Version:

Steps you didn't forgot to do

  • I checked if other PR isn't solving this issue.
  • I read Contribution details and did appropriate actions.
  • PR code is tested.
  • PR code is formatted.
  • Allow edits from maintainers pull request option is set (recommended).

Additional context

@Hadatko Hadatko added this to the 1.11.0 milestone Feb 8, 2023
@Hadatko Hadatko self-assigned this Feb 8, 2023
@Hadatko
Copy link
Member Author

Hadatko commented Feb 8, 2023

I need test compilation under VS

@Hadatko Hadatko linked an issue Feb 8, 2023 that may be closed by this pull request
Signed-off-by: Cervenka Dusan <[email protected]>
@Hadatko Hadatko force-pushed the feature/remove_boost_dependency branch from ecebb45 to b818c34 Compare February 8, 2023 11:26
@Hadatko
Copy link
Member Author

Hadatko commented Feb 8, 2023

@MichalPrincNXP compilation under VS works for me for debug and release

@Hadatko Hadatko mentioned this pull request Feb 8, 2023
6 tasks
Changing PlatformToolset to v142, as agreed
Copy link
Member

@MichalPrincNXP MichalPrincNXP left a comment

Choose a reason for hiding this comment

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

works fine on my side as well, PlatformToolset changed to v142 as agreed via email

@MichalPrincNXP MichalPrincNXP merged commit 236a68b into EmbeddedRPC:develop Feb 9, 2023
@MichalPrincNXP
Copy link
Member

Thank you for the effort!

@Hadatko Hadatko deleted the feature/remove_boost_dependency branch February 9, 2023 09:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

compile error with Mac Xcode 14

2 participants