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 GCC platform on some versions of GCC; Fix wiring/api tests for P2; Fix building of wiring/api tests on CI #2483

Merged
merged 8 commits into from
Jul 20, 2022

Conversation

avtolstoy
Copy link
Member

Description

As title says:

  1. Fixes builds of GCC platform under some versions of GCC
  2. Fixes building of wiring/api tests on CI
  3. Fixes wiring/api tests for P2
  4. Clean up weak serialEvent stuff and make sure unused serialEvent references are optimized out

Steps to Test

Wait for CI build.


Completeness

  • User is totes amazing for contributing!
  • Contributor has signed CLA (Info here)
  • Problem and Solution clearly stated
  • Run unit/integration/application tests on device
  • Added documentation
  • Added to CHANGELOG.md after merging (add links to docs and issues)

@avtolstoy avtolstoy added this to the 5.0.0 milestone Jul 20, 2022
@avtolstoy avtolstoy requested review from technobly and keeramis July 20, 2022 15:01
@avtolstoy avtolstoy requested a review from tstellanova July 20, 2022 15:42
… ProtocolFactory type is also declared in global namespace. Some versions of GCC fail to resolve this operator through ADL if it's defined in an anonymous namespace
…vent references are optimized out, fix builds for GCC platform on certain versions of GCC (sic!)
@avtolstoy avtolstoy merged commit 2e9ef58 into develop Jul 20, 2022
@avtolstoy avtolstoy deleted the fix/gcc-ci-tests branch July 20, 2022 18:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants